feat!: do not use manifest in dev or test environments (#170)

innocenzi · Jan 16, 2022 · 590c034 · 590c034
1 parent b87ac2d
commit 590c034
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/Vite.php b/src/Vite.php
@@ -205,7 +205,7 @@ protected function shouldUseManifest(): bool
             return false;
         }
 
-        if (! App::environment('local')) {
+        if (! App::environment(['local', 'testing'])) {
             return true;
         }
 

diff --git a/tests/Unit/ViteTest.php b/tests/Unit/ViteTest.php
@@ -125,6 +125,7 @@
 
     Config::set('vite.entrypoints', 'scripts');
     App::setBasePath(__DIR__);
+    set_env('production');
     expect(get_vite()->getClientAndEntrypointTags())
         ->toEqual('<script type="module" src="https://cdn.random.url/build/app.83b2e884.js"></script>');
 });
@@ -136,6 +137,7 @@
         'https://cdn.random.url'
     ));
 
+    set_env('production');
     expect(vite_asset('image.png'))->toBe('https://cdn.random.url/build/image.png');
 });