fix token-metadata decoder; add skip-preflight option (#64)

Author: samuelvanderwaal

test-helpers/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tensor-foundation/test-helpers",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "Utility functions for testing Solana programs with the Solana Web3.js library",
   "sideEffects": false,
   "module": "./dist/src/index.mjs",

test-helpers/src/token/extensions/index.ts

@@ -1,6 +1,7 @@
 import {
   addDecoderSizePrefix,
   Decoder,
+  getAddressDecoder,
   getArrayDecoder,
   getStructDecoder,
   getU32Decoder,
@@ -88,15 +89,18 @@ function deserializeTransferHook(extensionBytes: Uint8Array): TransferHook {
 }
 
 function deserializeTokenMetadata(extensionBytes: Uint8Array): TokenMetadata {
-  const updateAuthority = toAddress(extensionBytes.slice(0, 32));
-  const mint = toAddress(extensionBytes.slice(32, 64));
-  const name = getUtf8Decoder().decode(extensionBytes);
-  const symbol = getUtf8Decoder().decode(extensionBytes);
-  const uri = getUtf8Decoder().decode(extensionBytes);
-  const additionalMetadata =
-    getAdditionalMetadataDecoder().decode(extensionBytes);
-
-  return { updateAuthority, mint, name, symbol, uri, additionalMetadata };
+  return getTokenMetadataDecoder().decode(extensionBytes);
+}
+
+function getTokenMetadataDecoder(): Decoder<TokenMetadata> {
+  return getStructDecoder([
+    ['updateAuthority', getAddressDecoder()],
+    ['mint', getAddressDecoder()],
+    ['name', addDecoderSizePrefix(getUtf8Decoder(), getU32Decoder())],
+    ['symbol', addDecoderSizePrefix(getUtf8Decoder(), getU32Decoder())],
+    ['uri', addDecoderSizePrefix(getUtf8Decoder(), getU32Decoder())],
+    ['additionalMetadata', getAdditionalMetadataDecoder()],
+  ]);
 }
 
 function getAdditionalMetadataDecoder(): Decoder<AdditionalMetadata[]> {

test-helpers/src/wns/createNft.ts

@@ -46,6 +46,7 @@ export interface CreateNftArgs {
   group?: Address;
   data?: NftData;
   paymentMint?: Address;
+  skipPreflight?: boolean;
 }
 
 export interface NftData {
@@ -64,6 +65,7 @@ export interface CreateGroupArgs {
   mint?: TransactionSigner;
   paymentMint?: Address;
   data?: GroupData;
+  skipPreflight?: boolean;
 }
 
 export interface GroupData {
@@ -144,13 +146,14 @@ export async function createGroupWithRoyalties(
   await pipe(
     await createDefaultTransaction(client, payer),
     (tx) => appendTransactionMessageInstructions(instructions, tx),
-    (tx) => signAndSendTransaction(client, tx)
+    (tx) =>
+      signAndSendTransaction(client, tx, { skipPreflight: args.skipPreflight })
   );
 
   return { group, mint: mint.address, distribution: distributionAccount };
 }
 
-const testNftData: NftData = {
+export const testNftData: NftData = {
   name: 'Test WNS',
   symbol: 'TWNS',
   uri: 'https://example.com/wns',
@@ -234,7 +237,8 @@ export async function createNft(args: CreateNftArgs): Promise<{
             tx
           )
       : (tx) => tx,
-    (tx) => signAndSendTransaction(client, tx)
+    (tx) =>
+      signAndSendTransaction(client, tx, { skipPreflight: args.skipPreflight })
   );
 
   // Hacky--there's probably a better way to do this.

test-helpers/test/wns.test.ts

@@ -9,6 +9,7 @@ import {
   deserializeExtension,
   ExtensionType,
   findExtraAccountMetaAddress,
+  testNftData,
   TOKEN22_PROGRAM_ID,
   tokenSetup,
 } from '../src/index.js';
@@ -137,4 +138,19 @@ test('create nft with createWnsNftInGroup', async (t) => {
 
   t.assert(extraMetasAccount);
   t.assert(extraMetasAccount!.owner === WEN_NEW_STANDARD_PROGRAM_ADDRESS);
+
+  const tokenMetadata = deserializeExtension(data, ExtensionType.TokenMetadata);
+  t.assert(tokenMetadata?.updateAuthority === authority.address);
+  t.assert(tokenMetadata?.mint === mint);
+  t.assert(tokenMetadata?.name === testNftData.name);
+  t.assert(tokenMetadata?.symbol === testNftData.symbol);
+  t.assert(tokenMetadata?.uri === testNftData.uri);
+  t.assert(tokenMetadata?.additionalMetadata.length === 2);
+  t.assert(tokenMetadata?.additionalMetadata[0].key === 'royalty_basis_points');
+  t.assert(
+    tokenMetadata?.additionalMetadata[0].value ===
+      testNftData.royaltyBasisPoints.toString()
+  );
+  t.assert(tokenMetadata?.additionalMetadata[1].key === authority.address);
+  t.assert(tokenMetadata?.additionalMetadata[1].value === '100');
 });