fix: ensure is possible to forward 0 amount to contract (#3776)

Author: Torres-ssf

.changeset/cold-ears-double.md

@@ -0,0 +1,5 @@
+---
+"@fuel-ts/account": patch
+---
+
+fix: ensure is possible to forward 0 amount to contract

packages/account/src/providers/coin-quantity.ts

@@ -23,10 +23,9 @@ export const coinQuantityfy = (coinQuantityLike: CoinQuantityLike): CoinQuantity
     max = coinQuantityLike.max ?? undefined;
   }
 
-  const bnAmount = bn(amount);
   return {
     assetId: hexlify(assetId),
-    amount: bnAmount.lt(1) ? bn(1) : bnAmount,
+    amount: bn(amount),
     max: max ? bn(max) : undefined,
   };
 };

packages/account/src/providers/coin-quantityfy.test.ts

@@ -9,12 +9,13 @@ import { coinQuantityfy } from './coin-quantity';
 describe('coinQuantityfy', () => {
   const baseAssetId = ZeroBytes32;
 
-  it('amount that is less than 1 is rounded up to 1', () => {
-    expect(coinQuantityfy([Number.MIN_VALUE, baseAssetId]).amount.toNumber()).toEqual(1);
-    expect(coinQuantityfy([0, baseAssetId]).amount.toNumber()).toEqual(1);
-    expect(coinQuantityfy([1 - Number.EPSILON, baseAssetId]).amount.toNumber()).toEqual(1);
+  it('amount that is less than 1 is rounded to 0', () => {
+    expect(coinQuantityfy([Number.MIN_VALUE, baseAssetId]).amount.toNumber()).toEqual(0);
+    expect(coinQuantityfy([0.2, baseAssetId]).amount.toNumber()).toEqual(0);
+    expect(coinQuantityfy([1 - Number.EPSILON, baseAssetId]).amount.toNumber()).toEqual(0);
   });
-  test('amount of return value is set properly', () => {
+
+  it('amount of return value is set properly', () => {
     expect(coinQuantityfy([2, baseAssetId]).amount.toNumber()).toEqual(2);
     const maxPlusOne = new BN(Number.MAX_SAFE_INTEGER).add(new BN(1));
     expect(coinQuantityfy([maxPlusOne, baseAssetId]).amount.toString()).toEqual(

packages/account/src/providers/provider.ts

@@ -1554,7 +1554,7 @@ export default class Provider {
         .map(coinQuantityfy)
         .map(({ assetId, amount, max: maxPerAsset }) => ({
           assetId: hexlify(assetId),
-          amount: amount.toString(10),
+          amount: (amount.eqn(0) ? bn(1) : amount).toString(10),
           max: maxPerAsset ? maxPerAsset.toString(10) : undefined,
         })),
       excludedIds: idsToExclude,

packages/fuel-gauge/src/call-test-contract.test.ts

@@ -165,6 +165,21 @@ describe('CallTestContract', () => {
     expect(value.toHex()).toBe(bn(1_000_000).toHex());
   });
 
+  it('should be possible to forward amount 0 to contract call', async () => {
+    using contract = await setupContract();
+    const baseAssetId = await contract.provider.getBaseAssetId();
+    const { waitForResult } = await contract.functions
+      .return_context_amount()
+      .callParams({
+        forward: [0, baseAssetId],
+      })
+      .call();
+
+    const { value } = await waitForResult();
+
+    expect(value.toNumber()).toBe(0);
+  });
+
   it('Forward asset_id on contract call', async () => {
     using contract = await setupContract();