Skip to content

Commit 15baa8e

Browse files
committed
chore: remove unused minimum coin quantity code
it was probably replaced with something else
1 parent 08a708b commit 15baa8e

File tree

7 files changed

+7
-90
lines changed

7 files changed

+7
-90
lines changed

apps/browser-extension-wallet/src/features/send/__tests__/selectors.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,9 @@ describe('Testing sendTransactionSelector', () => {
2424
expect(result.current).toHaveProperty('transactionValue');
2525
expect(result.current).toHaveProperty('transaction');
2626
expect(result.current).toHaveProperty('transactionFeeLovelace');
27-
expect(result.current).toHaveProperty('minimumCoinQuantity');
2827
expect(result.current).toHaveProperty('setDestinationAddress');
2928
expect(result.current).toHaveProperty('setTransactionValue');
3029
expect(result.current).toHaveProperty('setTransaction');
3130
expect(result.current).toHaveProperty('setTransactionFeeLovelace');
32-
expect(result.current).toHaveProperty('setMinimumCoinQuantity');
3331
});
3432
});
Lines changed: 2 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,8 @@
1-
import { availableCoinsTransformer, displayedCoinsTransformer } from '../transformers';
2-
import { mockAvailableBalance, mockDisplayedCoins } from '../../../../utils/mocks/test-helpers';
3-
import { TxMinimumCoinQuantity } from '../../../../types';
1+
import { availableCoinsTransformer } from '../transformers';
2+
import { mockAvailableBalance } from '../../../../utils/mocks/test-helpers';
43
import { firstValueFrom } from 'rxjs';
54

65
const assetId = '659f2917fb63f12b33667463ee575eeac1845bbc736b9c0bbc40ba8254534c41';
7-
const minimumCoinQuantity: TxMinimumCoinQuantity = {
8-
coinMissing: '2000000',
9-
minimumCoin: '2500000'
10-
};
116

127
describe('Testing availableCoinsTransformer function', () => {
138
it('should return parsed available coins', async () => {
@@ -27,32 +22,3 @@ describe('Testing availableCoinsTransformer function', () => {
2722
]);
2823
});
2924
});
30-
31-
describe('Testing displayedCoinsTransformer function', () => {
32-
it('should return only coins field if no assets are displayed', () => {
33-
const result = displayedCoinsTransformer(mockDisplayedCoins);
34-
expect(result.coins.toString()).toBe('0');
35-
});
36-
37-
it('should return coins and assets fields when both are displayed', () => {
38-
const result = displayedCoinsTransformer([...mockDisplayedCoins, { amount: '5', coinId: assetId }]);
39-
expect(result.coins.toString()).toBe('0');
40-
expect(result.assets).toEqual(new Map([[assetId, BigInt('5')]]));
41-
});
42-
43-
it(
44-
'should return default minimum coin quantity as coins ' +
45-
'if displayed coins list only has assets and no minimum is provided',
46-
() => {
47-
const result = displayedCoinsTransformer([{ amount: '5', coinId: assetId }]);
48-
expect(result.coins.toString()).toBe('1000000');
49-
expect(result.assets).toEqual(new Map([[assetId, BigInt('5')]]));
50-
}
51-
);
52-
53-
it('should return provided minimum coin quantity as coins if displayed coins list only has assets', () => {
54-
const result = displayedCoinsTransformer([{ amount: '5', coinId: assetId }], minimumCoinQuantity);
55-
expect(result.coins.toString()).toBe('2500000');
56-
expect(result.assets).toEqual(new Map([[assetId, BigInt('5')]]));
57-
});
58-
});
Lines changed: 2 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,10 @@
11
/* eslint-disable no-magic-numbers */
22
import isEmpty from 'lodash/isEmpty';
3-
import { IAssetInfo, DisplayedCoinDetail } from '../types';
4-
import { Tokens, CardanoTxOutValue, TokensDetails, TxMinimumCoinQuantity } from '../../../types';
3+
import { IAssetInfo } from '../types';
4+
import { Tokens, TokensDetails } from '../../../types';
55
import { Wallet } from '@lace/cardano';
66
import { addEllipsis } from '@lace/common';
77
import { cardanoCoin } from '../../../utils/constants';
8-
import { MIN_COIN_TO_SEND } from '../config';
9-
import BigNumber from 'bignumber.js';
108

