feat: don't run Boost during testing

Adds tests for: enabled, testing, environment

Author: ashleyhindle

composer.json

@@ -19,7 +19,7 @@
         "illuminate/contracts": "^10.0|^11.0|^12.0",
         "illuminate/routing": "^10.0|^11.0|^12.0",
         "illuminate/support": "^10.0|^11.0|^12.0",
-        "laravel/mcp": "^0.1.0",
+        "laravel/mcp": "^0.1.1",
         "laravel/prompts": "^0.1.9|^0.3",
         "laravel/roster": "^0.2"
     },

src/BoostServiceProvider.php

@@ -19,13 +19,18 @@
 
 class BoostServiceProvider extends ServiceProvider
 {
+
     public function register(): void
     {
         $this->mergeConfigFrom(
             __DIR__.'/../config/boost.php',
             'boost'
         );
 
+        if (! $this->shouldRun()) {
+            return;
+        }
+
         $this->app->singleton(Roster::class, function () {
             $lockFiles = [
                 base_path('composer.lock'),
@@ -55,24 +60,21 @@ public function register(): void
 
     public function boot(Router $router): void
     {
-        if (! config('boost.enabled', true)) {
+        if (! $this->shouldRun()) {
             return;
         }
 
-        // Only enable Boost on local environments
-        if (! app()->environment(['local', 'testing']) && config('app.debug', false) !== true) {
-            return;
-        }
-
-        // @phpstan-ignore-next-line
         Mcp::local('laravel-boost', Boost::class);
 
         $this->registerPublishing();
         $this->registerCommands();
         $this->registerRoutes();
-        $this->registerBrowserLogger();
-        $this->callAfterResolving('blade.compiler', fn (BladeCompiler $bladeCompiler) => $this->registerBladeDirectives($bladeCompiler));
-        $this->hookIntoResponses($router);
+
+        if (config('boost.browser_logs_watcher', true)) {
+            $this->registerBrowserLogger();
+            $this->callAfterResolving('blade.compiler', fn (BladeCompiler $bladeCompiler) => $this->registerBladeDirectives($bladeCompiler));
+            $this->hookIntoResponses($router);
+        }
     }
 
     private function registerPublishing(): void
@@ -179,10 +181,24 @@ private static function mapJsTypeToPsr3Level(string $type): string
 
     private function hookIntoResponses(Router $router): void
     {
-        if (! config('boost.browser_logs_watcher', true)) {
-            return;
+        $router->pushMiddlewareToGroup('web', InjectBoost::class);
+    }
+
+    private function shouldRun(): bool
+    {
+        if (! config('boost.enabled', true)) {
+            return false;
         }
 
-        $router->pushMiddlewareToGroup('web', InjectBoost::class);
+        if (app()->runningUnitTests()) {
+            return false;
+        }
+
+        // Only enable Boost on local environments or when debug is true
+        if (! app()->environment('local') && config('app.debug', false) !== true) {
+            return false;
+        }
+
+        return true;
     }
 }

tests/Feature/BoostServiceProviderTest.php

@@ -0,0 +1,76 @@
+<?php
+
+declare(strict_types=1);
+
+use Illuminate\Support\Facades\Config;
+use Laravel\Boost\BoostServiceProvider;
+
+beforeEach(function () {
+    $this->refreshApplication();
+    Config::set('logging.channels.browser', null);
+});
+
+describe('boost.enabled configuration', function () {
+    it('does not boot boost when disabled', function () {
+        Config::set('boost.enabled', false);
+        app()->detectEnvironment(fn() => 'local');
+        
+        $provider = new BoostServiceProvider(app());
+        $provider->register();
+        $provider->boot(app('router'));
+        
+        $this->artisan('list')->expectsOutputToContain('boost:install');
+    });
+
+    it('boots boost when enabled in local environment', function () {
+        Config::set('boost.enabled', true);
+        app()->detectEnvironment(fn() => 'local');
+        
+        $provider = new BoostServiceProvider(app());
+        $provider->register();
+        $provider->boot(app('router'));
+        
+        expect(app()->bound(Laravel\Roster\Roster::class))->toBeTrue();
+        expect(config('logging.channels.browser'))->not->toBeNull();
+    });
+});
+
+describe('environment restrictions', function () {
+    it('does not boot boost in production even when enabled', function () {
+        Config::set('boost.enabled', true);
+        Config::set('app.debug', false);
+        app()->detectEnvironment(fn() => 'production');
+        
+        $provider = new BoostServiceProvider(app());
+        $provider->register();
+        $provider->boot(app('router'));
+        
+        expect(config('logging.channels.browser'))->toBeNull();
+    });
+
+    describe('testing environment', function () {
+        it('does not boot boost when debug is false', function () {
+            Config::set('boost.enabled', true);
+            Config::set('app.debug', false);
+            app()->detectEnvironment(fn() => 'testing');
+            
+            $provider = new BoostServiceProvider(app());
+            $provider->register();
+            $provider->boot(app('router'));
+            
+            expect(config('logging.channels.browser'))->toBeNull();
+        });
+
+        it('does not boot boost when debug is true', function () {
+            Config::set('boost.enabled', true);
+            Config::set('app.debug', true);
+            app()->detectEnvironment(fn() => 'testing');
+            
+            $provider = new BoostServiceProvider(app());
+            $provider->register();
+            $provider->boot(app('router'));
+            
+            expect(config('logging.channels.browser'))->toBeNull();
+        });
+    });
+});