[Navigation bar] Fix issues

UniversityOfHelsinkiCS · Dec 2, 2024 · c26728d · c26728d
1 parent 4c1e116
commit c26728d
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/cypress/e2e/Users.js b/cypress/e2e/Users.js
@@ -72,8 +72,7 @@ describe('Users tests', () => {
       })
 
       it("only the mocked user's programmes are visible", () => {
-        cy.get('[data-cy=nav-bar-button-studyProgramme]').click()
-        cy.get('[data-cy=nav-bar-button-class]').click()
+        cy.visit('/populations')
         cy.contains('label', 'Study programme')
           .siblings()
           .within(() => {

diff --git a/services/frontend/src/components/material/NavigationBar/NavigationButton.tsx b/services/frontend/src/components/material/NavigationBar/NavigationButton.tsx
@@ -14,7 +14,7 @@ export const NavigationButton = ({ item }: { item: NavigationItem }) => {
   const location = useLocation()
   const [anchorEl, setAnchorEl] = useState<null | HTMLElement>(null)
 
-  const { key, label, items } = item
+  const { label, items } = item
 
   const showItem = (subItemKey: string) => {
     if (['class', 'completedCoursesSearch', 'overview'].includes(subItemKey)) {
@@ -64,7 +64,6 @@ export const NavigationButton = ({ item }: { item: NavigationItem }) => {
       <>
         <Typography
           color="inherit"
-          data-cy={`nav-bar-button-${key}`}
           onClick={event => setAnchorEl(event.currentTarget)}
           sx={{ display: 'flex' }}
           variant="button"
@@ -77,10 +76,9 @@ export const NavigationButton = ({ item }: { item: NavigationItem }) => {
               showItem(subItem.key) && (
                 <MenuItem
                   component={Link}
-                  data-cy={`nav-bar-button-${subItem.key}`}
                   key={subItem.path}
                   onClick={() => setAnchorEl(null)}
-                  selected={location.pathname.includes(subItem.path)}
+                  selected={location.pathname?.includes(subItem.path)}
                   to={subItem.path}
                 >
                   {subItem.label}
@@ -93,7 +91,7 @@ export const NavigationButton = ({ item }: { item: NavigationItem }) => {
   }
 
   return (
-    <Typography color="inherit" data-cy={`nav-bar-button-${key}`} variant="button">
+    <Typography color="inherit" variant="button">
       {label}
     </Typography>
   )

diff --git a/services/frontend/src/components/material/NavigationBar/index.tsx b/services/frontend/src/components/material/NavigationBar/index.tsx
@@ -89,15 +89,16 @@ export const NavigationBar = () => {
             }}
           >
             <OodikoneLogo />
-            {!isLoading && (
+            {!isLoading && location && (
               <Tabs textColor="inherit" value={activeTab} variant="scrollable">
                 {Object.entries(visibleNavigationItems).map(([key, item]) => (
                   <Tab
-                    component={Link}
+                    component={item.path ? Link : 'div'}
+                    data-cy={`nav-bar-button-${key}`}
                     key={key}
                     label={<NavigationButton item={item} />}
                     sx={{ '&:hover': { color: 'inherit' } }}
-                    to={item.path}
+                    to={item.path ?? ''}
                   />
                 ))}
               </Tabs>