squash mutants

Jeroen-G · Jan 26, 2023 · 49eaf44 · 49eaf44
1 parent 543c304
commit 49eaf44
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 51 deletions.
diff --git a/infection.json b/infection.json
@@ -3,6 +3,9 @@
     "source": {
         "directories": [
             "src"
+        ],
+        "excludes": [
+            "BladeMacroServiceProvider.php"
         ]
     },
     "logs": {

diff --git a/src/BladeMacroServiceProvider.php b/src/BladeMacroServiceProvider.php
@@ -11,16 +11,8 @@ class BladeMacroServiceProvider extends ServiceProvider
 {
     public function boot(): void
     {
-        Blade::directive('macro', function ($macroName) {
-            return Macro::define($macroName);
-        });
-
-        Blade::directive('endMacro', function () {
-            return Macro::endDefinition();
-        });
-
-        Blade::directive('showMacro', function ($expression) {
-            return Macro::show($expression);
-        });
+        Blade::directive('macro', fn ($macroName) => Macro::define($macroName));
+        Blade::directive('endMacro', fn () => Macro::endDefinition());
+        Blade::directive('showMacro', fn ($expression) => Macro::show($expression));
     }
 }
diff --git a/src/Macro.php b/src/Macro.php
@@ -6,6 +6,7 @@
 
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\Blade;
+use InvalidArgumentException;
 
 final class Macro
 {
@@ -22,7 +23,7 @@ public static function show(string $expression): string
         $macroName = $arguments->first();
 
         if (!array_key_exists($macroName, self::$macros)) {
-            throw new \Exception("The `$macroName` macro is not defined.");
+            throw new InvalidArgumentException("The `$macroName` macro is not defined.");
         }
 
         $args = $arguments->except([0])->implode(',');

diff --git a/tests/BladeMacroServiceProviderTest.php b/tests/BladeMacroServiceProviderTest.php
diff --git a/tests/MacroTest.php b/tests/MacroTest.php
@@ -50,7 +50,7 @@ public function test_show_macro(): void
     {
         Macro::$macros['testcase'] = 'hello';
 
-        $macro = Macro::show('testcase');
+        $macro = Macro::show('testcase ');
 
         self::assertSame(
             "<?php echo \JeroenG\BladeMacro\Macro::\$macros['testcase'](); ?>",
@@ -60,7 +60,7 @@ public function test_show_macro(): void
 
     public function test_showing_undefined_macro_throws_exception(): void
     {
-        $this->expectException(\Exception::class);
+        $this->expectException(\InvalidArgumentException::class);
         $this->expectExceptionMessage('The `testcase` macro is not defined.');
 
         self::assertEmpty(Macro::$macros);