Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

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

wants to merge 4 commits into from

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

@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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@salaheldinsoliman salaheldinsoliman Awaiting requested review from salaheldinsoliman salaheldinsoliman is a code owner

@seanyoung seanyoung Awaiting requested review from seanyoung seanyoung is a code owner

@xermicus xermicus Awaiting requested review from xermicus

@LucasSte LucasSte Awaiting requested review from LucasSte

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Mentee Task 2: Add integration test for Stellar Asset Contract

2 participants

@guptapratykshh @salaheldinsoliman