Add coverage

Author: howdu

.github/workflows/run-tests.yml

@@ -52,23 +52,26 @@ jobs:
         uses: actions/cache@v4
         with:
           path: vendor
-          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
+          key: ${{ runner.os }}-composer-${{ matrix.php }}-${{ matrix.laravel }}-${{ matrix.filament }}-${{ matrix.testbench }}-${{ matrix.livewire }}-${{ hashFiles('**/composer.lock') }}
           restore-keys: |
+            ${{ runner.os }}-composer-${{ matrix.php }}-${{ matrix.laravel }}-${{ matrix.filament }}-${{ matrix.testbench }}-${{ matrix.livewire }}-
             ${{ runner.os }}-composer-
 
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php }}
-          extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick, fileinfo
-          coverage: none
+          extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick, fileinfo, pcov
+          coverage: pcov
 
       - name: Setup problem matchers
         run: |
           echo "::add-matcher::${{ runner.tool_cache }}/php.json"
           echo "::add-matcher::${{ runner.tool_cache }}/phpunit.json"
 
       - name: Install dependencies
+        env:
+          XDEBUG_MODE: off
         run: |
           composer require \
             "laravel/framework:${{ matrix.laravel }}" \
@@ -78,5 +81,15 @@ jobs:
             --no-interaction --no-update --no-progress --ansi
           composer update --prefer-stable --prefer-dist --no-interaction --no-progress --ansi
 
-      - name: Execute tests
-        run: vendor/bin/pest
+      - name: Prepare coverage directory
+        run: mkdir -p build/logs
+
+      - name: Execute tests with coverage
+        run: vendor/bin/pest --coverage --coverage-clover build/logs/clover.xml
+
+      - name: Upload coverage artifact
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-${{ matrix.php }}-${{ matrix.laravel }}-${{ matrix.filament }}
+          path: build/logs/clover.xml
+          if-no-files-found: error

composer.json

@@ -29,14 +29,14 @@
     "require-dev": {
         "larastan/larastan": "^3.0",
         "laravel/pint": "^1.0",
-        "orchestra/testbench": "*",
+        "orchestra/testbench": "^9.0 || ^10.0",
         "pestphp/pest": "^3.7 || ^4.0",
-        "pestphp/pest-plugin-arch": "*",
-        "pestphp/pest-plugin-laravel": "*",
-        "pestphp/pest-plugin-livewire": "*",
-        "phpstan/extension-installer": "*",
-        "phpstan/phpstan-deprecation-rules": "*",
-        "phpstan/phpstan-phpunit": "*"
+        "pestphp/pest-plugin-arch": "^1.0",
+        "pestphp/pest-plugin-laravel": "^3.0 || ^4.0",
+        "pestphp/pest-plugin-livewire": "^3.0 || ^4.0",
+        "phpstan/extension-installer": "^1.3",
+        "phpstan/phpstan-deprecation-rules": "^1.1",
+        "phpstan/phpstan-phpunit": "^1.4"
     },
     "autoload": {
         "psr-4": {
@@ -53,6 +53,7 @@
         "analyse": "phpstan analyse",
         "lint": "pint",
         "test": "pest",
+        "test:coverage": "pest --coverage",
         "test:lint": "pint --test"
     },
     "config": {

tests/ExampleTest.php tests/Integration/ClearCacheTest.phptests/ExampleTest.php renamed to tests/Integration/ClearCacheTest.php

@@ -0,0 +1,55 @@
+<?php
+
+use CmsMulti\FilamentClearCache\Facades\FilamentClearCache;
+use CmsMulti\FilamentClearCache\Commands\FilamentClearCacheCommand;
+use Illuminate\Support\Facades\Artisan;
+use Illuminate\Support\Facades\Cache;
+
+it('runs default commands from config', function () {
+    config()->set('filament-clear-cache.default_commands', [
+        'config:clear',
+        'cache:clear',
+    ]);
+
+    $exitCode = Artisan::call(FilamentClearCacheCommand::class);
+
+    expect($exitCode)->toBe(0);
+});
+
+it('runs custom string commands added via facade', function () {
+    FilamentClearCache::addCommand('route:clear');
+
+    $exitCode = Artisan::call(FilamentClearCacheCommand::class);
+
+    expect($exitCode)->toBe(0);
+});
+
+it('runs custom array command with parameters', function () {
+    FilamentClearCache::addCommand('config:cache', ['--no-ansi' => true]);
+
+    $exitCode = Artisan::call(FilamentClearCacheCommand::class);
+
+    expect($exitCode)->toBe(0);
+});
+
+it('runs custom closure command', function () {
+    $ran = false;
+    FilamentClearCache::addCommand(function () use (&$ran) {
+        $ran = true;
+    });
+
+    $exitCode = Artisan::call(FilamentClearCacheCommand::class);
+
+    expect($exitCode)->toBe(0);
+    expect($ran)->toBeTrue();
+});
+
+it('resets changes_count cache key when configured', function () {
+    $key = 'filament_clear_cache_changes';
+    config()->set('filament-clear-cache.changes_count', $key);
+    Cache::put($key, 5);
+
+    Artisan::call(FilamentClearCacheCommand::class);
+
+    expect(Cache::get($key))->toBe(0);
+});

tests/Integration/CommandTest.php

@@ -28,11 +28,11 @@ protected function getPackageProviders($app)
         return [
             BladeHeroiconsServiceProvider::class,
             BladeIconsServiceProvider::class,
-            LivewireServiceProvider::class,
             FilamentServiceProvider::class,
             NotificationsServiceProvider::class,
             SupportServiceProvider::class,
             FilamentClearCacheServiceProvider::class,
+            LivewireServiceProvider::class,
             AdminPanelProvider::class,
         ];
     }
@@ -52,6 +52,9 @@ protected function setUpDatabase($app)
             $table->string('name');
         });
 
