Merge pull request #433 from d8vjork/master

Add pivotSynced event listener to flush cache when performing BelongsToMany::sync

Author: mikebronner

src/Traits/ModelCaching.php

@@ -75,6 +75,10 @@ public static function bootCachable()
         //     $instance->checkCooldownAndFlushAfterPersisting($instance);
         // });
 
+        static::pivotSynced(function ($instance, $secondInstance, $relationship) {
+            $instance->checkCooldownAndFlushAfterPersisting($instance, $relationship);
+        });
+
         static::pivotAttached(function ($instance, $secondInstance, $relationship) {
             $instance->checkCooldownAndFlushAfterPersisting($instance, $relationship);
         });

tests/Integration/CachedBuilder/BelongsToManyTest.php

@@ -169,6 +169,49 @@ public function testUncachedRelatedModelDoesntCache()
         $this->assertNotNull($uncachedResult);
     }
 
+    public function testInvalidatingCacheWhenSyncing()
+    {
+        $bookId = (new Store)
+            ->disableModelCaching()
+            ->with("books")
+            ->first()
+            ->books
+            ->first()
+            ->id;
+        $key = sha1("genealabs:laravel-model-caching:testing:{$this->testingSqlitePath}testing.sqlite:stores:genealabslaravelmodelcachingtestsfixturesstore-testing:{$this->testingSqlitePath}testing.sqlite:books-first");
+        $tags = [
+            "genealabs:laravel-model-caching:testing:{$this->testingSqlitePath}testing.sqlite:genealabslaravelmodelcachingtestsfixturesstore",
+        ];
+
+        $newStores = factory(Store::class, 2)
+            ->create();
+
+        $result = Book::find($bookId)
+            ->stores;
+
+        Book::find($bookId)
+            ->stores()
+            ->attach($newStores[0]->id);
+        
+        Book::find($bookId)
+            ->stores()
+            ->sync($newStores->pluck('id'));
+
+        $this->assertEmpty(array_diff(
+            Book::find($bookId)->stores()->pluck((new Store)->getTable() . '.id')->toArray(),
+            $newStores->pluck('id')->toArray()
+        ));
+
+        $cachedResult = $this
+            ->cache()
+            ->tags($tags)
+            ->get($key)['value']
+            ?? null;
+
+        $this->assertNotEmpty($result);
+        $this->assertNull($cachedResult);
+    }
+
     // /** @group test */
     // public function testUncachedDetachesFromCached()
     // {