Skip to content

Commit c144c04

Browse files
committed
fix: modify tests for course card and import paths
1 parent 0a52025 commit c144c04

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

src/containers/CourseCard/components/CourseCardActions/index.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,11 @@ import { ActionRow } from '@openedx/paragon';
55

66
import { reduxHooks } from 'hooks';
77

8+
import CourseCardActionSlot from 'plugin-slots/CourseCardActionSlot';
89
import SelectSessionButton from './SelectSessionButton';
910
import BeginCourseButton from './BeginCourseButton';
1011
import ResumeButton from './ResumeButton';
1112
import ViewCourseButton from './ViewCourseButton';
12-
import CourseCardActionSlot from '../../../../plugin-slots/CourseCardActionSlot';
1313

1414
export const CourseCardActions = ({ cardId }) => {
1515
const { isEntitlement, isFulfilled } = reduxHooks.useCardEntitlementData(cardId);

src/containers/CourseCard/components/CourseCardActions/index.test.jsx

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import { shallow } from '@edx/react-unit-test-utils';
22

33
import { reduxHooks } from 'hooks';
44

5+
import CourseCardActionSlot from 'plugin-slots/CourseCardActionSlot';
56
import UpgradeButton from './UpgradeButton';
67
import SelectSessionButton from './SelectSessionButton';
78
import BeginCourseButton from './BeginCourseButton';
@@ -19,6 +20,7 @@ jest.mock('hooks', () => ({
1920
},
2021
}));
2122

23+
jest.mock('plugin-slots/CourseCardActionSlot', () => 'CustomActionButton');
2224
jest.mock('./UpgradeButton', () => 'UpgradeButton');
2325
jest.mock('./SelectSessionButton', () => 'SelectSessionButton');
2426
jest.mock('./ViewCourseButton', () => 'ViewCourseButton');
@@ -88,26 +90,26 @@ describe('CourseCardActions', () => {
8890
expect(el.instance.findByType(UpgradeButton).length).toEqual(0);
8991
});
9092
});
91-
describe('not entielement, verified, or exec ed', () => {
93+
describe('not entitlement, verified, or exec ed', () => {
9294
it('renders UpgradeButton and ViewCourseButton for archived courses', () => {
9395
mockHooks({ isArchived: true });
9496
render();
95-
expect(el.instance.findByType(UpgradeButton)[0].props.cardId).toEqual(cardId);
97+
expect(el.instance.findByType(CourseCardActionSlot)[0].props.cardId).toEqual(cardId);
9698
expect(el.instance.findByType(ViewCourseButton)[0].props.cardId).toEqual(cardId);
9799
});
98100
describe('unstarted courses', () => {
99101
it('renders UpgradeButton and BeginCourseButton', () => {
100102
mockHooks();
101103
render();
102-
expect(el.instance.findByType(UpgradeButton)[0].props.cardId).toEqual(cardId);
104+
expect(el.instance.findByType(CourseCardActionSlot)[0].props.cardId).toEqual(cardId);
103105
expect(el.instance.findByType(BeginCourseButton)[0].props.cardId).toEqual(cardId);
104106
});
105107
});
106108
describe('active courses (started, and not archived)', () => {
107109
it('renders UpgradeButton and ResumeButton', () => {
108110
mockHooks({ hasStarted: true });
109111
render();
110-
expect(el.instance.findByType(UpgradeButton)[0].props.cardId).toEqual(cardId);
112+
expect(el.instance.findByType(CourseCardActionSlot)[0].props.cardId).toEqual(cardId);
111113
expect(el.instance.findByType(ResumeButton)[0].props.cardId).toEqual(cardId);
112114
});
113115
});

src/plugin-slots/CourseCardActionSlot/index.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import PropTypes from 'prop-types';
33
import { PluginSlot } from '@openedx/frontend-plugin-framework';
44

55
import { reduxHooks } from 'hooks';
6-
import UpgradeButton from '../../containers/CourseCard/components/CourseCardActions/UpgradeButton';
6+
import UpgradeButton from 'containers/CourseCard/components/CourseCardActions/UpgradeButton';
77

88
const CourseCardActionSlot = ({ cardId }) => {
99
const { isEntitlement } = reduxHooks.useCardEntitlementData(cardId);

0 commit comments

Comments
 (0)