From 8269544f2fcc609e127e2cfcc64ae562003f1d68 Mon Sep 17 00:00:00 2001
From: Andrew Nicols <andrew@nicols.co.uk>
Date: Tue, 20 Mar 2018 08:08:29 +0800
Subject: [PATCH] MDL-61689 phpunit: Be stricter about output during unit tests

This change also add fail-on-risky to travis to help pick these up, and
verbosity too.

We only fail on risky during Travis and not during normal output because
developers may want to debug tests.
---
 .travis.yml      | 2 +-
 phpunit.xml.dist | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.travis.yml b/.travis.yml
index bf3be73b95f..14888ec3d7b 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -249,7 +249,7 @@ script:
     - >
       if [ "$TASK" = 'PHPUNIT' ];
       then
-        vendor/bin/phpunit;
+        vendor/bin/phpunit --fail-on-risky --disallow-test-output --verbose;
       fi
 
     - >
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
index 7d801485dfa..43349a16837 100644
--- a/phpunit.xml.dist
+++ b/phpunit.xml.dist
@@ -13,6 +13,7 @@
         stopOnFailure="false"
         stopOnIncomplete="false"
         stopOnSkipped="false"
+        beStrictAboutOutputDuringTests="true"
         printerClass="Hint_ResultPrinter"
         testSuiteLoaderClass="phpunit_autoloader"
         >