10502: fix api tests

ustaxcourt · Feb 11, 2025 · 403100a · 403100a
1 parent 0e5bcac
commit 403100a
Show file tree

Hide file tree

Showing 3 changed files with 175 additions and 239 deletions.
diff --git a/...c/business/useCases/courtIssuedDocument/fileAndServeCourtIssuedDocumentInteractor.test.ts b/...c/business/useCases/courtIssuedDocument/fileAndServeCourtIssuedDocumentInteractor.test.ts
@@ -1,3 +1,4 @@
+import '@web-api/persistence/postgres/cases/mocks.jest';
 import {
   AUTO_GENERATED_DEADLINE_DOCUMENT_TYPES,
   DOCKET_SECTION,
@@ -14,6 +15,9 @@ import {
 import { fileAndServeCourtIssuedDocumentInteractor } from './fileAndServeCourtIssuedDocumentInteractor';
 import { mockDocketClerkUser } from '@shared/test/mockAuthUsers';
 import { testPdfDoc } from '../../../../../shared/src/business/test/getFakeFile';
+import { getCaseByDocketNumber as getCaseByDocketNumberMock } from '@web-api/persistence/postgres/cases/getCaseByDocketNumber';
+
+const getCaseByDocketNumber = getCaseByDocketNumberMock as jest.Mock;
 
 let MOCK_DATE;
 
@@ -92,9 +96,7 @@ describe('fileAndServeCourtIssuedDocumentInteractor', () => {
       .getPersistenceGateway()
       .getUserById.mockReturnValue(docketClerkUser);
 
-    applicationContext
-      .getPersistenceGateway()
-      .getCaseByDocketNumber.mockReturnValue(caseRecord);
+    getCaseByDocketNumber.mockReturnValue(caseRecord);
 
     applicationContext
       .getUseCaseHelpers()
@@ -274,9 +276,8 @@ describe('fileAndServeCourtIssuedDocumentInteractor', () => {
 
     caseRecord.docketEntries.push(mockAutoGeneratedDeadlineDocketEntry);
 
-    applicationContext
-      .getPersistenceGateway()
-      .getCaseByDocketNumber.mockReturnValueOnce(caseRecord)
+    getCaseByDocketNumber
+      .mockReturnValueOnce(caseRecord)
       .mockReturnValueOnce(nonSubjectCaseRecord);
 
     const mockAutoGeneratedDeadlineForm = {

diff --git a/web-api/src/business/useCases/courtIssuedOrder/fileCourtIssuedOrderInteractor.test.ts b/web-api/src/business/useCases/courtIssuedOrder/fileCourtIssuedOrderInteractor.test.ts
@@ -22,6 +22,11 @@ import {
   mockPetitionerUser,
 } from '@shared/test/mockAuthUsers';
 import { updateMessage } from '@web-api/persistence/postgres/messages/updateMessage';
+import { getCaseByDocketNumber as getCaseByDocketNumberMock } from '@web-api/persistence/postgres/cases/getCaseByDocketNumber';
+import { updateCase as updateCaseMock } from '@web-api/persistence/postgres/cases/updateCase';
+
+const getCaseByDocketNumber = getCaseByDocketNumberMock as jest.Mock;
+const updateCase = updateCaseMock as jest.Mock;
 
 /* eslint-disable max-lines */
 describe('fileCourtIssuedOrderInteractor', () => {
@@ -101,9 +106,7 @@ describe('fileCourtIssuedOrderInteractor', () => {
       }),
     );
 
-    applicationContext
-      .getPersistenceGateway()
-      .getCaseByDocketNumber.mockReturnValue(caseRecord);
+    getCaseByDocketNumber.mockReturnValue(caseRecord);
   });
 
   it('should throw an error if not authorized', async () => {
@@ -140,12 +143,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
       mockDocketClerkUser,
     );
 
+    expect(getCaseByDocketNumber).toHaveBeenCalled();
     expect(
-      applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-    ).toHaveBeenCalled();
-    expect(
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries.length,
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries.length,
     ).toEqual(4);
   });
 
@@ -174,16 +174,16 @@ describe('fileCourtIssuedOrderInteractor', () => {
     );
 
     expect(
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries[3].draftOrderState.documentContents,
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3].draftOrderState
+        .documentContents,
     ).toBeUndefined();
     expect(
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries[3].draftOrderState.editorDelta,
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3].draftOrderState
+        .editorDelta,
     ).toBeUndefined();
     expect(
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries[3].draftOrderState.richText,
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3].draftOrderState
+        .richText,
     ).toBeUndefined();
   });
 
@@ -205,16 +205,11 @@ describe('fileCourtIssuedOrderInteractor', () => {
       mockDocketClerkUser,
     );
 
+    expect(updateCase).toHaveBeenCalled();
     expect(
-      applicationContext.getPersistenceGateway().updateCase,
-    ).toHaveBeenCalled();
-    expect(
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries.length,
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries.length,
     ).toEqual(4);
-    const result =
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries[3];
+    const result = updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3];
     expect(result).toMatchObject({ freeText: 'Notice to be nice' });
     expect(result.signedAt).toBeTruthy();
   });
