Merge branch 'release/0.2.45'

Author: mikebronner

CHANGELOG.md

@@ -4,6 +4,38 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [0.2.45] - 3 Mar 2018
+### Fixed
+- pagination cache key generation; fixes #85.
+
+## [0.2.44] - 3 Mar 2018
+### Fixed
+- disabling of caching using the query scope.
+
+## [0.2.43] - 2 Mar 2018
+### Fixed
+- actions on belongsToMany relationships not flushing cache when needed.
+
+## [0.2.42] - 28 Feb 2018
+### Added
+- additional integration tests for additional use cases.
+
+### Fixed
+- flushing a specific model from the command line that extended a base class and did not use the trait directly.
+
+## [0.2.41] - 26 Feb 2018
+### Fixes
+- cache invalidation when using ->insert() method.
+- cache invalidation when using ->update() method.
+
+## [0.2.40] - 24 Feb 2018
+### Updated
+- code with some home-cleaning and refactoring.
+
+## [0.2.39] - 24 Feb 2018
+### Updated
+- CachedBuilder class with some refactoring and cleanup.
+
 ## [0.2.38] - 24 Feb 2018
 ### Added
 - cache-invalidation-cool-down functionality.

composer.json

@@ -23,6 +23,7 @@
         "mockery/mockery": "0.9.*",
         "orchestra/database": "3.6.x-dev@dev",
         "orchestra/testbench": "^3.6",
+        "orchestra/testbench-browser-kit": "^3.6",
         "php-coveralls/php-coveralls" : "*",
         "phpmd/phpmd": "*",
         "phpunit/phpunit": "*",

phpunit.xml

@@ -13,6 +13,9 @@
     syntaxCheck="false"
 >
 <testsuites>
+    <testsuite name="Feature">
+        <directory suffix="Test.php">./tests/Feature</directory>
+    </testsuite>
     <testsuite name="Integration">
         <directory suffix="Test.php">./tests/Integration</directory>
     </testsuite>

src/CachedBuilder.php

@@ -115,7 +115,7 @@ public function paginate(
             return parent::paginate($perPage, $columns, $pageName, $page);
         }
 
-        $page = $page ?: 1;
+        $page = request("page", $page ?: 1);
         $cacheKey = $this->makeCacheKey($columns, null, "-paginate_by_{$perPage}_{$pageName}_{$page}");
 
         return $this->cachedValue(func_get_args(), $cacheKey);

tests/CreatesApplication.php

@@ -12,12 +12,26 @@ trait CreatesApplication
 {
     protected $cache;
 
+    protected function cache()
+    {
+        $cache = cache();
+
+        if (config('laravel-model-caching.store')) {
+            $cache = $cache->store(config('laravel-model-caching.store'));
+        }
+
+        return $cache;
+    }
+
     public function setUp()
     {
         parent::setUp();
 
+        require(__DIR__ . '/routes/web.php');
+
         $this->withFactories(__DIR__ . '/database/factories');
         $this->loadMigrationsFrom(__DIR__ . '/database/migrations');
+        view()->addLocation(__DIR__ . '/resources/views', 'laravel-model-caching');
 
         $this->cache = cache()
             ->store(config('laravel-model-caching.store'));

tests/Feature/PaginationTest.php

@@ -0,0 +1,23 @@
+<?php namespace GeneaLabs\LaravelModelCaching\Tests\Browser;
+
+use GeneaLabs\LaravelModelCaching\Tests\FeatureTestCase;
+use GeneaLabs\LaravelModelCaching\Tests\Fixtures\Book;
+
+class PaginationTest extends FeatureTestCase
+{
+    public function testPaginationProvidesDifferentLinksOnDifferentPages()
+    {
+        $book = (new Book)
+            ->take(11)
+            ->get()
+            ->last();
+        $page1 = $this->visit("pagination-test");
+
+        $this->assertTrue(str_contains($page1->response->getContent(), '<li class="page-item active"><span class="page-link">1</span></li>'));
+
+        $page2 = $page1->click("2");
+
+        $this->assertTrue(str_contains($page2->response->getContent(), '<li class="page-item active"><span class="page-link">2</span></li>'));
+        $page2->see($book->title);
+    }
+}

tests/FeatureTestCase.php

@@ -0,0 +1,8 @@
+<?php namespace GeneaLabs\LaravelModelCaching\Tests;
+
+use Orchestra\Testbench\BrowserKit\TestCase as BaseTestCase;
+
+abstract class FeatureTestCase extends BaseTestCase
+{
+    use CreatesApplication;
+}

tests/IntegrationTestCase.php

@@ -5,15 +5,4 @@
 abstract class IntegrationTestCase extends BaseTestCase
 {
     use CreatesApplication;
-
-    protected function cache()
-    {
-        $cache = cache();
-
-        if (config('laravel-model-caching.store')) {
-            $cache = $cache->store(config('laravel-model-caching.store'));
-        }
-
-        return $cache;
-    }
 }

tests/resources/views/model-caching-tests/pagination.blade.php

@@ -0,0 +1,22 @@
+<!DOCTYPE html>
+<html>
+    <body>
+        <table>
+
+            @foreach ($books as $book)
+                <tr>
+                    <td>
+                        {{ $book->id }}
+                    </td>
+                    <td>
+                        {{ $book->title }}
+                    </td>
+                </tr>
+            @endforeach
+
+        </table>
+
+        {{ $books->links() }}
+
+    </body>
+</html>

tests/routes/web.php

@@ -0,0 +1,13 @@
+<?php
+
+use GeneaLabs\LaravelModelCaching\Tests\Fixtures\Book;
+
+Route::get('pagination-test', function () {
+    $books = (new Book)
+        ->paginate(10);
+
+    return view("model-caching-tests.pagination")
+        ->with(compact(
+            'books'
+        ));
+});