Implement updates for better performance

Author: abrar2030

backend/accounting-service/src/invoice.controller.ts

@@ -51,11 +51,9 @@ class InvoiceController {
 
       // Check if user owns the invoice
       if (invoice.userId !== userId) {
-        res
-          .status(403)
-          .json({
-            message: "Forbidden: You do not have access to this invoice",
-          });
+        res.status(403).json({
+          message: "Forbidden: You do not have access to this invoice",
+        });
         return;
       }
 
@@ -104,11 +102,9 @@ class InvoiceController {
 
       // Check if user owns the invoice
       if (invoice.userId !== userId) {
-        res
-          .status(403)
-          .json({
-            message: "Forbidden: You do not have access to this invoice",
-          });
+        res.status(403).json({
+          message: "Forbidden: You do not have access to this invoice",
+        });
         return;
       }
 
@@ -149,11 +145,9 @@ class InvoiceController {
 
       // Check if user owns the invoice
       if (invoice.userId !== userId) {
-        res
-          .status(403)
-          .json({
-            message: "Forbidden: You do not have access to this invoice",
-          });
+        res.status(403).json({
+          message: "Forbidden: You do not have access to this invoice",
+        });
         return;
       }
 

backend/accounting-service/src/transaction.controller.ts

@@ -54,11 +54,9 @@ class TransactionController {
 
       // Check if user owns the transaction
       if (transaction.userId !== userId) {
-        res
-          .status(403)
-          .json({
-            message: "Forbidden: You do not have access to this transaction",
-          });
+        res.status(403).json({
+          message: "Forbidden: You do not have access to this transaction",
+        });
         return;
       }
 
@@ -107,11 +105,9 @@ class TransactionController {
 
       // Check if user owns the transaction
       if (transaction.userId !== userId) {
-        res
-          .status(403)
-          .json({
-            message: "Forbidden: You do not have access to this transaction",
-          });
+        res.status(403).json({
+          message: "Forbidden: You do not have access to this transaction",
+        });
         return;
       }
 
@@ -155,11 +151,9 @@ class TransactionController {
 
       // Check if user owns the transaction
       if (transaction.userId !== userId) {
-        res
-          .status(403)
-          .json({
-            message: "Forbidden: You do not have access to this transaction",
-          });
+        res.status(403).json({
+          message: "Forbidden: You do not have access to this transaction",
+        });
         return;
       }
 

backend/payments-service/src/payment.controller.ts

@@ -190,11 +190,9 @@ class PaymentController {
           paymentOwner: payment.userId,
           requestId,
         });
-        res
-          .status(403)
-          .json({
-            message: "Forbidden: You do not have access to this payment",
-          });
+        res.status(403).json({
+          message: "Forbidden: You do not have access to this payment",
+        });
         return;
       }
 
@@ -311,11 +309,9 @@ class PaymentController {
           paymentOwner: payment.userId,
           requestId,
         });
-        res
-          .status(403)
-          .json({
-            message: "Forbidden: You do not have access to this payment",
-          });
+        res.status(403).json({
+          message: "Forbidden: You do not have access to this payment",
+        });
         return;
       }
 

e2e/accounting.spec.ts

@@ -15,16 +15,14 @@ test.describe("Accounting and Reporting Flow", () => {
 
   test.beforeEach(async ({ page }) => {
     // Use saved authentication state
-    await page
-      .context()
-      .addCookies([
-        {
-          name: "accessToken",
-          value: "mock_access_token",
-          domain: "localhost",
-          path: "/",
-        },
-      ]);
+    await page.context().addCookies([
+      {
+        name: "accessToken",
+        value: "mock_access_token",
+        domain: "localhost",
+        path: "/",
+      },
+    ]);
     await page.goto("http://localhost:3000/accounting");
   });
 

e2e/dashboard.spec.ts

@@ -15,16 +15,14 @@ test.describe("Dashboard Functionality", () => {
 
   test.beforeEach(async ({ page }) => {
     // Use saved authentication state
-    await page
-      .context()
-      .addCookies([
-        {
-          name: "accessToken",
-          value: "mock_access_token",
-          domain: "localhost",
-          path: "/",
-        },
-      ]);
+    await page.context().addCookies([
+      {
+        name: "accessToken",
+        value: "mock_access_token",
+        domain: "localhost",
+        path: "/",
+      },
+    ]);
     await page.goto("http://localhost:3000/dashboard");
   });
 

e2e/payment.spec.ts

@@ -15,16 +15,14 @@ test.describe("Payment Processing Flow", () => {
 
   test.beforeEach(async ({ page }) => {
     // Use saved authentication state
-    await page
-      .context()
-      .addCookies([
-        {
-          name: "accessToken",
-          value: "mock_access_token",
-          domain: "localhost",
-          path: "/",
-        },
-      ]);
+    await page.context().addCookies([
+      {
+        name: "accessToken",
+        value: "mock_access_token",
+        domain: "localhost",
+        path: "/",
+      },
+    ]);
     await page.goto("http://localhost:3000/payments");
   });