Updates unit tests

island-is · Jun 4, 2024 · 6df8e20 · 6df8e20
1 parent f8339e7
commit 6df8e20
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 12 deletions.
diff --git a/...ackend/src/app/modules/file/test/limitedAccessFileController/createCaseFileGuards.spec.ts b/...ackend/src/app/modules/file/test/limitedAccessFileController/createCaseFileGuards.spec.ts
@@ -3,7 +3,11 @@ import {
   restrictionCases,
 } from '@island.is/judicial-system/types'
 
-import { CaseTypeGuard, CaseWriteGuard } from '../../../case'
+import {
+  CaseCompletedGuard,
+  CaseTypeGuard,
+  CaseWriteGuard,
+} from '../../../case'
 import { LimitedAccessWriteCaseFileGuard } from '../../guards/limitedAccessWriteCaseFile.guard'
 import { LimitedAccessFileController } from '../../limitedAccessFile.controller'
 
@@ -19,12 +23,13 @@ describe('LimitedAccessFileController - Create case file guards', () => {
   })
 
   it('should have the right guard configuration', () => {
-    expect(guards).toHaveLength(3)
+    expect(guards).toHaveLength(4)
     expect(guards[0]).toBeInstanceOf(CaseTypeGuard)
     expect(guards[0]).toEqual({
       allowedCaseTypes: [...restrictionCases, ...investigationCases],
     })
     expect(new guards[1]()).toBeInstanceOf(CaseWriteGuard)
-    expect(new guards[2]()).toBeInstanceOf(LimitedAccessWriteCaseFileGuard)
+    expect(new guards[2]()).toBeInstanceOf(CaseCompletedGuard)
+    expect(new guards[3]()).toBeInstanceOf(LimitedAccessWriteCaseFileGuard)
   })
 })
diff --git a/...d/src/app/modules/file/test/limitedAccessFileController/createPresignedPostGuards.spec.ts b/...d/src/app/modules/file/test/limitedAccessFileController/createPresignedPostGuards.spec.ts
@@ -3,7 +3,11 @@ import {
   restrictionCases,
 } from '@island.is/judicial-system/types'
 
-import { CaseTypeGuard, CaseWriteGuard } from '../../../case'
+import {
+  CaseCompletedGuard,
+  CaseTypeGuard,
+  CaseWriteGuard,
+} from '../../../case'
 import { LimitedAccessFileController } from '../../limitedAccessFile.controller'
 
 describe('LimitedAccessFileController - Create presigned post guards', () => {
@@ -18,11 +22,12 @@ describe('LimitedAccessFileController - Create presigned post guards', () => {
   })
 
   it('should have the right guard configuration', () => {
-    expect(guards).toHaveLength(2)
+    expect(guards).toHaveLength(3)
     expect(guards[0]).toBeInstanceOf(CaseTypeGuard)
     expect(guards[0]).toEqual({
       allowedCaseTypes: [...restrictionCases, ...investigationCases],
     })
     expect(new guards[1]()).toBeInstanceOf(CaseWriteGuard)
+    expect(new guards[2]()).toBeInstanceOf(CaseCompletedGuard)
   })
 })
diff --git a/...ackend/src/app/modules/file/test/limitedAccessFileController/deleteCaseFileGuards.spec.ts b/...ackend/src/app/modules/file/test/limitedAccessFileController/deleteCaseFileGuards.spec.ts
@@ -3,7 +3,11 @@ import {
   restrictionCases,
 } from '@island.is/judicial-system/types'
 
-import { CaseTypeGuard, CaseWriteGuard } from '../../../case'
+import {
+  CaseCompletedGuard,
+  CaseTypeGuard,
+  CaseWriteGuard,
+} from '../../../case'
 import { CaseFileExistsGuard } from '../../guards/caseFileExists.guard'
 import { LimitedAccessWriteCaseFileGuard } from '../../guards/limitedAccessWriteCaseFile.guard'
 import { LimitedAccessFileController } from '../../limitedAccessFile.controller'
@@ -20,13 +24,14 @@ describe('LimitedAccessFileController - Delete case file guards', () => {
   })
 
   it('should have the right guard configuration', () => {
-    expect(guards).toHaveLength(4)
+    expect(guards).toHaveLength(5)
     expect(guards[0]).toBeInstanceOf(CaseTypeGuard)
     expect(guards[0]).toEqual({
       allowedCaseTypes: [...restrictionCases, ...investigationCases],
     })
     expect(new guards[1]()).toBeInstanceOf(CaseWriteGuard)
-    expect(new guards[2]()).toBeInstanceOf(CaseFileExistsGuard)
-    expect(new guards[3]()).toBeInstanceOf(LimitedAccessWriteCaseFileGuard)
+    expect(new guards[2]()).toBeInstanceOf(CaseCompletedGuard)
+    expect(new guards[3]()).toBeInstanceOf(CaseFileExistsGuard)
+    expect(new guards[4]()).toBeInstanceOf(LimitedAccessWriteCaseFileGuard)
   })
 })
diff --git a/...p/modules/file/test/limitedAccessFileController/limitedAccessFileControllerGuards.spec.ts b/...p/modules/file/test/limitedAccessFileController/limitedAccessFileControllerGuards.spec.ts
@@ -1,6 +1,6 @@
 import { JwtAuthGuard, RolesGuard } from '@island.is/judicial-system/auth'
 
-import { CaseCompletedGuard, LimitedAccessCaseExistsGuard } from '../../../case'
+import { LimitedAccessCaseExistsGuard } from '../../../case'
 import { LimitedAccessFileController } from '../../limitedAccessFile.controller'
 
 describe('LimitedAccessFileController - guards', () => {
@@ -12,10 +12,9 @@ describe('LimitedAccessFileController - guards', () => {
   })
 
   it('should have the right guard configuration', () => {
-    expect(guards).toHaveLength(4)
+    expect(guards).toHaveLength(3)
     expect(new guards[0]()).toBeInstanceOf(JwtAuthGuard)
     expect(new guards[1]()).toBeInstanceOf(RolesGuard)
     expect(new guards[2]()).toBeInstanceOf(LimitedAccessCaseExistsGuard)
-    expect(new guards[3]()).toBeInstanceOf(CaseCompletedGuard)
   })
 })