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

Automatically verify the test suite #20

Closed
AlexandruIca opened this issue Apr 30, 2022 · 0 comments · Fixed by #21
Closed

Automatically verify the test suite #20

AlexandruIca opened this issue Apr 30, 2022 · 0 comments · Fixed by #21
Assignees
@AlexandruIca
Labels
documentation Improvements or additions to documentation improvement Improving an existing functionality setup Project setup
Projects
Backlog
Milestone
Polishing basic s...

Comments

@AlexandruIca
Copy link
Owner

Currently the tests are only run, they are not "checked" for validity automatically. I'd want to run SHA-1 (or something like it) on the resulting image of every test and save it. After that, every test would check its hash against the saved one, checking if it's the same. This way I can track changes in the library that affect rendering.
@AlexandruIca AlexandruIca self-assigned this Apr 30, 2022
@AlexandruIca AlexandruIca added documentation Improvements or additions to documentation improvement Improving an existing functionality setup Project setup labels Apr 30, 2022
@AlexandruIca AlexandruIca added this to ToDo in Backlog via automation Apr 30, 2022
@AlexandruIca AlexandruIca added this to the Polishing basic stuff milestone Apr 30, 2022
@AlexandruIca AlexandruIca mentioned this issue Apr 30, 2022
Closed
AlexandruIca added a commit that referenced this issue Apr 30, 2022
@AlexandruIca
build: Add sha2 as a dependency (#20)
6c12044 
This will be used for tests to automatically validate the integration
tests.
AlexandruIca added a commit that referenced this issue Apr 30, 2022
@AlexandruIca
core: Make PathOps Clone + Copy (#20)
02ab2ca
AlexandruIca added a commit that referenced this issue Apr 30, 2022
@AlexandruIca
test: Add tests/common.rs (#20)
c771c69 
This will store a bunch of helpers for each test.
AlexandruIca added a commit that referenced this issue Apr 30, 2022
@AlexandruIca
test: Use common.rs in the integration tests (#20)
bb9a89f 
Now we finally DRY!
@AlexandruIca AlexandruIca linked a pull request Apr 30, 2022 that will close this issue
#20 Automatic test validation #21
Merged
Backlog automation moved this from ToDo to Done Apr 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AlexandruIca AlexandruIca

Labels
documentation Improvements or additions to documentation improvement Improving an existing functionality setup Project setup
Projects
Backlog
Done
Milestone
Polishing basic stuff
Development

Successfully merging a pull request may close this issue.

#20 Automatic test validation AlexandruIca/verg
1 participant
@AlexandruIca