Fixes sebastianbergmann#573, Fixes sebastianbergmann#552, Fixes sebastianbergmann#582.

[BC BREAK]

All relative paths in a configuration file are now resolved relative to that
configuration file. When upgrading, you may need to update relative paths for
the following configurations:

 * `testSuiteLoaderFile`
 * `printerFile`
 * `testsuites/file`
 * `testsuites/exclude`

Author: whatthejeff

ChangeLog.md

@@ -23,6 +23,7 @@ PHPUnit 3.8.0
 * Implemented #877: Added new HTML5 tags to `PHPUnit_Util_XML::findNodes()`.
 * Added `--coverage-crap4j` switch to generate code coverage report in Crap4J XML format.
 * A test will now fail in strict mode when it uses the `@covers` annotation and code that is not expected to be covered is executed.
+* All relative paths in a configuration file are now resolved relative to that configuration file. When upgrading, you may need to update relative paths for the following configurations: `testSuiteLoaderFile`, `printerFile`, `testsuites/file`, `testsuites/exclude`.
 * Fixed #240: XML strings are escaped by removing invalid characters.
 * Fixed #261: `setUp()` and `setUpBeforeClass()` are run before filters are applied.
 * Fixed #541: Excluded groups are counted towards total number of tests being executed.

PHPUnit/Util/Configuration.php

@@ -680,8 +680,8 @@ public function getPHPUnitConfiguration()
         }
 
         if ($root->hasAttribute('testSuiteLoaderFile')) {
-            $result['testSuiteLoaderFile'] = (string)$root->getAttribute(
-              'testSuiteLoaderFile'
+            $result['testSuiteLoaderFile'] = $this->toAbsolutePath(
+              (string)$root->getAttribute('testSuiteLoaderFile')
             );
         }
 
@@ -692,8 +692,8 @@ public function getPHPUnitConfiguration()
         }
 
         if ($root->hasAttribute('printerFile')) {
-            $result['printerFile'] = (string)$root->getAttribute(
-              'printerFile'
+            $result['printerFile'] = $this->toAbsolutePath(
+              (string)$root->getAttribute('printerFile')
             );
         }
 
@@ -827,7 +827,9 @@ protected function getTestSuite(DOMElement $testSuiteNode, $testSuiteFilter=null
         $exclude = array();
 
         foreach ($testSuiteNode->getElementsByTagName('exclude') as $excludeNode) {
-            $exclude[] = (string)$excludeNode->nodeValue;
+            $exclude[] = $this->toAbsolutePath(
+              (string)$excludeNode->nodeValue
+            );
         }
 
         $fileIteratorFacade = new File_Iterator_Facade;
@@ -892,7 +894,9 @@ protected function getTestSuite(DOMElement $testSuiteNode, $testSuiteFilter=null
             }
 
             // Get the absolute path to the file
-            $file = $fileIteratorFacade->getFilesAsArray($file);
+            $file = $fileIteratorFacade->getFilesAsArray(
+              $this->toAbsolutePath($file)
+            );
 
             if (!isset($file[0])) {
                 continue;

Tests/_files/configuration.custom-printer.xml

@@ -1,2 +1,2 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit printerFile="Tests/_files/CustomPrinter.php" printerClass="CustomPrinter" />
+<phpunit printerFile="CustomPrinter.php" printerClass="CustomPrinter" />