chore: fix tests

Author: maxakuru

src/routes/catalog/fetch.js

@@ -18,11 +18,11 @@ import StorageClient from './StorageClient.js';
  * @returns {Promise<Response>} - A promise that resolves to the product response.
  */
 export default async function fetch(ctx) {
-  const storage = StorageClient.fromContext(ctx);
   const { sku } = ctx.config;
+
+  const storage = StorageClient.fromContext(ctx);
   const product = await storage.fetchProduct(sku);
 
-  // TODO: use long ttl, add cache keys
   return new Response(JSON.stringify(product), {
     headers: { 'Content-Type': 'application/json' },
   });

src/routes/catalog/handler.js

@@ -60,7 +60,7 @@ export default async function handler(ctx, request) {
     storeCode,
     storeViewCode,
     subRoute,
-    sku: sku.endsWith('.json') ? sku.slice(0, -5) : sku,
+    sku: sku && sku.endsWith('.json') ? sku.slice(0, -5) : sku,
   });
 
   const fn = handlers[subRoute]?.[method];

src/utils/media.js

@@ -127,13 +127,13 @@ export async function extractAndReplaceImages(ctx, product) {
   };
 
   await Promise.all([
-    processQueue([...product.images], async (image) => {
+    processQueue([...product.images ?? []], async (image) => {
       const newUrl = await processImage(image.url);
       if (newUrl) {
         image.url = newUrl;
       }
     }),
-    processQueue([...product.variants], async (variant) => {
+    processQueue([...product.variants ?? []], async (variant) => {
       const newUrl = await processImage(variant.image);
       if (newUrl) {
         variant.image = newUrl;

test/routes/catalog/fetch.test.js

@@ -41,6 +41,7 @@ describe('handleProductFetchRequest', () => {
 
   it('should return the product response when fetchProduct succeeds', async () => {
     const product = { sku: 'sku1', name: 'Product 1' };
+    ctx.config.sku = 'sku1';
 
     storageStub.fetchProduct.resolves(product);
     const response = await handleProductFetchRequest(ctx);