fix: UtxoIdCoder output index to be u16 (#3064)

* fix: utxo id output index to be u16 * chore: changeset * chore: lint --------- Co-authored-by: Sérgio Torres <30977845+Torres-ssf@users.noreply.github.com>
FuelLabs · Aug 30, 2024 · e286557 · e286557
1 parent 2b45e8d
commit e286557
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 2 deletions.
diff --git a/.changeset/thin-shrimps-remain.md b/.changeset/thin-shrimps-remain.md
@@ -0,0 +1,5 @@
+---
+"@fuel-ts/transactions": patch
+---
+
+fix: `UtxoIdCoder` output index to be `u16`
diff --git a/packages/transactions/src/coders/utxo-id.test.ts b/packages/transactions/src/coders/utxo-id.test.ts
@@ -10,7 +10,7 @@ const B256 = '0xd5579c46dfcc7f18207013e65b44e4cb4e2c2298f4ac457ba8f82743f31e930b
  * @group browser
  */
 describe('UtxoIdCoder', () => {
-  it('can encode UtxoId', () => {
+  it('can encode UtxoId [u8 output]', () => {
     const utxoId: UtxoId = {
       transactionId: B256,
       outputIndex: 0,
@@ -27,6 +27,20 @@ describe('UtxoIdCoder', () => {
     expect(offset).toEqual(40);
     expect(decoded).toEqual(utxoId);
   });
+
+  it('can encode UtxoId [u16 output]', () => {
+    const utxoId: UtxoId = {
+      transactionId: B256,
+      outputIndex: 256,
+    };
+
+    const encoded = hexlify(new UtxoIdCoder().encode(utxoId));
+
+    expect(encoded).toEqual(
+      '0xd5579c46dfcc7f18207013e65b44e4cb4e2c2298f4ac457ba8f82743f31e930b0000000000000100'
+    );
+  });
+
   it('does not encode bad UtxoId', () => {
     const utxoId: UtxoId = {
       // @ts-expect-error: Values shouldn't be assignable

diff --git a/packages/transactions/src/coders/utxo-id.ts b/packages/transactions/src/coders/utxo-id.ts
@@ -14,7 +14,7 @@ export class UtxoIdCoder extends StructCoder<{
   constructor() {
     super('UtxoId', {
       transactionId: new B256Coder(),
-      outputIndex: new NumberCoder('u8', { padToWordSize: true }),
+      outputIndex: new NumberCoder('u16', { padToWordSize: true }),
     });
   }
 }