Browse Source

It's only necessary to override abstract methods

So, don't just copy and paste the setup method call
AD7six 11 years ago
parent
commit
5a3a48b829
1 changed files with 3 additions and 8 deletions
  1. 3 8
      tests/TestCase/ORM/AssociationTest.php

+ 3 - 8
tests/TestCase/ORM/AssociationTest.php

@@ -187,12 +187,10 @@ class AssociationTest extends TestCase
             '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 @@ class AssociationTest extends TestCase
         ];
         $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());