Course Calatog: Stay on selected department (#1921)

mitodl · Sep 26, 2023 · c493295 · c493295
1 parent cc10c8a
commit c493295
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/frontend/public/src/containers/pages/CatalogPage.js b/frontend/public/src/containers/pages/CatalogPage.js
@@ -239,7 +239,6 @@ export class CatalogPage extends React.Component<Props> {
    */
   changeSelectedTab = (selectTabName: string) => {
     this.setState({ tabSelected: selectTabName })
-    this.changeSelectedDepartment(ALL_DEPARTMENTS, selectTabName)
 
     if (selectTabName === PROGRAMS_TAB) {
       const { programs, programsIsLoading } = this.props

diff --git a/frontend/public/src/containers/pages/CatalogPage_test.js b/frontend/public/src/containers/pages/CatalogPage_test.js
@@ -722,14 +722,14 @@ describe("CatalogPage", function() {
 
     // Change to the programs tab.
     inner.instance().changeSelectedTab("programs")
-    // Confirm that the selected departement resets to "All Departments".
-    expect(inner.state().selectedDepartment).equals("All Departments")
+    // Confirm that the selected department is the same as before.
+    expect(inner.state().selectedDepartment).equals("History")
 
     // Change back to the courses tab.
     inner.instance().changeSelectedTab("courses")
-    // All of the courses should be visible.
+    // Confirm the courses filtered are those which have a department name matching the selected department.
     expect(JSON.stringify(inner.state().filteredCourses)).equals(
-      JSON.stringify(courses)
+      JSON.stringify([course2, course3])
     )
   })