Skip to content
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.

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

chore(devnet): add sandbox & test contracts with drink #277

Closed
wants to merge 78 commits into from

Conversation

chungquantin
Copy link
Collaborator

@chungquantin chungquantin commented Sep 11, 2024

Description

Implementation of the pop_drink::Sandbox struct for the Pop Network runtimes (located in pop-node/runtime) required for the quasi testing with drink.

This PR creates a pop-sandbox crate that implements the pop-drink::Sandbox runtime environment for the Devnet => DevnetSandbox.

In the context of quasi-testing with pop-drink, a sandbox refers to an isolated runtime environment that simulates the behavior of a full node, without requiring an actual node. It can emulate key processes (where runtime pallets are involved) such as block initialization, execution, and block finalization.

Tasks

References

Daanvdplas and others added 30 commits May 19, 2024 17:38
# This is the 1st commit message:

refactor: general

# This is the commit message #2:

init

# This is the commit message #3:

begin refactor

# This is the commit message #4:

refactor: error handling

# This is the commit message #5:

tests: add error handling tests

# This is the commit message #6:

WIP

# This is the commit message #7:

finalise error handling

# This is the commit message #8:

refactor: easier review
Base automatically changed from daan/api to main September 13, 2024 20:18
Copy link
Collaborator

@Daanvdplas Daanvdplas left a comment

Choose a reason for hiding this comment

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

Looking good. I think we should be able to abstract the complexities away much more. The developers should be able to get a lot of info on how to use this test suite, thats it.

I think we should not put too much focus on this right now before we have the psp22 example to a good state. Then we can have better judgement on how to move forward with the this pop-drink / sandbox

pop-sandbox/README.md Outdated Show resolved Hide resolved
pop-sandbox/README.md Outdated Show resolved Hide resolved
pop-sandbox/src/lib.rs Outdated Show resolved Hide resolved
pop-sandbox/src/lib.rs Outdated Show resolved Hide resolved
@chungquantin chungquantin changed the base branch from main to chungquantin/feat-psp22_with_drink September 23, 2024 09:50
Copy link
Collaborator

@Daanvdplas Daanvdplas left a comment

Choose a reason for hiding this comment

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

LGTM

pop-sandbox/README.md Outdated Show resolved Hide resolved
pop-sandbox/README.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants