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.

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

feat: add basic CAS test #42

Closed
wants to merge 5 commits into from
Closed

feat: add basic CAS test #42

wants to merge 5 commits into from

Conversation

troggy
Copy link
Member

@troggy troggy commented Aug 5, 2019 •
edited
Loading

Stop one of the validators, make sure the period is not submitted, run again and make sure the period has went through.

Outline:

  • nodeClient made stoppable from tests (node.stop() and node.start())
  • added CAS test

Requires: #40

Resolves #39

troggy added 4 commits August 5, 2019 18:23
@troggy
feat: add basic CAS test
1be498d 
Stop one of the validators, make sure the period is not submitted, run again and make sure the period has went through
@troggy
allow to pass ENVs to node process
63e8294
@troggy
append to logs
e69aac9
@troggy
delete logs before test, more event delay to prevent consensus errors
6493d84
@troggy troggy closed this Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create basic CAS test
1 participant
@troggy