Merge pull request #170 from ahmed-deriv/ahmed/DAPI-819/chore--code-i…

…mprovement ahmed/DAPI-819/chore--code-improvement
deriv-com · Dec 9, 2024 · c4e61d7 · c4e61d7
2 parents fd00b8b + cae3ce3
commit c4e61d7
Show file tree

Hide file tree

Showing 91 changed files with 64 additions and 3,929 deletions.
diff --git a/src/components/AccountSwitcher/__tests__/AccountSwitcher.test.tsx b/src/components/AccountSwitcher/__tests__/AccountSwitcher.test.tsx
@@ -37,7 +37,7 @@ mockUseLogout.mockImplementation(() => {
   };
 });
 
-describe('HeroHeader', () => {
+describe.skip('HeroHeader', () => {
   beforeEach(() => {
     mockUseAuthContext.mockImplementation(() => {
       return {
@@ -64,13 +64,13 @@ describe('HeroHeader', () => {
     jest.clearAllMocks();
   });
 
-  it.skip('Should render current account ', () => {
+  it('Should render current account ', () => {
     const current_account_button = screen.getByRole('button', { name: /CR111111/i });
 
     expect(current_account_button).toBeInTheDocument();
   });
 
-  it.skip('Should call do logout on logout button click', async () => {
+  it('Should call do logout on logout button click', async () => {
     const current_account_button = await screen.findByRole('button', { name: /CR111111/i });
 
     await act(async () => {
@@ -86,7 +86,7 @@ describe('HeroHeader', () => {
     expect(mockLogout).toHaveBeenCalledTimes(1);
   });
 
-  it.skip('should be able to close the dropdown by clicking on the arrow', async () => {
+  it('should be able to close the dropdown by clicking on the arrow', async () => {
     const current_account_button = await screen.findByRole('button', { name: /CR111111/i });
 
     await act(async () => {
@@ -102,7 +102,7 @@ describe('HeroHeader', () => {
     expect(close_dropdown_button).not.toBeVisible();
   });
 
-  it.skip('Should render Accounts when no account is selected', () => {
+  it('Should render Accounts when no account is selected', () => {
     cleanup();
     mockUseAuthContext.mockImplementation(() => {
       return {
@@ -127,7 +127,7 @@ describe('HeroHeader', () => {
     expect(accounts_button).toBeInTheDocument();
   });
 
-  it.skip('Should render the dropdown menu on current account button click', async () => {
+  it('Should render the dropdown menu on current account button click', async () => {
     const current_account_button = screen.getByRole('button', { name: /USD/i });
 
     await act(async () => {
@@ -139,7 +139,7 @@ describe('HeroHeader', () => {
     expect(menu_items.length).toBe(1);
   });
 
-  it.skip('Should update current account on menu item click', async () => {
+  it('Should update current account on menu item click', async () => {
     mockUseAuthContext.mockImplementation(() => {
       return {
         loginAccounts: fake_accounts,

diff --git a/src/components/ApiTokenNavbarItem/__tests__/ApiTokenNavbarItem.test.tsx b/src/components/ApiTokenNavbarItem/__tests__/ApiTokenNavbarItem.test.tsx