From 9c78f639e0bf7f645391c2e67be4588f55943752 Mon Sep 17 00:00:00 2001
From: Philip Molares <philip.molares@udo.edu>
Date: Sun, 23 Oct 2022 20:31:17 +0200
Subject: [PATCH] test(groups): fix 'API requires authentication' test

Signed-off-by: Philip Molares <philip.molares@udo.edu>
---
 test/private-api/groups.e2e-spec.ts | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/test/private-api/groups.e2e-spec.ts b/test/private-api/groups.e2e-spec.ts
index f62093997..e719a08a7 100644
--- a/test/private-api/groups.e2e-spec.ts
+++ b/test/private-api/groups.e2e-spec.ts
@@ -5,6 +5,9 @@
  */
 import request from 'supertest';
 
+import { GuestAccess } from '../../src/config/guest_access.enum';
+import { createDefaultMockNoteConfig } from '../../src/config/mock/note.config.mock';
+import { NoteConfig } from '../../src/config/note.config';
 import { LoginDto } from '../../src/identity/local/login.dto';
 import {
   password1,
@@ -16,9 +19,14 @@ import {
 describe('Groups', () => {
   let testSetup: TestSetup;
   let testuser1Session: request.SuperAgentTest;
+  const noteConfigMock: NoteConfig = createDefaultMockNoteConfig();
 
   beforeEach(async () => {
-    testSetup = await TestSetupBuilder.create().withUsers().build();
+    testSetup = await TestSetupBuilder.create({
+      noteConfigMock: noteConfigMock,
+    })
+      .withUsers()
+      .build();
     await testSetup.app.init();
 
     // create a test group
@@ -57,10 +65,15 @@ describe('Groups', () => {
     expect(response.status).toBe(404);
   });
 
-  test('API requires authentication', async () => {
-    const response = await request(testSetup.app.getHttpServer()).get(
-      '/api/private/groups/testgroup1',
-    );
-    expect(response.status).toBe(401);
+  describe('API requires authentication', () => {
+    beforeAll(() => {
+      noteConfigMock.guestAccess = GuestAccess.DENY;
+    });
+    test('get group', async () => {
+      const response = await request(testSetup.app.getHttpServer()).get(
+        '/api/private/groups/testgroup1',
+      );
+      expect(response.status).toBe(401);
+    });
   });
 });