Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: solana-labs/solana-web3.js
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v1.56.1
Choose a base ref
...
head repository: solana-labs/solana-web3.js
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v1.56.2
Choose a head ref
  • 1 commit
  • 2 files changed
  • 1 contributor

Commits on Sep 1, 2022

  1. fix: add compute units to jsonrpc parser (#27554)

    @jstarry
    jstarry authored Sep 1, 2022

    Verified

    This commit was signed with the committer’s verified signature.
    KinectTheUnknown David-Joseph Xayavong
    GPG key ID: B1EA99A6C5178449
    Verified
    Learn about vigilant mode
    Copy the full SHA
    ae708ce View commit details
Showing with 10 additions and 0 deletions.
  1. +6 −0 src/connection.ts
  2. +4 −0 test/connection.test.ts
6 changes: 6 additions & 0 deletions src/connection.ts
View file
Original file line number Diff line number Diff line change
@@ -888,6 +888,8 @@ export type ParsedTransactionMeta = {
err: TransactionError | null;
/** The collection of addresses loaded using address lookup tables */
loadedAddresses?: LoadedAddresses;
/** The compute units consumed after processing the transaction */
computeUnitsConsumed?: number;
};

export type CompiledInnerInstruction = {
@@ -917,6 +919,8 @@ export type ConfirmedTransactionMeta = {
err: TransactionError | null;
/** The collection of addresses loaded using address lookup tables */
loadedAddresses?: LoadedAddresses;
/** The compute units consumed after processing the transaction */
computeUnitsConsumed?: number;
};

/**
@@ -1993,6 +1997,7 @@ const ConfirmedTransactionMetaResult = pick({
preTokenBalances: optional(nullable(array(TokenBalanceResult))),
postTokenBalances: optional(nullable(array(TokenBalanceResult))),
loadedAddresses: optional(LoadedAddressesResult),
computeUnitsConsumed: optional(number()),
});

/**
@@ -2017,6 +2022,7 @@ const ParsedConfirmedTransactionMetaResult = pick({
preTokenBalances: optional(nullable(array(TokenBalanceResult))),
postTokenBalances: optional(nullable(array(TokenBalanceResult))),
loadedAddresses: optional(LoadedAddressesResult),
computeUnitsConsumed: optional(number()),
});

const TransactionVersionStruct = union([literal(0), literal('legacy')]);
4 changes: 4 additions & 0 deletions test/connection.test.ts
View file
Original file line number Diff line number Diff line change
@@ -4460,6 +4460,8 @@ describe('Connection', function () {
readonly: [],
writable: [lookupTableAddresses[0]],
});
expect(fetchedTransaction.meta?.computeUnitsConsumed).to.not.be
.undefined;
expect(
fetchedTransaction.transaction.message.addressTableLookups,
).to.eql(addressTableLookups);
@@ -4489,6 +4491,8 @@ describe('Connection', function () {
readonly: [],
writable: [lookupTableAddresses[0]],
});
expect(parsedTransaction?.meta?.computeUnitsConsumed).to.not.be
.undefined;
expect(
parsedTransaction?.transaction.message.addressTableLookups,
).to.eql(addressTableLookups);