fix: narrow response type in test and fix ErrorBoundary error format

Amp-Thread-ID: https://ampcode.com/threads/T-019ca09b-2eda-7357-9a28-71f3648d5c02
Co-authored-by: Amp <amp@ampcode.com>

Author: wheeljackz

app/routes/__tests__/dashboard.transactions.$transactionId.$action.test.ts

@@ -87,9 +87,9 @@ describe('dashboard.transactions.$transactionId.$action action', () => {
       expect(error).toBeInstanceOf(Response)
       const response = error as Response
       expect(response.status).toBe(400)
-      await expect(response.json()).resolves.toMatchObject({
-        formError: 'Status transaksi tidak valid.',
-      })
+      await expect(response.json()).resolves.toBe(
+        'Status transaksi tidak valid.'
+      )
     }
 
     expect(updateTransactionStatus).not.toHaveBeenCalled()
@@ -109,10 +109,9 @@ describe('dashboard.transactions.$transactionId.$action action', () => {
       expect(error).toBeInstanceOf(Response)
       const response = error as Response
       expect(response.status).toBe(400)
-      await expect(response.json()).resolves.toMatchObject({
-        formError:
-          'Transaksi yang sudah diverifikasi tidak dapat diubah menjadi ditolak.',
-      })
+      await expect(response.json()).resolves.toBe(
+        'Transaksi yang sudah diverifikasi tidak dapat diubah menjadi ditolak.'
+      )
     }
 
     expect(updateTransactionStatus).not.toHaveBeenCalled()
@@ -122,6 +121,7 @@ describe('dashboard.transactions.$transactionId.$action action', () => {
     const response = await runAction(TRANSACTION_STATUS.VERIFIED)
 
     expect(response).toBeInstanceOf(Response)
+    if (!(response instanceof Response)) throw new Error('Expected Response')
     expect(response.status).toBe(302)
     expect(response.headers.get('Location')).toBe(
       '/dashboard/transactions/tx-1'

app/routes/dashboard.transactions.$transactionId.$action.tsx

@@ -80,12 +80,7 @@ export const action: ActionFunction = async ({ request, params }) => {
     status !== TRANSACTION_STATUS.VERIFIED &&
     status !== TRANSACTION_STATUS.REJECTED
   ) {
-    throw json(
-      {
-        formError: 'Status transaksi tidak valid.',
-      },
-      { status: 400 }
-    )
+    throw json('Status transaksi tidak valid.', { status: 400 })
   }
 
   const existingTransaction = await getTransactionById(transactionId)
@@ -100,10 +95,7 @@ export const action: ActionFunction = async ({ request, params }) => {
     )
   ) {
     throw json(
-      {
-        formError:
-          'Transaksi yang sudah diverifikasi tidak dapat diubah menjadi ditolak.',
-      },
+      'Transaksi yang sudah diverifikasi tidak dapat diubah menjadi ditolak.',
       { status: 400 }
     )
   }