Skip to content

Commit a87c3f3

Browse files
farhanclaude
andcommitted
test: update tests to reflect removed waffle flag conditionals
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent ee8588e commit a87c3f3

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

plugins/course-apps/proctoring/Settings.test.jsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -471,9 +471,8 @@ describe('ProctoredExamSettings', () => {
471471
screen.getByDisplayValue('mockproc');
472472
});
473473
// (1) for studio settings
474-
// (2) waffle flags
475-
// (3) for course details
476-
expect(axiosMock.history.get.length).toBe(3);
474+
// (2) for course details
475+
expect(axiosMock.history.get.length).toBe(2);
477476
expect(axiosMock.history.get[0].url.includes('proctored_exam_settings')).toEqual(true);
478477
});
479478

src/header/hooks.test.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -188,8 +188,8 @@ describe('header utils', () => {
188188
const actualItemsTitle = actualItems.map((item) => item.title);
189189
expect(actualItemsTitle).toContain(messages['header.links.updates'].defaultMessage);
190190
});
191-
it('when useNewUpdatesPage is false should use legacy studio URL for updates', () => {
192-
mockWaffleFlags({ enableAuthzCourseAuthoring: false, useNewUpdatesPage: false });
191+
it('should always use MFE URL for updates', () => {
192+
mockWaffleFlags({ enableAuthzCourseAuthoring: false });
193193
jest.mocked(useCourseUserPermissions).mockReturnValue({
194194
isLoading: false,
195195
isAuthzEnabled: false,
@@ -201,7 +201,7 @@ describe('header utils', () => {
201201
const actualItems =
202202
renderHook(() => useContentMenuItems('course-123'), { wrapper: createWrapper() }).result.current;
203203
const updatesItem = actualItems.find((item) => item.title === messages['header.links.updates'].defaultMessage);
204-
expect(updatesItem?.href).toContain('/course_info/course-123');
204+
expect(updatesItem?.href).toContain('/course/course-123/course_info');
205205
});
206206
it('when authz enabled and user has canViewPagesAndResources should include pages and resources option', async () => {
207207
mockWaffleFlags({ enableAuthzCourseAuthoring: true });

0 commit comments

Comments
 (0)