Eliminate code duplication.

sebastianbergmann · Nov 10, 2006 · c5aed5a · c5aed5a
1 parent f20dd09
commit c5aed5a
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 32 deletions.
diff --git a/PHPUnit/Framework/Constraint.php b/PHPUnit/Framework/Constraint.php
@@ -103,29 +103,34 @@ public function fail($other, $description, $not = FALSE)
         );
 
         if ($not) {
-            $failureDescription = str_replace(
-              array(
-                'contains ',
-                'exists ',
-                'has ',
-                'is ',
-                'matches '
-              ),
-              array(
-                'not contains ',
-                'not exists ',
-                'has not ',
-                'is not ',
-                'not matches '
-              ),
-              $failureDescription
-            );
+            $failureDescription = self::negate($failureDescription);
         }
 
         throw new PHPUnit_Framework_ExpectationFailedException(
           $failureDescription
         );
     }
+
+    public static function negate($string)
+    {
+        return str_replace(
+          array(
+            'contains ',
+            'exists ',
+            'has ',
+            'is ',
+            'matches '
+          ),
+          array(
+            'not contains ',
+            'not exists ',
+            'has not ',
+            'is not ',
+            'not matches '
+          ),
+          $string
+        );
+    }
 }
 
 }

diff --git a/PHPUnit/Framework/Constraint/Not.php b/PHPUnit/Framework/Constraint/Not.php
@@ -124,21 +124,7 @@ public function toString()
             break;
 
             default: {
-                return str_replace(
-                  array(
-                    'contains ',
-                    'exists ',
-                    'has ',
-                    'is ',
-                    'matches '
-                  ),
-                  array(
-                    'not contains ',
-                    'not exists ',
-                    'has not ',
-                    'is not ',
-                    'not matches '
-                  ),
+                return PHPUnit_Framework_Constraint::negate(
                   $this->constraint->toString()
                 );
             }