Skip to content

test: add litesvm ts testcases#585

Merged
dev-jodee merged 1 commit into
solana-foundation:mainfrom
aarjn:cpi-test
Jun 30, 2026
Merged

test: add litesvm ts testcases#585
dev-jodee merged 1 commit into
solana-foundation:mainfrom
aarjn:cpi-test

Conversation

@aarjn

@aarjn aarjn commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

hy @Perelyn-sama raised PR here addressing the closed PR #392

Also I have noticed almost every ts test file for native program in basic are in bankrun.

@dev-jodee dev-jodee self-requested a review June 24, 2026 15:26

@dev-jodee dev-jodee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you fix the CI gh action failure

@greptile-apps

greptile-apps Bot commented Jun 29, 2026

Copy link
Copy Markdown

Greptile Summary

This PR moves the native CPI TypeScript tests onto LiteSVM. The main changes are:

  • Adds litesvm to the native CPI package dependencies.
  • Builds the hand and lever SBF programs into test fixtures before running tests.
  • Rewrites the CPI test to load fixture programs and send transactions through LiteSVM.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
basics/cross-program-invocation/native/tests/test.ts Rewrites the CPI flow to use LiteSVM with Mocha tests and fixture-loaded programs.
basics/cross-program-invocation/native/package.json Adds LiteSVM and updates the build-and-test script to build both native programs before tests run.
basics/cross-program-invocation/native/pnpm-lock.yaml Locks LiteSVM and its platform-specific optional packages.

Reviews (2): Last reviewed commit: "test: add litesvm ts testcases" | Re-trigger Greptile

Comment thread basics/cross-program-invocation/native/tests/test.ts Outdated
Comment thread basics/cross-program-invocation/native/tests/test.ts Outdated
@dev-jodee dev-jodee merged commit e9387b9 into solana-foundation:main Jun 30, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants