Skip to content

Commit 2b8a351

Browse files
authored
Merge pull request #149 from ameerul-deriv/FEQ-2326-add-test-coverage-for-currency
Ameerul / FEQ-2326 Add test coverage for currency.ts
2 parents 6b019e6 + 866fdb2 commit 2b8a351

File tree

4 files changed

+9
-45
lines changed

4 files changed

+9
-45
lines changed

src/components/ProfileContent/ProfileBalance/ProfileBalance.tsx

+6-6
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,10 @@ import { useMemo, useState } from 'react';
22
import { DeepPartial, TAdvertiserStats } from 'types';
33
import { AvailableP2PBalanceModal } from '@/components/Modals';
44
import { api } from '@/hooks';
5-
import { numberToCurrencyText } from '@/utils';
65
import { LabelPairedCircleInfoMdRegularIcon } from '@deriv/quill-icons';
76
import { Localize } from '@deriv-com/translations';
87
import { Text, useDevice } from '@deriv-com/ui';
8+
import { FormatUtils } from '@deriv-com/utils';
99
import { ProfileDailyLimit } from '../ProfileDailyLimit';
1010
import './ProfileBalance.scss';
1111

@@ -18,13 +18,13 @@ const ProfileBalance = ({ advertiserStats }: { advertiserStats: DeepPartial<TAdv
1818
const dailyLimits = useMemo(
1919
() => [
2020
{
21-
available: `${numberToCurrencyText(advertiserStats?.dailyAvailableBuyLimit || 0)} ${currency}`,
22-
dailyLimit: `${advertiserStats?.daily_buy_limit || numberToCurrencyText(0)} ${currency}`,
21+
available: `${FormatUtils.formatMoney(advertiserStats?.dailyAvailableBuyLimit || 0)} ${currency}`,
22+
dailyLimit: `${advertiserStats?.daily_buy_limit || FormatUtils.formatMoney(0)} ${currency}`,
2323
type: 'Buy',
2424
},
2525
{
26-
available: `${numberToCurrencyText(advertiserStats?.dailyAvailableSellLimit || 0)} ${currency}`,
27-
dailyLimit: `${advertiserStats?.daily_sell_limit || numberToCurrencyText(0)} ${currency}`,
26+
available: `${FormatUtils.formatMoney(advertiserStats?.dailyAvailableSellLimit || 0)} ${currency}`,
27+
dailyLimit: `${advertiserStats?.daily_sell_limit || FormatUtils.formatMoney(0)} ${currency}`,
2828
type: 'Sell',
2929
},
3030
],
@@ -58,7 +58,7 @@ const ProfileBalance = ({ advertiserStats }: { advertiserStats: DeepPartial<TAdv
5858
/>
5959
</div>
6060
<Text size={isMobile ? '2xl' : 'xl'} weight='bold'>
61-
{numberToCurrencyText(advertiserStats?.balance_available || 0)} USD
61+
{FormatUtils.formatMoney(advertiserStats?.balance_available || 0)} USD
6262
</Text>
6363
</div>
6464
<div className='flex flex-col gap-[1.6rem]'>

src/pages/my-profile/screens/MyProfileStats/MyProfileStats.tsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
import { useState } from 'react';
22
import { api } from '@/hooks';
33
import { useAdvertiserStats } from '@/hooks/custom-hooks';
4-
import { numberToCurrencyText } from '@/utils';
54
import { useTranslations } from '@deriv-com/translations';
65
import { Loader } from '@deriv-com/ui';
6+
import { FormatUtils } from '@deriv-com/utils';
77
import MyProfileStatsItem from './MyProfileStatsItem';
88
import './MyProfileStats.scss';
99

@@ -66,8 +66,8 @@ const MyProfileStats = ({ advertiserId }: TMyProfileStatsProps) => {
6666
testId='dt_profile_stats_trade_volume'
6767
value={
6868
shouldShowTradeVolumeLifetime
69-
? numberToCurrencyText(tradeVolumeLifetime)
70-
: numberToCurrencyText(tradeVolume)
69+
? FormatUtils.formatMoney(tradeVolumeLifetime)
70+
: FormatUtils.formatMoney(tradeVolume)
7171
}
7272
/>
7373
<MyProfileStatsItem

src/utils/currency.ts

-35
This file was deleted.

src/utils/index.ts

-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
export * from './ad-utils';
22
export * from './adverts';
3-
export * from './currency';
43
export * from './date';
54
export * from './file-dropzone';
65
export * from './file-uploader';

0 commit comments

Comments
 (0)