Respects users base config option (#188)

Fixes #185
laravel · Dec 19, 2022 · a1de7cf · a1de7cf
1 parent 846ce36
commit a1de7cf
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/index.ts b/src/index.ts
@@ -121,7 +121,7 @@ function resolveLaravelPlugin(pluginConfig: Required<PluginConfig>): LaravelPlug
             ensureCommandShouldRunInEnvironment(command, env)
 
             return {
-                base: userConfig.base ?? command === 'build' ? resolveBase(pluginConfig, assetUrl) : '',
+                base: userConfig.base ?? (command === 'build' ? resolveBase(pluginConfig, assetUrl) : ''),
                 publicDir: userConfig.publicDir ?? false,
                 build: {
                     manifest: userConfig.build?.manifest ?? !ssr,

diff --git a/tests/index.test.ts b/tests/index.test.ts
@@ -64,6 +64,18 @@ describe('laravel-vite-plugin', () => {
         expect(ssrConfig.build.rollupOptions.input).toBe('resources/js/ssr.ts')
     })
 
+    it('respects users base config option', () => {
+        const plugin = laravel({
+            input: 'resources/js/app.ts',
+        })[0]
+
+        const userConfig = { base: '/foo/' }
+
+        const config = plugin.config(userConfig, { command: 'build', mode: 'production' })
+
+        expect(config.base).toBe('/foo/')
+    })
+
     it('accepts a partial configuration', () => {
         const plugin = laravel({
             input: 'resources/js/app.js',