119
export const availableCoinsTransformer = (
1210
coins: string,
@@ -31,28 +29,3 @@ export const availableCoinsTransformer = (
3129

3230
return [adaCoin, ...assetList];
3331
};
34-
35-
export const displayedCoinsTransformer = (
36-
displayedCoins: Array<DisplayedCoinDetail>,
37-
minimumCoinQuantity?: TxMinimumCoinQuantity
38-
): CardanoTxOutValue => {
39-
const ada = displayedCoins.find((item) => item.coinId === cardanoCoin.id);
40-
const nonAdaAssets = displayedCoins.filter((item) => item.coinId !== cardanoCoin.id);
41-
42-
const assets =
43-
nonAdaAssets.length > 0
44-
? new Map<Wallet.Cardano.AssetId, bigint>(
45-
nonAdaAssets.map((asset) => [Wallet.Cardano.AssetId(asset.coinId), BigInt(asset.amount)])
46-
)
47-
: undefined;
48-
49-
return {
50-
// The transaction must always have ADA on it,
51-
// if the displayedCoins list does not have ADA we send the minimum coin qty
52-
coins:
53-
ada && new BigNumber(ada.amount).gte(0)
54-
? BigInt(Wallet.util.adaToLovelacesString(ada.amount))
55-
: BigInt(minimumCoinQuantity?.minimumCoin ?? Wallet.util.adaToLovelacesString(MIN_COIN_TO_SEND.toString())),
56-
assets
57-
};
58-
};

apps/browser-extension-wallet/src/features/send/selectors.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,21 +11,17 @@ export const sendTransactionSelector: StateSelector<SendStore, SendTransactionSl
1111
transactionValue,
1212
transaction,
1313
transactionFeeLovelace,
14-
minimumCoinQuantity,
1514
setDestinationAddress,
1615
setTransactionValue,
1716
setTransaction,
18-
setTransactionFeeLovelace,
19-
setMinimumCoinQuantity
17+
setTransactionFeeLovelace
2018
}) => ({
2119
destinationAddress,
2220
transactionValue,
2321
transaction,
2422
transactionFeeLovelace,
25-
minimumCoinQuantity,
2623
setDestinationAddress,
2724
setTransactionValue,
2825
setTransaction,
29-
setTransactionFeeLovelace,
30-
setMinimumCoinQuantity
26+
setTransactionFeeLovelace
3127
});

apps/browser-extension-wallet/src/features/send/stores/slices/__tests__/send-transaction-slice.test.ts

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,10 @@ describe('Testing send transaction store slice', () => {
2222
expect(result.current.transaction).toBeUndefined();
2323
expect(result.current.transactionValue).toBeUndefined();
2424
expect(result.current.transactionFeeLovelace).toEqual('0');
25-
expect(result.current.minimumCoinQuantity).toEqual({ coinMissing: '0', minimumCoin: '0' });
2625
expect(typeof result.current.setDestinationAddress).toBe('function');
2726
expect(typeof result.current.setTransaction).toBe('function');
2827
expect(typeof result.current.setTransactionValue).toBe('function');
2928
expect(typeof result.current.setTransactionFeeLovelace).toBe('function');
30-
expect(typeof result.current.setMinimumCoinQuantity).toBe('function');
3129
});
3230

3331
test('should be able to set destination address', () => {
@@ -62,13 +60,4 @@ describe('Testing send transaction store slice', () => {
6260

6361
expect(result.current.transactionFeeLovelace).toEqual('2000000');
6462
});
65-
66-
test('should be able to set minimum coin quantity', () => {
67-
const { result } = renderHook(() => useSendTxHook());
68-
act(() => {
69-
result.current.setMinimumCoinQuantity({ coinMissing: '0', minimumCoin: '2000000' });
70-
});
71-
72-
expect(result.current.minimumCoinQuantity).toEqual({ coinMissing: '0', minimumCoin: '2000000' });
73-
});
7463
});

apps/browser-extension-wallet/src/features/send/stores/slices/send-transaction-slice.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,8 @@ export const sendTransactionSlice: StateCreator<SendTransactionSlice> = (set) =>
66
destinationAddress: undefined,
77
transactionValue: undefined,
88
transaction: undefined,
9-
minimumCoinQuantity: { coinMissing: '0', minimumCoin: '0' },
109
setDestinationAddress: (address) => set(() => ({ destinationAddress: address })),
1110
setTransactionValue: (value) => set(() => ({ transactionValue: value })),
1211
setTransaction: (tx) => set(() => ({ transaction: tx })),
13-
setTransactionFeeLovelace: (fee) => set(() => ({ transactionFeeLovelace: fee })),
14-
setMinimumCoinQuantity: (coinQty) => set(() => ({ minimumCoinQuantity: coinQty }))
12+
setTransactionFeeLovelace: (fee) => set(() => ({ transactionFeeLovelace: fee }))
1513
});

apps/browser-extension-wallet/src/features/send/stores/types.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import { TxMinimumCoinQuantity } from '../../../types';
21
import { CardanoTxBuild, CardanoTxOutAddress, CardanoTxOutValue } from '../../../types/cardano';
32

43
export interface SendCancelModalSlice {
@@ -11,12 +10,10 @@ export interface SendTransactionSlice {
1110
transactionValue: CardanoTxOutValue;
1211
transaction: CardanoTxBuild;
1312
transactionFeeLovelace: string;
14-
minimumCoinQuantity: TxMinimumCoinQuantity;
1513
setDestinationAddress: (address: CardanoTxOutAddress) => void;
1614
setTransactionValue: (value: CardanoTxOutValue) => void;
1715
setTransaction: (tx: CardanoTxBuild) => void;
1816
setTransactionFeeLovelace: (fee: string) => void;
19-
setMinimumCoinQuantity: (coinQty: TxMinimumCoinQuantity) => void;
2017
}
2118

2219
export type SendStore = SendTransactionSlice & SendCancelModalSlice;

0 commit comments

Comments
 (0)