From 3f7966c4030934d8043c9a191b649d9d840629d7 Mon Sep 17 00:00:00 2001
From: Robert Sesek <rsesek@bluestatic.org>
Date: Sat, 3 Dec 2016 23:29:14 -0500
Subject: [PATCH] Upgrade PHPUnit to 5.6.7.

---
 testing/{test_runner.php => bootstrap.php} | 26 +++-------------------
 testing/phpunit.xml                        | 19 +++++++++++-----
 testing/test_listener.php                  | 11 ++++++++-
 3 files changed, 27 insertions(+), 29 deletions(-)
 rename testing/{test_runner.php => bootstrap.php} (62%)

diff --git a/testing/test_runner.php b/testing/bootstrap.php
similarity index 62%
rename from testing/test_runner.php
rename to testing/bootstrap.php
index 0be30f8..eb090e9 100644
--- a/testing/test_runner.php
+++ b/testing/bootstrap.php
@@ -1,11 +1,11 @@
 <?php
 // Hoplite
-// Copyright (c) 2011 Blue Static
-// 
+// Copyright (c) 2016 Blue Static
+//
 // This program is free software: you can redistribute it and/or modify it
 // under the terms of the GNU General Public License as published by the Free
 // Software Foundation, either version 3 of the License, or any later version.
-// 
+//
 // This program is distributed in the hope that it will be useful, but WITHOUT
 // ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
 // FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
@@ -20,23 +20,3 @@ if (!defined('HOPLITE_ROOT')) {
     define('HOPLITE_ROOT', dirname(dirname(__FILE__)));
     define('TEST_ROOT', dirname(__FILE__));
 }
-
-// PHPUnit 3.5.5.
-require_once 'PHPUnit/Autoload.php';
-require_once TEST_ROOT . '/test_listener.php';
-
-class HopliteTestRunner extends \PHPUnit_TextUI_Command
-{
-    static public function Main($exit = TRUE)
-    {
-        $command = new self();
-        $command->Run($_SERVER['argv'], $exit);
-    }
-
-    protected function HandleCustomTestSuite()
-    {
-        $this->arguments['printer'] = new TestListener();
-    }
-}
-
-HopliteTestRunner::Main();
diff --git a/testing/phpunit.xml b/testing/phpunit.xml
index fc4de5b..5a630a1 100644
--- a/testing/phpunit.xml
+++ b/testing/phpunit.xml
@@ -1,24 +1,28 @@
 <!--
   Hoplite
   Copyright (c) 2011 Blue Static
-  
+
   This program is free software: you can redistribute it and/or modify it
   under the terms of the GNU General Public License as published by the Free
   Software Foundation, either version 3 of the License, or any later version.
-  
+
   This program is distributed in the hope that it will be useful, but WITHOUT
   ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
   FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
   more details.
- 
+
   You should have received a copy of the GNU General Public License along with
   this program.  If not, see <http://www.gnu.org/licenses/>.
 -->
 
-<!-- Tested with PHPUnit 3.5.5 -->
+<!-- Tested with PHPUnit 5.6.7 -->
 <phpunit backupGlobals="FALSE"
          backupStaticAttributes="FALSE"
-         colors="TRUE">
+         colors="TRUE"
+         bootstrap="bootstrap.php"
+         printerFile="test_listener.php"
+         printerClass="hoplite\test\TestListener"
+         verbose="TRUE">
 
     <testsuites>
         <testsuite name="Hoplite Unit Tests">
@@ -27,12 +31,17 @@
     </testsuites>
 
     <filter>
+        <whitelist>
+            <directory suffix=".php">../</directory>
+        </whitelist>
         <blacklist>
             <directory>./</directory>  <!-- Don't get coverage for test files. -->
         </blacklist>
     </filter>
 
+    <!--
     <logging>
         <log type="coverage-html" target="./unittest_coverage" yui="TRUE" highlight="TRUE"/>
     </logging>
+    -->
 </phpunit>
diff --git a/testing/test_listener.php b/testing/test_listener.php
index 1c65d93..1504d5a 100644
--- a/testing/test_listener.php
+++ b/testing/test_listener.php
@@ -48,6 +48,7 @@ class TestListener extends \PHPUnit_Util_Printer implements \PHPUnit_Framework_T
 
     // Array of incomplete tests.
     private $incomplete = array();
+    private $risky = array();
 
     // An error occurred.
     public function addError(\PHPUnit_Framework_Test $test,
@@ -70,7 +71,7 @@ class TestListener extends \PHPUnit_Util_Printer implements \PHPUnit_Framework_T
         $this->_Print('  ', $e->GetMessage());
         if ($e instanceof \PHPUnit_Framework_ExpectationFailedException) {
             $comp = $e->GetComparisonFailure();
-            if ($comp instanceof \PHPUnit_Framework_ComparisonFailure) {
+            if ($comp) {
                 $this->_Print(' ==> ', $comp->GetExpectedAsString());
                 $this->_Print('', 'does not match');
                 $this->_Print(' ==> ', $comp->GetActualAsString());
@@ -81,6 +82,14 @@ class TestListener extends \PHPUnit_Util_Printer implements \PHPUnit_Framework_T
         $this->failing[] = $test->ToString();
     }
 
+    // Risky test.
+    public function addRiskyTest(\PHPUnit_Framework_Test $test,
+                                 \Exception $e, $time)
+    {
+        $this->risky[] = $test->ToString();
+        $this->_Print('RISKY', $e->GetMessage(), self::COLOR_PURPLE);
+    }
+
     // Incomplete test.
     public function addIncompleteTest(\PHPUnit_Framework_Test $test,
                                       \Exception $e, $time)
-- 
2.22.5