test: remove references to deleted legacy waffle flags from test files

Updates apiHooks.test.tsx to use useReactMarkdownEditor as the example
flag (still present, default true). Removes useNewCourseOutlinePage and
useNewVideoUploadsPage from all test mocks and HTTP response fixtures.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: farhan

src/course-checklist/ChecklistSection/ChecklistSection.test.jsx

@@ -39,9 +39,7 @@ describe('ChecklistSection', () => {
     const { axiosMock } = initializeMocks();
     axiosMock
       .onGet(getApiWaffleFlagsUrl(courseId))
-      .reply(200, {
-        useNewCourseOutlinePage: true,
-      });
+      .reply(200, {});
   });
 
   it('a heading using the dataHeading prop', () => {

src/course-unit/breadcrumbs/Breadcrumbs.test.tsx

@@ -53,7 +53,7 @@ describe('<Breadcrumbs />', () => {
     await executeThunk(fetchCourseSectionVerticalData(courseId), reduxStore.dispatch);
     axiosMock
       .onGet(getApiWaffleFlagsUrl(courseId))
-      .reply(200, { useNewCourseOutlinePage: true });
+      .reply(200, {});
   });
 
   it('render Breadcrumbs component correctly', async () => {

src/custom-pages/CustomPages.test.tsx

@@ -61,9 +61,7 @@ describe('CustomPages', () => {
     axiosMock = mocks.axiosMock;
     axiosMock
       .onGet(getApiWaffleFlagsUrl(courseId))
-      .reply(200, {
-        useNewCourseOutlinePage: true,
-      });
+      .reply(200, {});
   });
   it('should ', async () => {
     renderComponent();

src/data/apiHooks.mock.ts

@@ -7,7 +7,7 @@ import * as apiHooks from './apiHooks';
  * loading; if you need more realistic handling, use:
  *   axiosMock
  *    .onGet(getApiWaffleFlagsUrl(courseId))
- *    .reply(200, { useNewCourseOutlinePage: true }); // etc
+ *    .reply(200, { enableCourseOptimizer: true }); // etc
  */
 export function mockWaffleFlags(overrides: Partial<Record<WaffleFlagName, boolean>> = {}) {
   return jest.spyOn(apiHooks, 'useWaffleFlags').mockImplementation(() => ({

src/data/apiHooks.test.tsx

@@ -15,7 +15,7 @@ const FlagComponent = ({ courseId }: { courseId?: string; }) => {
     <ul>
       <li aria-label="isLoading">{waffleFlags.isLoading ? 'loading' : 'false'}</li>
       <li aria-label="isError">{waffleFlags.isError ? 'error' : 'false'}</li>
-      <li aria-label="useNewCourseOutlinePage">{waffleFlags.useNewCourseOutlinePage ? 'enabled' : 'disabled'}</li>
+      <li aria-label="useReactMarkdownEditor">{waffleFlags.useReactMarkdownEditor ? 'enabled' : 'disabled'}</li>
     </ul>
   );
 };
@@ -34,17 +34,17 @@ describe('useWaffleFlags', () => {
     expect(await screen.findByLabelText('isLoading')).toHaveTextContent('loading');
     expect(await screen.findByLabelText('isError')).toHaveTextContent('false');
     // The default should be enabled, even before we hear back from the server:
-    expect(await screen.findByLabelText('useNewCourseOutlinePage')).toHaveTextContent('enabled');
+    expect(await screen.findByLabelText('useReactMarkdownEditor')).toHaveTextContent('enabled');
 
     // Then, the server responds with a new value:
-    resolveResponse([200, { useNewCourseOutlinePage: false }]);
+    resolveResponse([200, { useReactMarkdownEditor: false }]);
 
     // Now, we're no longer loading and we have the new value:
     await waitFor(async () => {
       expect(await screen.findByLabelText('isLoading')).toHaveTextContent('false');
     });
     expect(await screen.findByLabelText('isError')).toHaveTextContent('false');
-    expect(await screen.findByLabelText('useNewCourseOutlinePage')).toHaveTextContent('disabled');
+    expect(await screen.findByLabelText('useReactMarkdownEditor')).toHaveTextContent('disabled');
   });
 
   it('uses the default values if there\'s an error', async () => {
@@ -60,7 +60,7 @@ describe('useWaffleFlags', () => {
     expect(await screen.findByLabelText('isLoading')).toHaveTextContent('loading');
     expect(await screen.findByLabelText('isError')).toHaveTextContent('false');
     // The default should be enabled, even before we hear back from the server:
-    expect(await screen.findByLabelText('useNewCourseOutlinePage')).toHaveTextContent('enabled');
+    expect(await screen.findByLabelText('useReactMarkdownEditor')).toHaveTextContent('enabled');
 
     // Then, the server responds with an error
     resolveResponse([500, {}]);
@@ -70,14 +70,14 @@ describe('useWaffleFlags', () => {
       expect(await screen.findByLabelText('isLoading')).toHaveTextContent('false');
     });
     expect(await screen.findByLabelText('isError')).toHaveTextContent('error');
-    expect(await screen.findByLabelText('useNewCourseOutlinePage')).toHaveTextContent('enabled');
+    expect(await screen.findByLabelText('useReactMarkdownEditor')).toHaveTextContent('enabled');
   });
 
   it('uses the global flag values while loading the course-specific flags', async () => {
     const { axiosMock } = initializeMocks();
     const courseId = 'course-v1:A+b+C';
     // Set the global flag OFF:
-    axiosMock.onGet(getApiWaffleFlagsUrl()).reply(200, { useNewCourseOutlinePage: false });
+    axiosMock.onGet(getApiWaffleFlagsUrl()).reply(200, { useReactMarkdownEditor: false });
     // Control when we respond with the course-specific flag value:
     let resolveResponse;
     const promise = new Promise<[number, unknown]>(resolve => {
@@ -89,7 +89,7 @@ describe('useWaffleFlags', () => {
     render(<FlagComponent />);
     await waitFor(async () => {
       // Once it loads the flags from the server, the global 'false' value will override the default 'true':
-      expect(await screen.findByLabelText('useNewCourseOutlinePage')).toHaveTextContent('disabled');
+      expect(await screen.findByLabelText('useReactMarkdownEditor')).toHaveTextContent('disabled');
     });
 
     // Now check the course-specific flag:
@@ -99,14 +99,14 @@ describe('useWaffleFlags', () => {
     // Now, the course-specific value is loading but in the meantime we use the global default:
     expect(await screen.findByLabelText('isLoading')).toHaveTextContent('loading');
     expect(await screen.findByLabelText('isError')).toHaveTextContent('false');
-    expect(await screen.findByLabelText('useNewCourseOutlinePage')).toHaveTextContent('disabled');
+    expect(await screen.findByLabelText('useReactMarkdownEditor')).toHaveTextContent('disabled');
 
     // Now the server responds: the course-specific flag is ON:
-    resolveResponse([200, { useNewCourseOutlinePage: true }]);
+    resolveResponse([200, { useReactMarkdownEditor: true }]);
     await waitFor(async () => {
       expect(await screen.findByLabelText('isLoading')).toHaveTextContent('false');
     });
     expect(await screen.findByLabelText('isError')).toHaveTextContent('false');
-    expect(await screen.findByLabelText('useNewCourseOutlinePage')).toHaveTextContent('enabled');
+    expect(await screen.findByLabelText('useReactMarkdownEditor')).toHaveTextContent('enabled');
   });
 });

src/header/hooks.test.tsx

@@ -85,7 +85,6 @@ describe('header utils', () => {
     it('when video upload page disabled should not include Video Uploads option', () => {
       mockWaffleFlags({
         enableAuthzCourseAuthoring: false,
-        useNewVideoUploadsPage: false,
       });
       jest.mocked(useCourseUserPermissions).mockReturnValue({
         isLoading: false,
@@ -237,7 +236,6 @@ describe('header utils', () => {
     beforeAll(() => {
       mockWaffleFlags({
         enableAuthzCourseAuthoring: false,
-        useNewVideoUploadsPage: false,
       });
     });
     beforeEach(() => {

src/pages-and-resources/pages/PageCard.test.jsx

@@ -50,9 +50,7 @@ describe('LiveSettings', () => {
     axiosMock = mocks.axiosMock;
     axiosMock
       .onGet(getApiWaffleFlagsUrl(courseId))
-      .reply(200, {
-        useNewCourseOutlinePage: true,
-      });
+      .reply(200, {});
   });
 
   it('should render three cards', async () => {