Skip to content

Commit 737000c

Browse files
fix: correct destructuring for context hook call
1 parent 2911409 commit 737000c

2 files changed

Lines changed: 10 additions & 4 deletions

File tree

src/containers/CourseCard/components/CourseCardDetails/hooks.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ export const useCardDetailsData = ({ cardId }) => {
4747
isFulfilled,
4848
canChange,
4949
} = useEntitlementInfo(courseData);
50-
const updateSelectSessionModal = useSelectSessionModal();
50+
const { updateSelectSessionModal } = useSelectSessionModal();
5151

5252
return {
5353
providerName: providerName || formatMessage(messages.unknownProviderName),

src/containers/CourseCard/components/CourseCardDetails/hooks.test.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { useIntl } from '@edx/frontend-platform/i18n';
22

33
import { keyStore } from 'utils';
44
import { utilHooks, useCourseData } from 'hooks';
5-
5+
import { useSelectSessionModal } from 'data/context';
66
import * as hooks from './hooks';
77
import messages from './messages';
88

@@ -11,8 +11,9 @@ jest.mock('react', () => ({
1111
useMemo: (fn) => fn(),
1212
}));
1313

14-
jest.mock('data/context/SelectSessionProvider', () => ({
15-
useSelectSessionModal: jest.fn(() => jest.fn()),
14+
const updateSelectSessionModalMock = jest.fn().mockName('updateSelectSessionModal');
15+
jest.mock('data/context', () => ({
16+
useSelectSessionModal: jest.fn(),
1617
}));
1718
jest.mock('hooks', () => ({
1819
...jest.requireActual('hooks'),
@@ -68,6 +69,7 @@ describe('CourseCardDetails hooks', () => {
6869
courseRun: {},
6970
entitlement: { ...entitlementData, ...entitlement },
7071
});
72+
useSelectSessionModal.mockReturnValue({ updateSelectSessionModal: updateSelectSessionModalMock });
7173
out = hooks.useCardDetailsData({ cardId });
7274
};
7375
beforeEach(() => {
@@ -84,6 +86,10 @@ describe('CourseCardDetails hooks', () => {
8486
it('forward changeOrLeaveSessionMessage', () => {
8587
expect(out.changeOrLeaveSessionMessage).toEqual(formatMessage(messages.changeOrLeaveSessionButton));
8688
});
89+
it('calls updateSelectSessionModal when openSessionModal is called', () => {
90+
out.openSessionModal();
91+
expect(updateSelectSessionModalMock).toHaveBeenCalledWith(cardId);
92+
});
8793
});
8894

8995
describe('useAccessMessage', () => {

0 commit comments

Comments
 (0)