Merge pull request DesignPatternsPHP#262 from mtdavidson/patch-1

Dominik Liebler · web-flow · commit 97c507886d58 · 2017-01-11T12:55:46.000+01:00
Correct variable names in specification test
diff --git a/Behavioral/Specification/Tests/SpecificationTest.php b/Behavioral/Specification/Tests/SpecificationTest.php
@@ -27,20 +27,20 @@ public function testCanAnd()
         $spec1 = new PriceSpecification(50, 100);
         $spec2 = new PriceSpecification(80, 200);
 
-        $orSpec = new AndSpecification($spec1, $spec2);
+        $andSpec = new AndSpecification($spec1, $spec2);
 
-        $this->assertFalse($orSpec->isSatisfiedBy(new Item(150)));
-        $this->assertFalse($orSpec->isSatisfiedBy(new Item(1)));
-        $this->assertFalse($orSpec->isSatisfiedBy(new Item(51)));
-        $this->assertTrue($orSpec->isSatisfiedBy(new Item(100)));
+        $this->assertFalse($andSpec->isSatisfiedBy(new Item(150)));
+        $this->assertFalse($andSpec->isSatisfiedBy(new Item(1)));
+        $this->assertFalse($andSpec->isSatisfiedBy(new Item(51)));
+        $this->assertTrue($andSpec->isSatisfiedBy(new Item(100)));
     }
 
     public function testCanNot()
     {
         $spec1 = new PriceSpecification(50, 100);
-        $orSpec = new NotSpecification($spec1);
+        $notSpec = new NotSpecification($spec1);
 
-        $this->assertTrue($orSpec->isSatisfiedBy(new Item(150)));
-        $this->assertFalse($orSpec->isSatisfiedBy(new Item(50)));
+        $this->assertTrue($notSpec->isSatisfiedBy(new Item(150)));
+        $this->assertFalse($notSpec->isSatisfiedBy(new Item(50)));
     }
 }
