refactor: init request and response tests

src/interceptors/request.ts

@@ -6,10 +6,9 @@ import { CACHED_RESPONSE_STATUS, CACHED_RESPONSE_STATUS_TEXT } from '../util/sta
 export function applyRequestInterceptor(axios: AxiosCacheInstance): void {
   axios.interceptors.request.use(async (config) => {
     // Only cache specified methods
-    if (
-      config.cache?.methods &&
-      !config.cache.methods.some((method) => (config.method || 'get').toLowerCase() == method)
-    ) {
+    const allowedMethods = config.cache?.methods || axios.defaults.cache.methods;
+
+    if (!allowedMethods.some((method) => (config.method || 'get').toLowerCase() == method)) {
       return config;
     }
 

src/interceptors/response.ts

@@ -18,7 +18,7 @@ export function applyResponseInterceptor(axios: AxiosCacheInstance): void {
     const key = axios.generateKey(response.config);
     const cache = await axios.storage.get(key);
 
-    // Response is empty or was already cached
+    // Response shouldn't be cached or was already cached
     if (cache.state !== 'loading') {
       return response;
     }

src/util/cache-predicate.ts

@@ -34,7 +34,10 @@ export function checkPredicateObject(
       }
     } else {
       const [start, end] = statusCheck;
-      if (response.status <= start || response.status >= end) {
+      if (
+        response.status < start || //
+        response.status > end
+      ) {
         return false;
       }
     }

test/interceptors/request.test.ts

@@ -0,0 +1,29 @@
+import { axiosMock, mockAxios } from '../mocks/axios';
+
+describe('test request interceptor', () => {
+  it('tests against specified methods', async () => {
+    const axios = mockAxios({
+      // only cache post methods
+      methods: ['post']
+    });
+
+    const response = await axios.get(axiosMock.url);
+    const cacheKey = await axios.generateKey(response.config);
+    const cache = await axios.storage.get(cacheKey);
+
+    expect(cache.state).toBe('empty');
+  });
+
+  it('tests specified methods', async () => {
+    const axios = mockAxios({
+      // only cache get methods
+      methods: ['get']
+    });
+
+    const response = await axios.get(axiosMock.url);
+    const cacheKey = await axios.generateKey(response.config);
+    const cache = await axios.storage.get(cacheKey);
+
+    expect(cache.state).toBe('cached');
+  });
+});

test/interceptors/response.test.ts

@@ -0,0 +1,7 @@
+// import { mockAxios } from '../mocks/axios';
+
+describe('test request interceptor', () => {
+  it('tests', () => {
+    //const axios = mockAxios();
+  });
+});

test/mocks/axios.ts

@@ -20,7 +20,6 @@ export function mockAxios(
 
   const cachedApi = createCache(api, {
     // Defaults to cache every request
-    // cachePredicate: () => true,
     ...options
   });