From 4a275ea53c8bedcabe30396d7ce88a06650568b1 Mon Sep 17 00:00:00 2001
From: Brad Jorsch <bjorsch@wikimedia.org>
Date: Wed, 14 Feb 2018 13:01:19 -0500
Subject: [PATCH] Don't write exceptions to STDERR from BadTitleErrorTest or
 ThrottledErrorTest

It's annoying and pointless. Instead, have MWException write them to
standard output where we can catch them with ob_start().

Bug: T170028
Bug: T170029
Change-Id: Icd99c1c39d4a30d78c511d33948ef639e1b92455
---
 includes/exception/MWException.php                      | 2 +-
 tests/phpunit/includes/exception/BadTitleErrorTest.php  | 4 +++-
 tests/phpunit/includes/exception/ThrottledErrorTest.php | 4 +++-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/includes/exception/MWException.php b/includes/exception/MWException.php
index 16f226c78f..b3e9422b8d 100644
--- a/includes/exception/MWException.php
+++ b/includes/exception/MWException.php
@@ -189,7 +189,7 @@ class MWException extends Exception {
 		} elseif ( self::isCommandLine() ) {
 			$message = $this->getText();
 			// T17602: STDERR may not be available
-			if ( defined( 'STDERR' ) ) {
+			if ( !defined( 'MW_PHPUNIT_TEST' ) && defined( 'STDERR' ) ) {
 				fwrite( STDERR, $message );
 			} else {
 				echo $message;
diff --git a/tests/phpunit/includes/exception/BadTitleErrorTest.php b/tests/phpunit/includes/exception/BadTitleErrorTest.php
index dcdb1cd198..b706facedc 100644
--- a/tests/phpunit/includes/exception/BadTitleErrorTest.php
+++ b/tests/phpunit/includes/exception/BadTitleErrorTest.php
@@ -10,8 +10,10 @@ class BadTitleErrorTest extends MediaWikiTestCase {
 		try {
 			throw new BadTitleError();
 		} catch ( BadTitleError $e ) {
+			ob_start();
 			$e->report();
-			$this->assertTrue( true );
+			$text = ob_get_clean();
+			$this->assertContains( $e->getText(), $text );
 		}
 	}
 
diff --git a/tests/phpunit/includes/exception/ThrottledErrorTest.php b/tests/phpunit/includes/exception/ThrottledErrorTest.php
index 15f08966dc..5214b6d4d1 100644
--- a/tests/phpunit/includes/exception/ThrottledErrorTest.php
+++ b/tests/phpunit/includes/exception/ThrottledErrorTest.php
@@ -11,8 +11,10 @@ class ThrottledErrorTest extends MediaWikiTestCase {
 		try {
 			throw new ThrottledError();
 		} catch ( ThrottledError $e ) {
+			ob_start();
 			$e->report();
-			$this->assertTrue( true );
+			$text = ob_get_clean();
+			$this->assertContains( $e->getText(), $text );
 		}
 	}
 
-- 
2.20.1