Increase coverage by covering another failure case.

This path was untested before.
cakephp · Dec 19, 2015 · 15638b1 · 15638b1
1 parent 46847ac
commit 15638b1
Showing 1 changed file with 22 additions and 0 deletions.
diff --git a/tests/TestCase/ORM/Association/BelongsToManyTest.php b/tests/TestCase/ORM/Association/BelongsToManyTest.php
@@ -671,6 +671,28 @@ public function emptyProvider()
         ];
     }
 
+    /**
+     * Test that saveAssociated() fails on non-empty, non-iterable value
+     *
+     * @expectedException InvalidArgumentException
+     * @expectedExceptionMessage Could not save tags, it cannot be traversed
+     * @return void
+     */
+    public function testSaveAssociatedNotEmptyNotIterable()
+    {
+        $articles = TableRegistry::get('Articles');
+        $assoc = $articles->belongsToMany('Tags', [
+            'saveStrategy' => BelongsToMany::SAVE_APPEND,
+            'joinTable' => 'articles_tags',
+        ]);
+        $entity = new Entity([
+            'id' => 1,
+            'tags' => 'oh noes',
+        ], ['markNew' => true]);
+
+        $assoc->saveAssociated($entity);
+    }
+
     /**
      * Test that saving an empty set on create works.
      *