@@ -244,12 +239,11 @@ describe('fileCourtIssuedOrderInteractor', () => {
       useTempBucket: false,
     });
     expect(
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries[3].documentContents,
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3]
+        .documentContents,
     ).toBeUndefined();
     expect(
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries[3],
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
     ).toMatchObject({
       documentContentsId: expect.anything(),
       draftOrderState: {},
@@ -403,12 +397,10 @@ describe('fileCourtIssuedOrderInteractor', () => {
     );
 
     const lastDocumentIndex =
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries.length - 1;
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries.length - 1;
 
     const newlyFiledDocument =
-      applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-        .caseToUpdate.docketEntries[lastDocumentIndex];
+      updateCase.mock.calls[0][0].caseToUpdate.docketEntries[lastDocumentIndex];
 
     expect(newlyFiledDocument).toMatchObject({
       isDraft: true,
@@ -437,9 +429,7 @@ describe('fileCourtIssuedOrderInteractor', () => {
       ),
     ).rejects.toThrow(ServiceUnavailableError);
 
-    expect(
-      applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-    ).not.toHaveBeenCalled();
+    expect(getCaseByDocketNumber).not.toHaveBeenCalled();
   });
 
   it('should acquire and remove the lock on the case', async () => {
@@ -497,16 +487,12 @@ describe('fileCourtIssuedOrderInteractor', () => {
           mockDocketClerkUser,
         );
 
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries.length,
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries.length,
         ).toEqual(4);
         expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: { freeText: 'Order to do anything' },
           freeText: 'Order to do anything',
@@ -532,12 +518,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           mockDocketClerkUser,
         );
 
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText: 'Order parties by 11/05/2024 shall file a status report.',
@@ -563,12 +546,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           mockDocketClerkUser,
         );
 
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText:
@@ -598,12 +578,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           mockDocketClerkUser,
         );
 
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText:
@@ -631,12 +608,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           mockDocketClerkUser,
         );
 
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText:
@@ -664,12 +638,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           mockDocketClerkUser,
         );
 
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText:
@@ -699,12 +670,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           mockDocketClerkUser,
         );
 
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText:
@@ -736,12 +704,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           },
           mockDocketClerkUser,
         );
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText:
@@ -771,12 +736,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           },
           mockDocketClerkUser,
         );
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText:
@@ -803,12 +765,9 @@ describe('fileCourtIssuedOrderInteractor', () => {
           },
           mockDocketClerkUser,
         );
+        expect(getCaseByDocketNumber).toHaveBeenCalled();
         expect(
-          applicationContext.getPersistenceGateway().getCaseByDocketNumber,
-        ).toHaveBeenCalled();
-        expect(
-          applicationContext.getPersistenceGateway().updateCase.mock.calls[0][0]
-            .caseToUpdate.docketEntries[3],
+          updateCase.mock.calls[0][0].caseToUpdate.docketEntries[3],
         ).toMatchObject({
           draftOrderState: {
             freeText: '. Case is stricken from the current trial session.',