From a5f102e85afefd36d70f3460fdab5b6b95305e33 Mon Sep 17 00:00:00 2001 From: Lucas Di Pentima Date: Wed, 17 Mar 2021 18:41:35 -0300 Subject: [PATCH] 17119: Fixes unit tests improving store mocking and error reporting. Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima --- src/store/context-menu/context-menu-actions.test.ts | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/store/context-menu/context-menu-actions.test.ts b/src/store/context-menu/context-menu-actions.test.ts index 2778568e..2f4f38b3 100644 --- a/src/store/context-menu/context-menu-actions.test.ts +++ b/src/store/context-menu/context-menu-actions.test.ts @@ -6,6 +6,8 @@ import { ContextMenuKind } from '~/views-components/context-menu/context-menu'; import { resourceUuidToContextMenuKind } from './context-menu-actions'; import configureStore from 'redux-mock-store'; import thunk from 'redux-thunk'; +import { PROJECT_PANEL_CURRENT_UUID } from '../project-panel/project-panel-action'; +import { GroupClass } from '~/models/group'; describe('context-menu-actions', () => { describe('resourceUuidToContextMenuKind', () => { @@ -80,6 +82,9 @@ describe('context-menu-actions', () => { cases.forEach(([resourceUuid, isAdminUser, isEditable, isTrashed, expected]) => { const initialState = { + properties: { + [PROJECT_PANEL_CURRENT_UUID]: projectUuid, + }, resources: { [headCollectionUuid]: { uuid: headCollectionUuid, @@ -91,12 +96,12 @@ describe('context-menu-actions', () => { uuid: oldCollectionUuid, currentVersionUuid: headCollectionUuid, isTrashed: isTrashed, - }, [projectUuid]: { uuid: projectUuid, ownerUuid: isEditable ? userUuid : otherUserUuid, writableBy: isEditable ? [userUuid] : [otherUserUuid], + groupClass: GroupClass.PROJECT, }, [linkUuid]: { uuid: linkUuid, @@ -118,8 +123,9 @@ describe('context-menu-actions', () => { }; const store = mockStore(initialState); - const menuKind = store.dispatch(resourceUuidToContextMenuKind(resourceUuid as string)) + let menuKind: any; try { + menuKind = store.dispatch(resourceUuidToContextMenuKind(resourceUuid as string)) expect(menuKind).toBe(expected); } catch (err) { throw new Error(`menuKind for resource ${JSON.stringify(initialState.resources[resourceUuid as string])} expected to be ${expected} but got ${menuKind}.`); -- 2.30.2