test a scenario in execution #2079
Assignees
Labels
need spec
This issue need more specification
smart-contracts
Issue related to the smart contract world
tests
Issues related to testing the node
Milestone
Comments
AureliaDolo
added
smart-contracts
|
@damip I would happily take suggestions on this one And I'm not sure I see where we have anyway of seeing if everything is ok in the execution component |
|
for most execution tests to be possible, we first need to implement SCE ledger readout functions (also useful for the API/client) |
bors bot
added a commit
that referenced
this issue
Feb 23, 2022
2296: Add tests for massa execution r=adrien-zinger a=adrien-zinger Add tests for the execution module & fix events. Related with the issue #2079, we want to add test in the execution module. I suggest to not duplicate all tests, but test the layer of the massa usage. For example, we add the events and the current PR test if the implementation is good. Add a unit test for the event generation, build a local wasm project on massa-execution build. Then execute the test. Fixes: the events were not stored correctly in `final_events` Changes: Filter slot, include last slot. _this PR was a draft because we need to merge #2290 before_ Co-authored-by: Adrien Zinger <zinger.ad@gmail.com>
bors bot
added a commit
that referenced
this issue
Mar 22, 2022
2296: Add tests for massa execution r=yvan-sraka a=adrien-zinger Add tests for the execution module & fix events. Related with the issue #2079, we want to add test in the execution module. I suggest to not duplicate all tests, but test the layer of the massa usage. For example, we add the events and the current PR test if the implementation is good. Add a unit test for the event generation, build a local wasm project on massa-execution build. Then execute the test. Fixes: the events were not stored correctly in `final_events` Changes: Filter slot, include last slot. _this PR was a draft because we need to merge #2290 before_ Co-authored-by: Adrien Zinger <zinger.ad@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: