fix: remove fixed precision on Ada and fiat values

- Ada has a 6 decimal point precision, but currently we're rounding to 2
- For consistency this change also now only shows decimals when needed,
Since displaying 6 decimal points on all Ada values is not ideal.

Author: rhyslbw

apps/browser-extension-wallet/src/api/__tests__/input-output-transformer.test.ts

@@ -32,7 +32,7 @@ describe('Testing inputOutputTransformer function', () => {
     expect(result.addr).toEqual(
       'addr_test1qz2fxv2umyhttkxyxp8x0dlpdt3k6cwng5pxj3jhsydzer3jcu5d8ps7zex2k2xt3uqxgjqnnj83ws8lhrn648jjxtwq2ytjqp'
     );
-    expect(result.amount).toBe('1.00');
+    expect(result.amount).toBe('1');
     expect(result.assetList.length).toEqual(0);
   });
 
@@ -46,7 +46,7 @@ describe('Testing inputOutputTransformer function', () => {
     expect(result.addr).toEqual(
       'addr_test1qz2fxv2umyhttkxyxp8x0dlpdt3k6cwng5pxj3jhsydzer3jcu5d8ps7zex2k2xt3uqxgjqnnj83ws8lhrn648jjxtwq2ytjqp'
     );
-    expect(result.amount).toBe('3.00');
+    expect(result.amount).toBe('3');
     expect(result.assetList.length).toEqual(0);
   });
 
@@ -66,7 +66,7 @@ describe('Testing inputOutputTransformer function', () => {
     expect(result.addr).toEqual(
       'addr_test1qz2fxv2umyhttkxyxp8x0dlpdt3k6cwng5pxj3jhsydzer3jcu5d8ps7zex2k2xt3uqxgjqnnj83ws8lhrn648jjxtwq2ytjqp'
     );
-    expect(result.amount).toBe('3.00');
+    expect(result.amount).toBe('3');
     expect(result.assetList.length).toEqual(1);
   });
 });

apps/browser-extension-wallet/src/api/__tests__/wallet-balance-transformer.test.ts

@@ -5,13 +5,13 @@ import { walletBalanceTransformer } from '../transformers';
 describe('Testing walletBalanceTransformer function', () => {
   test('given a wallet balance in lovelace should return the balance in ada and undefined for fiat', () => {
     const result = walletBalanceTransformer('10000000');
-    expect(result.coinBalance).toBe('10.00');
+    expect(result.coinBalance).toBe('10');
     expect(result.fiatBalance).toBeUndefined();
   });
 
   test('given a wallet balance in lovelace and a fiat price should return the balance in ada and in fiat', () => {
     const result = walletBalanceTransformer('10000000', 2);
-    expect(result.coinBalance).toBe('10.00');
+    expect(result.coinBalance).toBe('10');
     expect(result.fiatBalance).toBe('20.00');
   });
 });

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

