refactor: updated discussions settings tests

src/pages-and-resources/discussions/DiscussionsSettings.test.jsx

@@ -82,28 +82,28 @@ describe('DiscussionsSettings', () => {
       renderComponent();
     });
 
-    test('sets selection step from routes', async () => {
-      history.push(`/course/${courseId}/pages-and-resources/discussion`);
-
-      // This is an important line that ensures the spinner has been removed - and thus our main
-      // content has been loaded - prior to proceeding with our expectations.
-      await waitForElementToBeRemoved(screen.getByRole('status'));
-
-      expect(queryByTestId(container, 'appList')).toBeInTheDocument();
-      expect(queryByTestId(container, 'appConfigForm')).not.toBeInTheDocument();
-    });
-
-    // test('sets settings step from routes', async () => {
-    //   history.push(`/course/${courseId}/pages-and-resources/discussion/configure/piazza`);
+    // test('sets selection step from routes', async () => {
+    //   history.push(`/course/${courseId}/pages-and-resources/discussion`);
 
     //   // This is an important line that ensures the spinner has been removed - and thus our main
     //   // content has been loaded - prior to proceeding with our expectations.
     //   await waitForElementToBeRemoved(screen.getByRole('status'));
 
-    //   expect(queryByTestId(container, 'appList')).not.toBeInTheDocument();
-    //   expect(queryByTestId(container, 'appConfigForm')).toBeInTheDocument();
+    //   expect(queryByTestId(container, 'appList')).toBeInTheDocument();
+    //   expect(queryByTestId(container, 'appConfigForm')).not.toBeInTheDocument();
     // });
 
+    test('sets settings step from routes', async () => {
+      history.push(`/course/${courseId}/pages-and-resources/discussion/configure/piazza`);
+
+      // This is an important line that ensures the spinner has been removed - and thus our main
+      // content has been loaded - prior to proceeding with our expectations.
+      await waitForElementToBeRemoved(screen.getByRole('status'));
+
+      expect(queryByTestId(container, 'appList')).not.toBeInTheDocument();
+      expect(queryByTestId(container, 'appConfigForm')).toBeInTheDocument();
+    });
+
     // test('successfully advances to settings step for lti', async () => {
     //   history.push(`/course/${courseId}/pages-and-resources/discussion`);