-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improve test coverage #81
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
5ad58f3
to
5819899
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Todo: Improve the way cubes are being compared in the write test, to make it deterministic and not randomly fail. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Use tempfile properly. It doesn't like it when we use a property of the object in a context manager.
Fixes ambiguity around passing a cubelist to a function that takes a cube. Also add some more comments. The aim here is to fix the random test failures.
Current syntax uses "|". "or" is not used. | is from 3.10 however, so we instead fall back to the older syntax: typing.Union[a, b]
We don't know that we are going to use these, and can easily recreate them if we do.
e9d9970
to
d47ba0b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice, all looks good eyeballing changelog
I've got it to 100% code coverage, but realistically that is going to drop a bit when we start adding lots of code. But it makes sense to at least start high. |
Now we are measuring test coverage, it makes sense to improve it.