test: fix tests forward for 1.16.x

[steveluscher]

Summary

1.16 added a space param to gPA and changed an error message. This PR updates the web3.js tests to handle that, and a whole lot more fiddling with object shapes and strictness.

NOTE

The really bad thing about this is that when the tests break, you get this inscrutable error from Jest:

FAIL   Unit Test (Node)  src/rpc-methods/__tests__/get-multiple-accounts-test.ts
  ● Test suite failed to run

    DataCloneError: Symbol(jest.asymmetricMatcher) could not be cloned.

      at reportSuccess (../../node_modules/.pnpm/jest-worker@29.7.0/node_modules/jest-worker/build/workers/threadChild.js:98:34)

I think this is because of the presence of BigInts and Jest's inability to…

TypeError: Do not know how to serialize a BigInt

I haven't filed a bug with Jest yet, but the workaround I have is to:

• Convert all of the toStrictMatch() to toMatchObject()
• Re-run the tests and figure out what broke
• Undo undo undo to put them all back
• Fix the problem

Test Plan

pnpm turbo test:unit:browser
pnpm turbo test:unit:node

Closes #1634.

[steveluscher]

• test: fix tests forward for 1.16.x #1655 : 2 dependent PRs (#1306 , #1640 ) 👈
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @steveluscher and the rest of your teammates on Graphite

[buffalojoec]

I think you forgot getAccountInfo.

Heads up they call it size in this one, not space.
https://docs.solana.com/api/http#getaccountinfo

[steveluscher]

Oh, hunh. All the tests passed, so I stopped. Looking…

[steveluscher]

Oh dear. Those other tests didn't use expect.objectContaining so they weren't exhaustive checks.

[buffalojoec]

Oh dear. Those other tests didn't use expect.objectContaining so they weren't exhaustive checks.

This is what I was afraid of

[steveluscher]

Uh, no really. How do you combine toStrictEqual with inner expect.any(...).

[buffalojoec]

Uh, no really. How do you combine toStrictEqual with inner expect.any(...).

You just put the value instead of expect right? Or are you saying for the whole object?

[steveluscher]

There's a bug in Jest. I'm working through it. Will have a PR in a bit.

[steveluscher]

Crap. Missed the legacy test runner.

[steveluscher]

K, the tests passed at long last. I'm going to slam this in so that we can see a few more test runs over time.

[buffalojoec]

K, the tests passed at long last. I'm going to slam this in so that we can see a few more test runs over time.

[github-actions]

🎉 This PR is included in version 1.78.6 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[github-actions]

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.