Make default eagerness configurable

InnoGE · Aug 9, 2024 · c9c2ae9 · c9c2ae9
1 parent 357c5e8
commit c9c2ae9
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 4 deletions.
diff --git a/src/LaravelSpeculationRulesApiServiceProvider.php b/src/LaravelSpeculationRulesApiServiceProvider.php
@@ -18,18 +18,18 @@ public function configurePackage(Package $package): void
 
     public function bootingPackage(): void
     {
-        Route::macro('prerender', function (string $eagerness = 'moderate') {
+        Route::macro('prerender', function (?string $eagerness = null) {
             LaravelSpeculationRulesApi::$routeSpeculationRules['prerender'][] = [
-                'eagerness' => $eagerness,
+                'eagerness' => $eagerness ?? config('speculation-rules-api.default_eagerness'),
                 'uri' => $this->uri,
             ];
 
             return $this;
         });
 
-        Route::macro('prefetch', function (string $eagerness = 'moderate') {
+        Route::macro('prefetch', function (?string $eagerness = null) {
             LaravelSpeculationRulesApi::$routeSpeculationRules['prefetch'][] = [
-                'eagerness' => $eagerness,
+                'eagerness' => $eagerness ?? config('speculation-rules-api.default_eagerness'),
                 'uri' => $this->uri,
             ];
 

diff --git a/tests/SpeculationRulesApiTest.php b/tests/SpeculationRulesApiTest.php
@@ -28,6 +28,25 @@
     expect(LaravelSpeculationRulesApi::$routeSpeculationRules)->toMatchSnapshot();
 });
 
+test('default eagerness', function () {
+    LaravelSpeculationRulesApi::$routeSpeculationRules = [
+        'prerender' => [],
+        'prefetch' => [],
+    ];
+
+    Route::get('page-1', fn () => null)->prerender();
+
+    expect(data_get(LaravelSpeculationRulesApi::$routeSpeculationRules, 'prerender.0.eagerness'))
+        ->toBe('moderate');
+
+    config()->set('speculation-rules-api.default_eagerness', 'eager');
+
+    Route::get('page-1', fn () => null)->prerender();
+
+    expect(data_get(LaravelSpeculationRulesApi::$routeSpeculationRules, 'prerender.1.eagerness'))
+        ->toBe('eager');
+});
+
 test('speculation rules are merged properly', function () {
     LaravelSpeculationRulesApi::$routeSpeculationRules = [
         'prerender' => [],