diff --git a/src/features/dashboard/components/Tabs/__tests__/tabs.test.tsx b/src/features/dashboard/components/Tabs/__tests__/tabs.test.tsx index 0ce93ee66..7e3fc0a2b 100644 --- a/src/features/dashboard/components/Tabs/__tests__/tabs.test.tsx +++ b/src/features/dashboard/components/Tabs/__tests__/tabs.test.tsx @@ -22,7 +22,7 @@ mockUseAppManager.mockImplementation(() => ({ updateCurrentTab: mockUpdateCurrentTab, })); -describe.skip('Dashboard Tabs', () => { +describe('Dashboard Tabs', () => { beforeEach(() => { render(); }); @@ -36,11 +36,12 @@ describe.skip('Dashboard Tabs', () => { it('Should render all tabs properly', () => { const tabs = screen.getAllByRole('tab'); - expect(tabs).toHaveLength(3); + expect(tabs).toHaveLength(4); const registerApplicationTab = screen.getByRole('tab', { name: /register application/i }); const manageApplicationsTab = screen.getByRole('tab', { name: /manage tokens/i }); const manageTokensTab = screen.getByRole('tab', { name: /manage applications/i }); + const registerTokenTab = screen.getByRole('tab', { name: /register tokens/i }); expect(registerApplicationTab).toBeInTheDocument(); expect(registerApplicationTab).toBeVisible(); @@ -50,6 +51,9 @@ describe.skip('Dashboard Tabs', () => { expect(manageTokensTab).toBeInTheDocument(); expect(manageTokensTab).toBeVisible(); + + expect(registerTokenTab).toBeInTheDocument(); + expect(registerTokenTab).toBeVisible(); }); it('Should change the current tab on tabs click', async () => { diff --git a/src/features/dashboard/manage-apps/manage-apps.tsx b/src/features/dashboard/manage-apps/manage-apps.tsx index d7f042052..5376f3f24 100644 --- a/src/features/dashboard/manage-apps/manage-apps.tsx +++ b/src/features/dashboard/manage-apps/manage-apps.tsx @@ -1,5 +1,5 @@ +import React from 'react'; import useAppManager from '@site/src/hooks/useAppManager'; -import React, { useEffect } from 'react'; import AppManagePage from './app-manage-page'; import TokenManagePage from '../manage-tokens/token-manage-page'; import CustomTabs from '@site/src/components/custom-tabs'; @@ -8,11 +8,7 @@ import { TDashboardTab } from '@site/src/contexts/app-manager/app-manager.contex import { translate } from '@docusaurus/Translate'; const AppManagement = () => { - const { getApps, apps, currentTab } = useAppManager(); - - useEffect(() => { - getApps(); - }, [getApps]); + const { apps, currentTab } = useAppManager(); const tabs = [ { @@ -27,7 +23,9 @@ const AppManagement = () => { return (
- +
); }; diff --git a/src/features/dashboard/manage-dashboard/index.tsx b/src/features/dashboard/manage-dashboard/index.tsx index f78d878e5..06d0b5837 100644 --- a/src/features/dashboard/manage-dashboard/index.tsx +++ b/src/features/dashboard/manage-dashboard/index.tsx @@ -60,10 +60,12 @@ const ManageDashboard = () => { }, [getApps]); useEffect(() => { - if (!apps?.length) { - updateCurrentTab(TDashboardTab.REGISTER_APP); - } else { - updateCurrentTab(TDashboardTab.MANAGE_APPS); + if (currentTab != TDashboardTab.REGISTER_TOKENS && currentTab != TDashboardTab.UPDATE_APP) { + if (!apps?.length) { + updateCurrentTab(TDashboardTab.REGISTER_APP); + } else { + updateCurrentTab(TDashboardTab.MANAGE_APPS); + } } }, [apps, updateCurrentTab]); @@ -93,10 +95,10 @@ const ManageDashboard = () => { return ; case TDashboardTab.UPDATE_APP: return ; - case TDashboardTab.MANAGE_TOKENS: - return ; case TDashboardTab.REGISTER_TOKENS: return ; + case TDashboardTab.MANAGE_TOKENS: + return ; default: return ; } @@ -113,7 +115,7 @@ const ManageDashboard = () => { { content: translate({ message: 'Dashboard' }), href: locale_Links.dashboard, target: '_self' }, ]; - const tabSecndryLinks = { + const tabSecondaryLinks = { [TDashboardTab.REGISTER_APP]: { content: translate({ message: 'Register application' }), href: locale_Links.dashboard, @@ -131,7 +133,7 @@ const ManageDashboard = () => { }, }; - const breadcrumbsLinks = [...commonLinks, tabSecndryLinks[currentTab]].filter(Boolean); + const breadcrumbsLinks = [...commonLinks, tabSecondaryLinks[currentTab]].filter(Boolean); return ( <>