chore: fix test

Author: maxakuru

src/routes/catalog/update.js

@@ -10,7 +10,7 @@
  * governing permissions and limitations under the License.
  */
 
-import { assertValidProduct } from '../../utils/product.js';
+import { assertValidProduct, hasUppercase } from '../../utils/product.js';
 import { errorResponse } from '../../utils/http.js';
 import StorageClient from './StorageClient.js';
 
@@ -20,17 +20,24 @@ import StorageClient from './StorageClient.js';
  * @returns {Promise<Response>} - A promise that resolves to the product response.
  */
 export default async function update(ctx) {
-  const { config, log } = ctx;
+  const { config, log, data: product } = ctx;
 
   if (config.sku === '*') {
     return errorResponse(501, 'not implemented');
   }
 
-  const product = ctx.data;
   if (!product || typeof product !== 'object') {
     return errorResponse(400, 'invalid product data');
   }
 
+  if (hasUppercase(config.sku)) {
+    return errorResponse(400, 'sku must be lowercase');
+  }
+
+  if (config.sku !== product.sku) {
+    return errorResponse(400, 'sku must match the product data');
+  }
+
   assertValidProduct(product);
 
   const storage = StorageClient.fromContext(ctx);

test/routes/catalog/handler.test.js

@@ -44,7 +44,7 @@ describe('catalogHandler Tests', () => {
   it('should return 405 when method is not allowed', async () => {
     const ctx = DEFAULT_CONTEXT({
       info: { method: 'HEAD' },
-      url: { pathname: '/org/site/catalog/store/view/product/sku1' },
+      url: { pathname: '/org/site/catalog/store/view/products/sku1' },
       config: {},
     });
     const request = {};
@@ -53,23 +53,10 @@ describe('catalogHandler Tests', () => {
     assert.equal(response.status, 405);
   });
 
-  it('should return 400 when sku is uppercase', async () => {
-    const ctx = DEFAULT_CONTEXT({
-      info: { method: 'GET' },
-      url: { pathname: '/org/site/catalog/store/view/product/PRODUCT-SKU' },
-      config: {},
-    });
-    const request = {};
-    const response = await catalogHandler(ctx, request);
-
-    assert.equal(response.status, 400);
-    assert.equal(response.headers.get('x-error'), 'Invalid SKU: SKU cannot contain uppercase letters');
-  });
-
   it('should call handleProductLookupRequest when method is GET and subRoute is "lookup"', async () => {
     const ctx = DEFAULT_CONTEXT({
       info: { method: 'GET' },
-      url: { pathname: '/org/site/catalog/store/view/lookup/sku' },
+      url: { pathname: '/org/site/catalog/store/view/lookup' },
       config: {},
     });
     const request = {};
@@ -86,7 +73,7 @@ describe('catalogHandler Tests', () => {
   it('should return 405 if subRoute is "lookup" but method is not GET', async () => {
     const ctx = DEFAULT_CONTEXT({
       info: { method: 'PUT' },
-      url: { pathname: '/org/site/catalog/store/view/lookup/sku' },
+      url: { pathname: '/org/site/catalog/store/view/lookup' },
       config: {},
     });
     const request = {};