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

Wire simulation targets into github workflows #95

Closed
9 tasks
iramiller opened this issue Feb 23, 2021 · 2 comments · Fixed by #197
Closed
9 tasks

Wire simulation targets into github workflows #95

iramiller opened this issue Feb 23, 2021 · 2 comments · Fixed by #197
Assignees
Labels
CI Continuous Integration and release process enhancement New feature or request simulation Issues related to simulation testing
Milestone

Comments

@iramiller
Copy link
Member

Summary

Simulation testing is not currently performed on PR or release targets. Update github workflows to run simulation test targets and report on their status as pass/fail.

Problem Definition

Currently simulation targets exist for the base SDK modules with further coverage planned for each of the added Provenance modules. These targets should be run to ensure modifications to the system do not destabilize the blockchain.

Proposal

Wire in simulation testing targets for PR and Release.

PR to Main

  • Non-determinism simulation
  • Import/Export simulation
  • Simulation after import
  • Multi-seed short form simulation

Release

  • Multi-seed long form simulation

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@iramiller iramiller added enhancement New feature or request CI Continuous Integration and release process labels Feb 23, 2021
@iramiller iramiller added this to the 1.0.0 milestone Feb 23, 2021
@iramiller
Copy link
Member Author

this one might be an easy one to wire after the release build stuff @mtps

@iramiller iramiller added the simulation Issues related to simulation testing label Mar 6, 2021
@iramiller
Copy link
Member Author

This issue can be resolved using the existing sdk simulation as a starting point. This would make it easier to complete adding simulation to the Provenance modules

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration and release process enhancement New feature or request simulation Issues related to simulation testing
Projects
Development

Successfully merging a pull request may close this issue.

1 participant