test(js): Add the connected groups to existing tests

arawa · Dec 11, 2024 · 2ee5544 · 2ee5544
1 parent 4c5a027
commit 2ee5544
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 7 deletions.
diff --git a/src/tests/unit/groupfoldersService.test.js b/src/tests/unit/groupfoldersService.test.js
@@ -20,6 +20,7 @@
  *
  */
 
+import { expect } from '@jest/globals'
 import { getAll, formatGroups, formatUsers, checkGroupfolderNameExist, enableAcl, addGroupToGroupfolder, addGroupToManageACLForGroupfolder, removeGroupToManageACLForGroupfolder, createGroupfolder, destroy, rename } from '../../services/groupfoldersService.js'
 import axios from '@nextcloud/axios'
 
@@ -285,11 +286,33 @@ describe('destroy', () => {
 		jest.resetAllMocks()
 	})
 	it('calls axios.delete method with proper parameters', async () => {
-		axios.delete.mockResolvedValue(responseValue)
-		await destroy('foobar')
-		expect(axios.delete).toHaveBeenCalledWith('/apps/workspace/api/delete/space', {
-			data: { workspace: 'foobar' },
+		const workspace = { id: 1, groupfolderId: 1 }
+		const spaceId = workspace.id
+
+		axios.delete.mockResolvedValue(
+			{
+				http: {
+					statuscode: 200,
+					message: 'The space is deleted.'
+				},
+				data: { 
+					name: 'foobar',
+					groups: ['SPACE-WM-1', 'SPACE-U-1'],
+          space_id: 1,
+          groupfolder_id: 1,
+          state: 'delete'
+				}
+			}
+		)
+		axios.delete.mockResolvedValue({ status: 200, data: { ocs: { meta: { status: 'ok' } } } })
+
+		await destroy(workspace)
+
+		expect(axios.delete).toHaveBeenCalledWith(`/apps/workspace/spaces/${spaceId}`, {
+			data: { workspace },
 		})
+
+		expect(axios.delete).toHaveBeenCalledWith(`/apps/groupfolders/folders/${workspace.groupfolderId}`)
 	})
 	it('calls axios.delete 2 times and returns resp.data value', async () => {
 		const mockAxios = axios.delete.mockImplementationOnce(() => Promise.resolve({ status: 200, ...responseValue }))

diff --git a/src/tests/unit/selectUsers.test.js b/src/tests/unit/selectUsers.test.js
@@ -83,7 +83,7 @@ describe('SelectUsers component tests', () => {
 		]
 
 		wrappedSelectUsers.vm.$route.params.space = 'foobar'
-		wrappedSelectUsers.vm.$route.params.group = 'subgroup-42'
+		wrappedSelectUsers.vm.$route.params.slug = 'subgroup-42'
 
 		wrappedSelectUsers.vm.addUsersToWorkspaceOrGroup()
 		const count = wrappedSelectUsers.vm.$store.getters.groupUserCount('foobar', 'subgroup-42')
@@ -102,7 +102,7 @@ describe('SelectUsers component tests', () => {
 			},
 		]
 
-		wrappedSelectUsers.vm.$route.params.group = undefined
+		wrappedSelectUsers.vm.$route.params.slug = undefined
 
 		wrappedSelectUsers.vm.addUsersToWorkspaceOrGroup()
 		const count = wrappedSelectUsers.vm.$store.getters.groupUserCount('foobar', 'SPACE-U-42')
@@ -123,7 +123,7 @@ describe('SelectUsers component tests', () => {
 				},
 			]
 
-			wrappedSelectUsers.vm.$route.params.group = undefined
+			wrappedSelectUsers.vm.$route.params.slug = undefined
 
 			wrappedSelectUsers.vm.addUsersToWorkspaceOrGroup()
 			const count = wrappedSelectUsers.vm.$store.getters.groupUserCount('foobar', 'SPACE-GE-42')

diff --git a/src/tests/unit/store.test.js b/src/tests/unit/store.test.js
@@ -81,11 +81,15 @@ describe('Vuex store tests', () => {
 		mutations.addGroupToSpace(state, {
 			name: 'test-space',
 			gid: 'test-group',
+      slug: 'test-group',
+      usersCount: 0,
 		})
 
 		expect(state.spaces['test-space'].groups['test-group']).toEqual({
 			gid: 'test-group',
 			displayName: 'test-group',
+      slug: 'test-group',
+      usersCount: 0,
 		})
 	})