Changeset 20954

Timestamp:

08/09/09 12:38:45 (4 years ago)

Author:

bschussek

Message:

Fixed: Classes with final methods can be mocked

Files:

• tools/lime/branches/2.0-experimental/lib/mock/LimeMock.php (modified) (1 diff)
• tools/lime/branches/2.0-experimental/test/unit/mock/LimeMockTest.php (modified) (2 diffs)

tools/lime/branches/2.0-experimental/lib/mock/LimeMock.php

@@ -201 +201 @@
-    foreach ($class->getMethods() as $method)
-    {
-
+
+
-      {
-        /* @var $method ReflectionMethod */
-        $modifiers = Reflection::getModifierNames($method->getModifiers());
-        $modifiers = array_diff($modifiers, array('abstract'));

tools/lime/branches/2.0-experimental/test/unit/mock/LimeMockTest.php

@@ -53 +53 @@
-}
-
+class TestClassWithFinalMethods
+{
+  public static $calls = 0;
+
+  public final function testMethod()
+  {
+    ++self::$calls;
+  }
+}
+
-class TestException extends Exception {}
-
-
-79
+80
-
-
@@ -133 +143 @@
-  $t->is(TestClass::$calls, 0, 'The method has not been called');
-
+
+// @Test: Final methods cannot be mocked
+
+  // fixtures
+  TestClassWithFinalMethods::$calls = 0;
+  $m = LimeMock::create('TestClassWithFinalMethods');
+  $m->replay();
+  // test
+  $m->testMethod();
+  // assertions
+  $t->is(TestClassWithFinalMethods::$calls, 1, 'The method has been called');
-
-