+        // Cache array
+       $app['cache']->set('cache.default', 'array');
+
         // self::$migration->up();
     }
 }

tests/TestCase.php

@@ -0,0 +1,16 @@
+<?php
+
+use CmsMulti\FilamentClearCache\Facades\FilamentClearCache;
+
+it('facade adds commands and exposes them', function () {
+    FilamentClearCache::addCommand('view:clear');
+    FilamentClearCache::addCommand('config:cache', ['--force' => true]);
+
+    $commands = FilamentClearCache::getCommands();
+
+    expect($commands)
+        ->toHaveCount(2)
+        ->toContain('view:clear');
+    expect($commands[1])
+        ->toBe(['config:cache', ['--force' => true]]);
+});

tests/Unit/FacadeTest.php

@@ -0,0 +1,36 @@
+<?php
+
+use CmsMulti\FilamentClearCache\FilamentClearCacheManager;
+
+it('adds string command', function () {
+    $manager = new FilamentClearCacheManager();
+    $manager->addCommand('cache:clear');
+
+    expect($manager->getCommands())
+        ->toHaveCount(1)
+        ->toContain('cache:clear');
+});
+
+it('adds array command with params', function () {
+    $manager = new FilamentClearCacheManager();
+    $manager->addCommand('config:cache', ['--force' => true]);
+
+    expect($manager->getCommands()[0])
+        ->toBe(['config:cache', ['--force' => true]]);
+});
+
+it('adds closure command', function () {
+    $manager = new FilamentClearCacheManager();
+    $ran = false;
+    $manager->addCommand(function () use (&$ran) {
+        $ran = true;
+    });
+
+    $commands = $manager->getCommands();
+    expect($commands)->toHaveCount(1);
+    // Invoke the closure to ensure it's callable
+    $closure = $commands[0];
+    expect(is_callable($closure))->toBeTrue();
+    $closure();
+    expect($ran)->toBeTrue();
+});