fix error handler

web/app.js

@@ -13,6 +13,23 @@ import GDPRWebhookHandlers from './gdpr.js';
 import addSessionShopToReqParams from './middleware/addSessionShopToReqParameters.js';
 
 init();
+
+const errorMiddleware = (err, _req, res, _next) => {
+  if (err.name === 'ValidationError') {
+    return res.status(400).json({
+      success: false,
+      // @ts-ignore
+      message: err.message
+    });
+  }
+
+  // @ts-ignore
+  return res.status(500).json({
+    message: err.message,
+    stack: err.stack
+  });
+};
+
 const STATIC_PATH =
   process.env.NODE_ENV === 'production'
     ? `${process.cwd()}/frontend/dist`
@@ -27,7 +44,7 @@ app.get(
   shopify.redirectToShopifyOrAppRoot()
 );
 
-app.use('/fdc', express.json(), fdcRouters);
+app.use('/fdc', express.json(), fdcRouters, errorMiddleware);
 app.use('/api/*', shopify.validateAuthenticatedSession());
 
 app.use('/*', addSessionShopToReqParams);
@@ -43,32 +60,15 @@ app.use('/*', shopify.ensureInstalledOnShop(), async (_req, res) =>
     .send(readFileSync(join(STATIC_PATH, 'index.html')))
 );
 
+app.use(errorMiddleware);
+
 app.post(
   shopify.config.webhooks.path,
   shopify.processWebhooks({
     webhookHandlers: GDPRWebhookHandlers
   })
 );
 
-app.use((err, _req, res) => {
-  console.log('errrrrrrrr', err);
-  console.error(err, {
-    name: err.name
-  });
-
-  if (err.name === 'ValidationError') {
-    return res.status(400).json({
-      success: false,
-      // @ts-ignore
-      message: err.message
-    });
-  }
-
-  // @ts-ignore
-  return res.status(500).json({
-    message: err.message,
-    stack: err.stack
-  });
-});
+// add error handler
 
 export default app;

web/fdc-modules/orders/controllers/retrieve-order.test.js

@@ -245,7 +245,7 @@ describe('retrieve-order', () => {
           ]
         }
       })
-      .expect(500)
+      .expect(400)
       .expect('Content-Type', /json/)
 
       .expect({