fix: add server plugin when any feature is enabled (#69)

dulnan · Aug 25, 2024 · d1248e2 · d1248e2
1 parent bf54b93
commit d1248e2
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 14 deletions.
diff --git a/src/module.ts b/src/module.ts
@@ -146,10 +146,15 @@ export default defineNuxtModule<ModuleOptions>({
       })
     }
 
-    // Serves cached routes.
+    // Add the server plugin if any of the features is enabled.
+    // During local development these might all be disabled, so it's not
+    // necessary to add the plugin.
     if (
       options.route?.enabled ||
       options.cdn?.enabled ||
+      options.data?.enabled ||
+      options.component?.enabled ||
+      options.api?.enabled ||
       nuxt.options._prepare
     ) {
       addServerPlugin(resolve('./runtime/server/plugins/multiCache'))

diff --git a/test/cacheComponent.e2e.spec.ts b/test/cacheComponent.e2e.spec.ts
@@ -10,13 +10,13 @@ const multiCache: NuxtMultiCacheOptions = {
     enabled: true,
   },
   data: {
-    enabled: true,
+    enabled: false,
   },
   route: {
-    enabled: true,
+    enabled: false,
   },
   cdn: {
-    enabled: true,
+    enabled: false,
   },
   api: {
     enabled: true,

diff --git a/test/customStorageDriver.e2e.spec.ts b/test/customStorageDriver.e2e.spec.ts
@@ -5,19 +5,19 @@ import type { NuxtMultiCacheOptions } from '../src/runtime/types'
 
 const multiCache: NuxtMultiCacheOptions = {
   component: {
-    enabled: true,
+    enabled: false,
   },
   data: {
     enabled: true,
   },
   route: {
-    enabled: true,
+    enabled: false,
   },
   cdn: {
-    enabled: true,
+    enabled: false,
   },
   api: {
-    enabled: true,
+    enabled: false,
     authorization: false,
     cacheTagInvalidationDelay: 5000,
   },

diff --git a/test/dataCache.e2e.spec.ts b/test/dataCache.e2e.spec.ts
@@ -7,16 +7,16 @@ import purgeByKey from './__helpers__/purgeByKey'
 
 const multiCache: NuxtMultiCacheOptions = {
   component: {
-    enabled: true,
+    enabled: false,
   },
   data: {
     enabled: true,
   },
   route: {
-    enabled: true,
+    enabled: false,
   },
   cdn: {
-    enabled: true,
+    enabled: false,
   },
   api: {
     enabled: true,

diff --git a/test/routeCache.e2e.spec.ts b/test/routeCache.e2e.spec.ts
@@ -7,16 +7,16 @@ import purgeAll from './__helpers__/purgeAll'
 
 const multiCache: NuxtMultiCacheOptions = {
   component: {
-    enabled: true,
+    enabled: false,
   },
   data: {
-    enabled: true,
+    enabled: false,
   },
   route: {
     enabled: true,
   },
   cdn: {
-    enabled: true,
+    enabled: false,
   },
   api: {
     enabled: true,