From eee0ba9f2f466aa4a05602a2bf828a1ddf695312 Mon Sep 17 00:00:00 2001 From: Sandeep Rajput <90243468+sandeep-deriv@users.noreply.github.com> Date: Wed, 10 May 2023 16:25:57 +0800 Subject: [PATCH] Revert "thisyahlen/chore: add test coverage in asset-summary (#8329)" This reverts commit b45765072fee704fa6240e906aa0fa2a8fe7b74c. --- .../__tests__/asset-summary.spec.tsx | 233 ------------------ packages/stores/src/mockStore.ts | 17 -- packages/stores/types.ts | 17 -- 3 files changed, 267 deletions(-) delete mode 100644 packages/appstore/src/components/main-title-bar/__tests__/asset-summary.spec.tsx diff --git a/packages/appstore/src/components/main-title-bar/__tests__/asset-summary.spec.tsx b/packages/appstore/src/components/main-title-bar/__tests__/asset-summary.spec.tsx deleted file mode 100644 index ede98873e952..000000000000 --- a/packages/appstore/src/components/main-title-bar/__tests__/asset-summary.spec.tsx +++ /dev/null @@ -1,233 +0,0 @@ -import React from 'react'; -import AssetSummary from '../asset-summary'; -import { render, screen } from '@testing-library/react'; -import { StoreProvider, mockStore } from '@deriv/stores'; -import { isMobile } from '@deriv/shared'; - -jest.mock('../../pre-loader/total-assets-loader', () => ({ - __esModule: true, - default: () =>
TotalAssetsLoader
, -})); - -jest.mock('@deriv/shared', () => ({ - ...jest.requireActual('@deriv/shared'), - isMobile: jest.fn(), -})); - -jest.mock('@deriv/hooks', () => ({ - ...jest.requireActual('@deriv/hooks'), - usePlatformAccounts: jest.fn(() => ({ - real: [ - { - balance: 100, - currency: 'USD', - }, - ], - demo: { - balance: 10000, - currency: 'USD', - }, - })), - useCfdAccounts: jest.fn(() => ({ - real: [ - { - balance: 15000, - currency: 'USD', - }, - ], - demo: { - balance: 123213, - currency: 'USD', - }, - })), -})); - -describe('AssetSummary', () => { - it('should render correctly', () => { - const mock = mockStore({}); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - }); - - it('should render the text and balance correctly', () => { - const mock = mockStore({}); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.getByText('Total asset')).toBeInTheDocument(); - expect(screen.getByText('15,100.00')).toBeInTheDocument(); - }); - - it('should not show Total Assets title if isMobile is true ', () => { - isMobile.mockReturnValue(true); - const mock = mockStore({}); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.queryByText('Total assets')).not.toBeInTheDocument(); - }); - - it('should show loader if is_switching is true ', () => { - const mock = mockStore({ - client: { - is_switching: true, - }, - }); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.getByText('TotalAssetsLoader')).toBeInTheDocument(); - }); - - it('should not render if user has no real account ', () => { - const mock = mockStore({ - client: { - has_maltainvest_account: false, - }, - traders_hub: { - selected_account_type: 'real', - is_eu_user: true, - }, - }); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - }); - - it('should show the correct amount in real tab', () => { - const mock = mockStore({ - traders_hub: { - selected_account_type: 'real', - }, - }); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.getByText('15,100.00')).toBeInTheDocument(); - }); - - it('should not show component if user has no real MF account and in eu regulation', () => { - const mock = mockStore({ - traders_hub: { - selected_account_type: 'real', - is_eu_user: true, - no_MF_account: true, - }, - }); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.queryByText('Total assets')).not.toBeInTheDocument(); - }); - - it('should not show component if user has no real CR account and in non-eu regulation', () => { - const mock = mockStore({ - traders_hub: { - selected_account_type: 'real', - no_CR_account: true, - }, - }); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.queryByText('Total assets')).not.toBeInTheDocument(); - }); - - it('should show the correct balance for demo account ', () => { - const mock = mockStore({ - traders_hub: { - selected_account_type: 'demo', - platform_demo_balance: { - balance: 10000, - }, - }, - }); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.getByText('10,000.00')).toBeInTheDocument(); - }); - - it('should show the correct real total amount with total cfd currency', () => { - const mock = mockStore({}); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.getByText('15,100.00')).toBeInTheDocument(); - }); - - it('should show the correct total demo amount with total demo cfd currency', () => { - const mock = mockStore({}); - - const wrapper = ({ children }: { children: JSX.Element }) => ( - {children} - ); - - const { container } = render(, { - wrapper, - }); - expect(container).toBeInTheDocument(); - expect(screen.getByText('15,100.00')).toBeInTheDocument(); - }); -}); diff --git a/packages/stores/src/mockStore.ts b/packages/stores/src/mockStore.ts index e25010324f36..41de5c479e09 100644 --- a/packages/stores/src/mockStore.ts +++ b/packages/stores/src/mockStore.ts @@ -214,7 +214,6 @@ const mock = (): TStores & { is_mock: boolean } => { current_language: 'EN', is_network_online: false, is_language_changing: false, - getExchangeRate: jest.fn(), }, ui: { app_contents_scroll_ref: { @@ -261,22 +260,6 @@ const mock = (): TStores & { is_mock: boolean } => { is_real: false, selectRegion: jest.fn(), is_low_risk_cr_eu_real: false, - platform_real_balance: { - currency: '', - balance: 0, - }, - cfd_demo_balance: { - currency: '', - balance: 0, - }, - platform_demo_balance: { - currency: '', - balance: 0, - }, - cfd_real_balance: { - currency: '', - balance: 0, - }, financial_restricted_countries: false, selected_account_type: 'real', no_CR_account: false, diff --git a/packages/stores/types.ts b/packages/stores/types.ts index e4a76c2a7940..2911df21fcc1 100644 --- a/packages/stores/types.ts +++ b/packages/stores/types.ts @@ -219,7 +219,6 @@ type TCommonStore = { changeSelectedLanguage: (key: string) => void; current_language: string; is_language_changing: boolean; - getExchangeRate: (from_currency: string, to_currency: string) => Promise; }; type TUiStore = { @@ -290,22 +289,6 @@ type TTradersHubStore = { setTogglePlatformType: (platform_type: string) => void; is_real: boolean; selectRegion: (region: string) => void; - platform_real_balance: { - currency: string; - balance: number; - }; - cfd_demo_balance: { - currency: string; - balance: number; - }; - platform_demo_balance: { - currency: string; - balance: number; - }; - cfd_real_balance: { - currency: string; - balance: number; - }; financial_restricted_countries: boolean; selected_account_type: string; no_CR_account: boolean;