-
Notifications
You must be signed in to change notification settings - Fork 61
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
Missing pytest coverage #217
Comments
Ideally, these should all be covered. I think writing tests to cover all of these items is probably a mult-issue type of thing, but I would opt to (eventually) have them covered as opposed to ignoring them (as these are actual gaps in the present testing framework). |
Option 1: Would you prefer to create an issue for each? Hovering over each item lets me create a separate issue. Option 2: Or you mean |
That's neat that you can create an issue from the list here that you created. Honestly, in that case, I think the consolidated list you have here could just stay as it is. If/when we are able to address the coverage issues for a given test, we can convert the item in question to an issue and close it out. Capturing it all here is great though! |
Pytest coverage is marking some lines as uncovered through tests. Either have pytest ignore these lines or add tests for these lines.
The text was updated successfully, but these errors were encountered: