test: fix completely broken tests

Author: bradenmacdonald

src/generic/create-or-rerun-course/CreateOrRerunCourseForm.test.jsx

@@ -1,4 +1,3 @@
-import React from 'react';
 import {
   fireEvent,
   screen,
@@ -212,7 +211,7 @@ describe('<CreateOrRerunCourseForm />', () => {
     fireEvent.change(numberInput, { target: { value: 'long-name-which-is-longer-than-65-characters-to-check-for-errors' } });
 
     await waitFor(() => {
-      expect(screen.getByText(messages.totalLengthError)).toBeInTheDocument();
+      expect(screen.getByText(messages.totalLengthError.defaultMessage)).toBeInTheDocument();
     });
   });
 
@@ -278,9 +277,10 @@ describe('<CreateOrRerunCourseForm />', () => {
     const numberInput = await screen.findByPlaceholderText(messages.courseNumberPlaceholder.defaultMessage);
 
     fireEvent.change(numberInput, { target: { value: 'number with invalid (+) symbol' } });
+    fireEvent.blur(numberInput);
 
     await waitFor(() => {
-      expect(screen.getByText(messages.noSpaceError)).toBeInTheDocument();
+      expect(screen.getByText(messages.disallowedCharsError.defaultMessage)).toBeInTheDocument();
     });
   });
 });

…neric/create-or-rerun-course/messages.js …neric/create-or-rerun-course/messages.tssrc/generic/create-or-rerun-course/messages.js renamed to src/generic/create-or-rerun-course/messages.ts src/generic/create-or-rerun-course/messages.js renamed to src/generic/create-or-rerun-course/messages.ts

@@ -326,9 +326,10 @@ describe('Data layer integration tests', () => {
     test('successfully saves an LTI configuration', async () => {
       axiosMock.onGet(getDiscussionsProvidersUrl(courseId)).reply(200, generateProvidersApiResponse());
       axiosMock.onGet(getDiscussionsSettingsUrl(courseId)).reply(200, piazzaApiResponse);
+      // Note: if this test is failing, it's likely because the POSTed data has changed and no longer exactly matches
+      // the expected data in the mock below.
       axiosMock.onPost(getDiscussionsSettingsUrl(courseId), {
         context_key: courseId,
-        enabled: true,
         lti_configuration: {
           lti_1p1_client_key: 'new_consumer_key',
           lti_1p1_client_secret: 'new_consumer_secret',
@@ -388,9 +389,10 @@ describe('Data layer integration tests', () => {
     test('successfully saves a Legacy configuration', async () => {
       axiosMock.onGet(getDiscussionsProvidersUrl(courseId)).reply(200, generateProvidersApiResponse(false, 'legacy'));
       axiosMock.onGet(getDiscussionsSettingsUrl(courseId)).reply(200, legacyApiResponse);
+      // Note: if this test is failing, it's likely because the POSTed data has changed and no longer exactly matches
+      // the expected data in the mock below.
       axiosMock.onPost(getDiscussionsSettingsUrl(courseId), {
         context_key: courseId,
-        enabled: true,
         lti_configuration: {},
         plugin_configuration: {
           allow_anonymous: true,