fix: sort addresses

[rictorlome]

Preventing errors such as:

Details: couldn't issue tx: failed verification: failed execution: standard tx 5BFNdkBJWZhqx3NkrnHMEhetJ4YtZC5QxLdNREGJGF9XznGvf failed execution: output failed verification: addresses not sorted and unique

reproduction steps (on master):
in examples/p-chain/base.ts, testing against fuji:
add the following lines options:

  const P_CHAIN_ADDRESS_2 = 'P-fuji14n4duc4qkmh4q5vvgyh03yrcf4yxtk2ap8mj8q';
  const changeAddresses = [
    bech32ToBytes(P_CHAIN_ADDRESS_2),
    bech32ToBytes(P_CHAIN_ADDRESS),
  ];
  changeAddresses.sort(bytesCompare);
  changeAddresses.reverse(); // NOTE: this will cause the error
  const tx = newBaseTx(context, [bech32ToBytes(P_CHAIN_ADDRESS)], utxos, [
    TransferableOutput.fromNative(context.avaxAssetID, BigInt(0.01 * 1e9), [
      bech32ToBytes(P_CHAIN_ADDRESS)
    ]),
  ], 
  {
    changeAddresses
  }

this should throw the error on master, but not on this branch.

after applying this fix, even this is fine:

  const { utxos } = await pvmapi.getUTXOs({ addresses: [P_CHAIN_ADDRESS, P_CHAIN_ADDRESS_2] });
  // NOTE: backwards sorted fromAddressBytes
  const tx = newBaseTx(context, [bech32ToBytes(P_CHAIN_ADDRESS_2), bech32ToBytes(P_CHAIN_ADDRESS)], utxos, [
    TransferableOutput.fromNative(context.avaxAssetID, BigInt(0.01 * 1e9), [
      bech32ToBytes(P_CHAIN_ADDRESS)
    ]),
  ], 
  {
    changeAddresses
  }

https://subnets-test.avax.network/p-chain/tx/Jjw7T3exyvJeBkEZqQhK87SBTrTu4Wj5DUrhj8Dc2RjdwAPra

[bferenc]

Nice addition, thank you! 👏

[erictaylor]

Seems good to me. Thanks for adding the multi-sig indicies test.