test: updated tests

openedx · Nov 4, 2024 · 07e0ec8 · 07e0ec8
1 parent f851641
commit 07e0ec8
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 19 deletions.
diff --git a/src/data/reducers.js b/src/data/reducers.js
@@ -5,9 +5,8 @@ import { reducer as NewProfilePageReducer } from '../profile-v2';
 
 const isNewProfileEnabled = process.env.ENABLE_NEW_PROFILE_VIEW === 'true';
 
-const createRootReducer = () =>
-  combineReducers({
-    profilePage: isNewProfileEnabled ? NewProfilePageReducer : profilePage,
-  });
+const createRootReducer = () => combineReducers({
+  profilePage: isNewProfileEnabled ? NewProfilePageReducer : profilePage,
+});
 
 export default createRootReducer;
diff --git a/src/routes/AppRoutes.jsx b/src/routes/AppRoutes.jsx
@@ -14,9 +14,11 @@ const AppRoutes = () => (
     <Route
       path="/u/:username"
       element={
-        <AuthenticatedPageRoute>
-          {isNewProfileEnabled ? <NewProfilePage /> : <ProfilePage />}
-        </AuthenticatedPageRoute>
+        (
+          <AuthenticatedPageRoute>
+            {isNewProfileEnabled ? <NewProfilePage /> : <ProfilePage />}
+          </AuthenticatedPageRoute>
+        )
       }
     />
     <Route

diff --git a/src/routes/routes.test.jsx b/src/routes/routes.test.jsx
@@ -12,19 +12,15 @@ jest.mock('@edx/frontend-platform/auth', () => ({
   getLoginRedirectUrl: jest.fn(),
 }));
 
-const isNewProfileEnabled = process.env.ENABLE_NEW_PROFILE_VIEW === 'true';
+jest.mock('../profile', () => ({
+  ProfilePage: () => (<div>Profile page</div>),
+  NotFoundPage: () => (<div>Not found page</div>),
+}));
 
-if (isNewProfileEnabled) {
-  jest.mock('../profile-v2', () => ({
-    ProfilePage: () => (<div>New Profile page</div>),
-    NotFoundPage: () => (<div>New Not found page</div>),
-  }));
-} else {
-  jest.mock('../profile', () => ({
-    ProfilePage: () => (<div>Profile page</div>),
-    NotFoundPage: () => (<div>Not found page</div>),
-  }));
-}
+jest.mock('../profile-v2', () => ({
+  ProfilePage: () => (<div>Profile page</div>),
+  NotFoundPage: () => (<div>Not found page</div>),
+}));
 
 const RoutesWithProvider = (context, path) => (
   <AppContext.Provider value={context}>