fix: fixed some tests

openedx · Oct 20, 2024 · 4ed52ac · 4ed52ac
1 parent d757a99
commit 4ed52ac
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/src/data/selectors.js b/src/data/selectors.js
@@ -1,2 +1,2 @@
 // eslint-disable-next-line import/prefer-default-export
-export const getWaffleFlags = (state) => state.courseDetail.waffleFlags;
+export const getWaffleFlags = (state) => state.courseDetail?.waffleFlags;
diff --git a/src/data/thunks.js b/src/data/thunks.js
@@ -39,11 +39,10 @@ export function fetchWaffleFlags(courseId) {
       dispatch(updateStatus({ courseId, status: RequestStatus.SUCCESSFUL }));
       dispatch(fetchWaffleFlagsSuccess({ waffleFlags }));
     } catch (error) {
-      if (error.response && error.response.status === 404) {
-        dispatch(updateStatus({ courseId, status: RequestStatus.NOT_FOUND }));
-      } else {
-        dispatch(updateStatus({ courseId, status: RequestStatus.FAILED }));
-      }
+      // If fetching the waffle flags is unsuccessful,
+      // the pages will still be accessible and display without any issues.
+      // eslint-disable-next-line no-console
+      console.error({ courseId, status: RequestStatus.NOT_FOUND });
     }
   };
 }