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

[ci] Test stepping function of tvla in CI #151

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

vrozic
Copy link
Contributor

@vrozic vrozic commented Aug 10, 2023

This PR modifies test_general_nonleaking_project() to run TVLA with --number-of-steps 10 and compare the produced result with the checked-in numerical value, rather than just checking for leakage.

Partially addresses #96

@vrozic vrozic force-pushed the ci-test-nstep branch 6 times, most recently from 792d9fa to 7965169 Compare August 15, 2023 14:14
This commit modifies `test_general_nonleaking_project()` to run
TVLA with `--number-of-steps 10` and compare the produced result
with the checked-in numerical value, rather than just checking for
leakage.

Signed-off-by: Vladimir Rozic <[email protected]>
Copy link
Collaborator

@vogelpi vogelpi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @vrozic !

@vrozic vrozic merged commit f1b7617 into lowRISC:master Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants