It's only necessary to override abstract methods

So, don't just copy and paste the setup method call
cakephp · Feb 10, 2015 · 5a3a48b · 5a3a48b
1 parent 95fe4ce
commit 5a3a48b
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/tests/TestCase/ORM/AssociationTest.php b/tests/TestCase/ORM/AssociationTest.php
@@ -187,12 +187,10 @@ public function testTargetPlugin()
             'sourceTable' => $this->source,
             'joinType' => 'INNER'
         ];
+
         $this->association = $this->getMock(
             '\Cake\ORM\Association',
-            [
-                '_options', 'attachTo', '_joinCondition', 'cascadeDelete', 'isOwningSide',
-                'saveAssociated', 'eagerLoader', 'type'
-            ],
+            ['type', 'eagerLoader', 'cascadeDelete', 'isOwningSide', 'saveAssociated'],
             ['ThisAssociationName', $config]
         );
 
@@ -302,10 +300,7 @@ public function testFinderInConstructor()
         ];
         $assoc = $this->getMock(
             '\Cake\ORM\Association',
-            [
-                '_options', 'attachTo', '_joinCondition', 'cascadeDelete', 'isOwningSide',
-                'saveAssociated', 'eagerLoader', 'type'
-            ],
+            ['type', 'eagerLoader', 'cascadeDelete', 'isOwningSide', 'saveAssociated'],
             ['Foo', $config]
         );
         $this->assertEquals('published', $assoc->finder());