Do not require PHP 5.3.

Author: sebastianbergmann

Tests/PHP/CodeCoverage/FilterTest.php

@@ -101,9 +101,10 @@ protected function setUp()
           TEST_FILES_PATH . 'NamespaceCoveragePublicTest.php',
           TEST_FILES_PATH . 'NamespaceCoveredClass.php',
           TEST_FILES_PATH . 'NotExistingCoveredElementTest.php',
-          TEST_FILES_PATH . 'source.php',
           TEST_FILES_PATH . 'source_with_ignore.php',
-          TEST_FILES_PATH . 'source_without_ignore.php'
+          TEST_FILES_PATH . 'source_with_namespace.php',
+          TEST_FILES_PATH . 'source_without_ignore.php',
+          TEST_FILES_PATH . 'source_without_namespace.php'
         );
     }
 

Tests/PHP/CodeCoverage/UtilTest.php

@@ -125,13 +125,36 @@ public function testBuildDirectoryStructure()
      */
     public function testCountLines()
     {
+        $this->assertEquals(
+          array(
+            'loc' => 18,
+            'cloc' => 6,
+            'ncloc' => 12
+          ),
+          PHP_CodeCoverage_Util::countLines(
+            TEST_FILES_PATH . 'source_without_namespace.php'
+          )
+        );
+    }
+
+    /**
+     * @covers PHP_CodeCoverage_Util::countLines
+     */
+    public function testCountLines2()
+    {
+        if (!version_compare(PHP_VERSION, '5.3.0', '>')) {
+            $this->markTestSkipped('PHP 5.3 (or later) is required.');
+        }
+
         $this->assertEquals(
           array(
             'loc' => 20,
             'cloc' => 6,
             'ncloc' => 14
           ),
-          PHP_CodeCoverage_Util::countLines(TEST_FILES_PATH . 'source.php')
+          PHP_CodeCoverage_Util::countLines(
+            TEST_FILES_PATH . 'source_with_namespace.php'
+          )
         );
     }
 
@@ -202,6 +225,35 @@ public function testGetClassesInFile()
      */
     public function testGetFunctionsInFile()
     {
+        $this->assertEquals(
+          array(
+            'foo' => array (
+              'docComment' => '/**
+ * @param mixed $bar
+ */',
+              'signature' => 'function &foo($bar)',
+              'startLine' => 12,
+              'endLine' => 18,
+              'ccn' => 2
+            )
+          ),
+          PHP_CodeCoverage_Util::getFunctionsInFile(
+            TEST_FILES_PATH . 'source_without_namespace.php',
+            FALSE
+          )
+        );
+    }
+
+    /**
+     * @covers PHP_CodeCoverage_Util::getFunctionsInFile
+     * @covers PHP_CodeCoverage_Util::parseFile
+     */
+    public function testGetFunctionsInFile2()
+    {
+        if (!version_compare(PHP_VERSION, '5.3.0', '>')) {
+            $this->markTestSkipped('PHP 5.3 (or later) is required.');
+        }
+
         $this->assertEquals(
           array(
             'bar\\baz\\foo' => array (
@@ -215,7 +267,7 @@ public function testGetFunctionsInFile()
             )
           ),
           PHP_CodeCoverage_Util::getFunctionsInFile(
-            TEST_FILES_PATH . 'source.php',
+            TEST_FILES_PATH . 'source_with_namespace.php',
             FALSE
           )
         );

Tests/_files/source.php Tests/_files/source_with_namespace.phpTests/_files/source.php renamed to Tests/_files/source_with_namespace.php

@@ -0,0 +1,18 @@
+<?php
+/**
+ * Represents foo.
+ */
+class foo
+{
+}
+
+/**
+ * @param mixed $bar
+ */
+function &foo($bar)
+{
+    $baz = function() {};
+    $a   = TRUE ? TRUE : FALSE;
+    $b = "{$a}";
+    $c = "${b}";
+}

Tests/_files/source_without_namespace.php

@@ -144,7 +144,7 @@
  <dependencies>
   <required>
    <php>
-    <min>5.3.0</min>
+    <min>5.2.7</min>
    </php>
    <pearinstaller>
     <min>1.8.1</min>