Refactor.

Author: sebastianbergmann

Tests/PHP/CodeCoverageTest.php

@@ -106,27 +106,6 @@ public function testConstructor2()
      */
     public function testCollect()
     {
-        $coverage = new PHP_CodeCoverage($this->setUpXdebugStub());
-
-        $coverage->start(
-          new BankAccountTest('testBalanceIsInitiallyZero'), TRUE
-        );
-        $coverage->stop();
-
-        $coverage->start(new BankAccountTest(
-          'testBalanceCannotBecomeNegative')
-        );
-        $coverage->stop();
-
-        $coverage->start(
-          new BankAccountTest('testBalanceCannotBecomeNegative2')
-        );
-        $coverage->stop();
-
-        $coverage->start(
-          new BankAccountTest('testDepositWithdrawMoney')
-        );
-        $coverage->stop();
 
         $this->assertAttributeEquals(
           array(
@@ -191,17 +170,14 @@ public function testCollect()
             )
           ),
           'data',
-          $coverage
+          $this->getBankAccountCoverage()
         );
-
-        return $coverage;
     }
 
     /**
-     * @covers  PHP_CodeCoverage::getSummary
-     * @depends testCollect
+     * @covers PHP_CodeCoverage::getSummary
      */
-    public function testSummary(PHP_CodeCoverage $coverage)
+    public function testSummary()
     {
         $this->assertEquals(
           array(
@@ -234,7 +210,7 @@ public function testSummary(PHP_CodeCoverage $coverage)
               32 => -2
             )
           ),
-          $coverage->getSummary()
+          $this->getBankAccountCoverage()->getSummary()
         );
     }
 }

Tests/TestCase.php

@@ -59,6 +59,33 @@
  */
 abstract class PHP_CodeCoverage_TestCase extends PHPUnit_Framework_TestCase
 {
+    protected function getBankAccountCoverage()
+    {
+        $coverage = new PHP_CodeCoverage($this->setUpXdebugStub());
+
+        $coverage->start(
+          new BankAccountTest('testBalanceIsInitiallyZero'), TRUE
+        );
+        $coverage->stop();
+
+        $coverage->start(new BankAccountTest(
+          'testBalanceCannotBecomeNegative')
+        );
+        $coverage->stop();
+
+        $coverage->start(
+          new BankAccountTest('testBalanceCannotBecomeNegative2')
+        );
+        $coverage->stop();
+
+        $coverage->start(
+          new BankAccountTest('testDepositWithdrawMoney')
+        );
+        $coverage->stop();
+
+        return $coverage;
+    }
+
     protected function setUpXdebugStub()
     {
         $stub = $this->getMock('PHP_CodeCoverage_Driver_Xdebug');