@@ -15,7 +15,7 @@ describe('Testing availableCoinsTransformer function', () => {
     const coins = availableCoinsTransformer(totalCoins.toString(), assets);
     expect(coins).toEqual([
       {
-        balance: '10.00',
+        balance: '10',
         id: '1',
         symbol: 'ADA'
       },

apps/browser-extension-wallet/src/utils/mocks/test-helpers.tsx

@@ -569,7 +569,7 @@ export const cardanoStakePoolSelectedDetails = {
   status: 'active',
   ticker: 'STTST',
   blocks: '20',
-  pledge: '2000.00'
+  pledge: '2000'
 };
 
 export const transformedStakePool = {
@@ -585,7 +585,7 @@ export const transformedStakePool = {
     'stake_test1uqrw9tjymlm8wrwq7jk68n6v7fs9qz8z0tkdkve26dylmfc2ux2hj',
     'stake_test1uq7g7kqeucnqfweqzgxk3dw34e8zg4swnc7nagysug2mm4cm77jrx'
   ],
-  pledge: '2000.00ADA',
+  pledge: '2000ADA',
   retired: false,
   saturation: '5.12',
   size: '- %',

apps/browser-extension-wallet/src/views/browser-view/features/activity/helpers/__tests__/pending-tx-transformer.test.ts

@@ -117,10 +117,10 @@ describe('Testing tx transformers utils', () => {
           deposit: undefined,
           depositReclaim: undefined,
           direction: 'Outgoing',
-          fee: '1.00',
-          fiatAmount: '1.00 USD',
+          fee: '1',
+          fiatAmount: '1 USD',
           id: '6804edf9712d2b619edb6ac86861fe93a730693183a262b165fcc1ba1bc99cad',
-          amount: '1.00 ADA',
+          amount: '1 ADA',
           assets: [
             {
               id: '6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7',

apps/browser-extension-wallet/src/views/browser-view/features/send-transaction/components/Form/CoinInput/__tests__/util.test.ts

@@ -31,7 +31,7 @@ describe('CoinInput util', () => {
   describe('getADACoinProperties', () => {
     test('returns 0 for availableADA and false for hasMaxBtn when balance is 0', () => {
       expect(getADACoinProperties('0', '1000000', '0', '0')).toEqual({
-        availableADA: '0.00',
+        availableADA: '0',
         max: '1',
         hasMaxBtn: false,
         hasReachedMaxAmount: false,
@@ -40,7 +40,7 @@ describe('CoinInput util', () => {
     });
     test('returns 0 for max and true for hasReachedMaxAmount when spendable coins is 0', () => {
       expect(getADACoinProperties('1000000', '0', '0', '0')).toEqual({
-        availableADA: '1.00',
+        availableADA: '1',
         max: '0',
         hasMaxBtn: true,
         hasReachedMaxAmount: true,
@@ -49,7 +49,7 @@ describe('CoinInput util', () => {
     });
     test('returns formatted balance as availableADA, and the spendable coin in ADA as max when there is no spending', () => {
       expect(getADACoinProperties('20000000', '10000000', '0', '0')).toEqual({
-        availableADA: '20.00',
+        availableADA: '20',
         max: '10',
         hasMaxBtn: true,
         hasReachedMaxAmount: false,
@@ -58,7 +58,7 @@ describe('CoinInput util', () => {
     });
     test('returns the calculated max amount when there is less spent coin than spendable coin', () => {
       expect(getADACoinProperties('20000000', '10000000', '5', '2')).toEqual({
-        availableADA: '20.00',
+        availableADA: '20',
         max: '7',
         hasMaxBtn: true,
         hasReachedMaxAmount: false,
@@ -67,7 +67,7 @@ describe('CoinInput util', () => {
     });
     test('returns max amount as 0 and hasReachedMaxAmount as true when there is more spent coin than spendable coin', () => {
       expect(getADACoinProperties('20000000', '10000000', '10', '0')).toEqual({
-        availableADA: '20.00',
+        availableADA: '20',
         max: '0',
         hasMaxBtn: true,
         hasReachedMaxAmount: true,

apps/browser-extension-wallet/src/views/browser-view/features/send-transaction/components/Form/CoinInput/util.ts

@@ -47,7 +47,7 @@ export const getADACoinProperties = (
 ): ADARow => {
   // Convert to ADA
   const availableADA = Wallet.util.lovelacesToAdaString(balance);
-  const spendableCoinInAda = Wallet.util.lovelacesToAdaString(spendableCoin, undefined, BigNumber.ROUND_DOWN);
+  const spendableCoinInAda = Wallet.util.lovelacesToAdaString(spendableCoin);
   // Calculate max amount in ADA
   const max = getMaxSpendableAmount(spendableCoinInAda, spentCoins, currentSpendingAmount);
   return {

packages/cardano/src/wallet/util/__tests__/unit-converters.test.ts

@@ -10,12 +10,7 @@ import {
 describe('Testing lovelacesToAdaString function', () => {
   test('should convert lovelaces to ada', async () => {
     const result = lovelacesToAdaString('1000000');
-    expect(result).toBe('1.00');
-  });
-
-  test('should return ada value with 4 decimals', async () => {
-    const result = lovelacesToAdaString('1000000', 4);
-    expect(result).toBe('1.0000');
+    expect(result).toBe('1');
   });
 });
 

packages/cardano/src/wallet/util/unit-converters.ts

@@ -4,11 +4,8 @@ import { CoinId } from '../types';
 const LOVELACE_VALUE = 1_000_000;
 const DEFAULT_DECIMALS = 2;
 
-export const lovelacesToAdaString = (
-  lovelaces: string,
-  decimalValues: number = DEFAULT_DECIMALS,
-  roudingMode: BigNumber.RoundingMode = BigNumber.ROUND_HALF_UP
-): string => new BigNumber(lovelaces).dividedBy(LOVELACE_VALUE).toFixed(decimalValues, roudingMode);
+export const lovelacesToAdaString = (lovelaces: string): string =>
+  new BigNumber(lovelaces).dividedBy(LOVELACE_VALUE).toString();
 
 export const adaToLovelacesString = (ada: string): string => new BigNumber(ada).multipliedBy(LOVELACE_VALUE).toString();