Skip to content

Add Integration Test for Stellar Asset Contract (#1809)#1815

Open
guptapratykshh wants to merge 12 commits intohyperledger-solang:mainfrom
guptapratykshh:feature-stellar-asset-integration-test
Open

Add Integration Test for Stellar Asset Contract (#1809)#1815
guptapratykshh wants to merge 12 commits intohyperledger-solang:mainfrom
guptapratykshh:feature-stellar-asset-integration-test

Conversation

@guptapratykshh
Copy link
Contributor

Description

This PR adds comprehensive JavaScript integration test for the Stellar Asset Contract. The test demonstrates how to interact with the contract on the Stellar Soroban testnet, covering deployment, invocation, transfers, approvals, error handling, and edge cases.

Fixes #1809

Test Coverage -

  • Balance queries
  • Token transfers between accounts
  • Approvals and transferFrom operations
  • Error handling for invalid operations
  • Edge cases
  • Multiple rapid transfers
  • All secrets and contract IDs are read from files, not hardcoded

…ets and contract ID

Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
@guptapratykshh
Copy link
Contributor Author

@salaheldinsoliman

@salaheldinsoliman
Copy link
Contributor

@guptapratykshh Soroban integration tests are failing, could you make sure they pass by running them locally first?
The steps for running and preparing the tests are in integration/soroban/package.json.
You should run:

npm install
npm run build
npm run setup
npm run test

@salaheldinsoliman
Copy link
Contributor

@guptapratykshh I have done some re-writing for the soroban integration test, since the protocol 23 release.
Could you take another look at this one please?

@salaheldinsoliman
Copy link
Contributor

Hi @guptapratykshh, the soroban integration tests is failing, could you please re-check that?

@guptapratykshh guptapratykshh force-pushed the feature-stellar-asset-integration-test branch 2 times, most recently from d0583a4 to 8c6dc1c Compare November 15, 2025 22:04
@salaheldinsoliman
Copy link
Contributor

salaheldinsoliman commented Nov 16, 2025

@guptapratykshh looks good 👍
It is missing a DCO check, can you sign the commits please?
Check: https://github.com/hyperledger-solang/solang/pull/1815/checks?check_run_id=55496655096

@guptapratykshh guptapratykshh force-pushed the feature-stellar-asset-integration-test branch from 84b6dd3 to 29213c6 Compare November 16, 2025 22:35
…ets and contract ID

Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
…Desktop/solang && find . -name *llvm* -o -name *env* -o -name *setup* | head -20

Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
@guptapratykshh guptapratykshh force-pushed the feature-stellar-asset-integration-test branch from 044d3f1 to ea33da5 Compare November 16, 2025 22:38
Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
@guptapratykshh guptapratykshh force-pushed the feature-stellar-asset-integration-test branch from f1a654c to 3deb074 Compare November 17, 2025 09:58
Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
@guptapratykshh
Copy link
Contributor Author

guptapratykshh commented Nov 17, 2025

Please look into this . @salaheldinsoliman

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.

Mentee Task 2: Add integration test for Stellar Asset Contract

2 participants