Skip to content

Commit 6fea960

Browse files
committed
RI-7278 - Remove the old navigation for DBs - removed unused variables and updated tests
1 parent f1958ac commit 6fea960

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

redisinsight/ui/src/components/navigation-menu/NavigationMenu.spec.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ describe('NavigationMenu', () => {
7676
}))
7777
render(<NavigationMenu />)
7878

79-
expect(screen.queryByTestId('browser-page-btn"')).not.toBeInTheDocument()
79+
expect(screen.queryByTestId('browser-page-btn')).not.toBeInTheDocument()
8080
})
8181

8282
it('should render help menu', () => {
@@ -137,7 +137,7 @@ describe('NavigationMenu', () => {
137137
expect(render(<NavigationMenu />)).toBeTruthy()
138138
})
139139

140-
it('should render private routes with instanceId', () => {
140+
it('should not render private routes with instanceId', () => {
141141
;(appInfoSelector as jest.Mock).mockImplementation(() => ({
142142
...mockAppInfoSelector,
143143
server: {
@@ -146,8 +146,8 @@ describe('NavigationMenu', () => {
146146
}))
147147
render(<NavigationMenu />)
148148

149-
expect(screen.getByTestId('browser-page-btn')).toBeTruthy()
150-
expect(screen.getByTestId('workbench-page-btn')).toBeTruthy()
149+
expect(screen.queryByTestId('browser-page-btn')).not.toBeInTheDocument()
150+
expect(screen.queryByTestId('workbench-page-btn')).not.toBeInTheDocument()
151151
})
152152

153153
it('should render public routes', () => {

redisinsight/ui/src/components/navigation-menu/NavigationMenu.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,13 +29,11 @@ import styles from './styles.module.scss'
2929

3030
const NavigationMenu = () => {
3131
const {
32-
privateRoutes,
3332
privateRdiRoutes,
3433
isRdiWorkspace,
3534
publicRoutes,
3635
getAdditionPropsForHighlighting,
3736
highlightedPages,
38-
connectedInstanceId,
3937
connectedRdiInstanceId,
4038
} = useNavigation()
4139

0 commit comments

Comments
 (0)