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

implemented a simple test which will take snapshots over a whole year… #174

Merged
merged 4 commits into from Jun 28, 2018

Conversation

Projects
None yet
2 participants
@phreaker0
Collaborator

phreaker0 commented Dec 6, 2017

… and checks the resulting snapshot list

The test is intended to be ran in a dedicated VM as it will mess with the system time to properly test
sanoid. A simple check is in place to make sure it is running in a vm, if not it asks the user if he wan't to proceed.
It is very time consuming but I use it to test my changes and it really help's a lot with making sure nothing broke. And so it may be useful for others too.

phreaker0 added some commits Dec 6, 2017

implemented a simple test which will take snapshots over a whole year…
… and

checks the resulting snapshot list
@jimsalterjrs

This comment has been minimized.

Owner

jimsalterjrs commented Jun 28, 2018

thank you for this! I've been kinda stressing out over having no automated tests for a while now. I haven't had a chance to really look at this yet, but it's obviously not going to break anything for normal users so I'm going to go ahead and merge. I might want to chat with you some about usage later. I'd LOVE to get automated continuous integration set up here.

@jimsalterjrs jimsalterjrs merged commit 50bd897 into jimsalterjrs:master Jun 28, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment