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

Update and/or document missing coverage #215

Closed
3 tasks done
skliper opened this issue Mar 25, 2022 · 0 comments · Fixed by #267
Closed
3 tasks done

Update and/or document missing coverage #215

skliper opened this issue Mar 25, 2022 · 0 comments · Fixed by #267
Assignees
@skliper
Labels
draco-rc2 unit-test Related to coverage or functional tests
Milestone
Draco

Comments

@skliper
Copy link
Contributor

skliper commented Mar 25, 2022

Checklist (Please check before submitting)

  • I reviewed the Contributing Guide.
  • I reviewed the CF README.md file to see if the feature is in the major future work.
  • I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.

Is your feature request related to a problem? Please describe.
Failure: Coverage cf lines 99.9% functions 100.0% branches 99.2%

Describe the solution you'd like
Fix where possible, elsewise issues should document all uncovered lines/branches and disposition (why it's ok as-is)

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added the unit-test Related to coverage or functional tests label Mar 25, 2022
@skliper skliper added this to the Draco milestone Mar 25, 2022
@skliper skliper self-assigned this Jun 16, 2022
skliper added a commit to skliper/CF that referenced this issue Jun 17, 2022
@skliper
Fix nasa#215, Improve unit test coverage and simplify tests
e38770e
skliper added a commit to skliper/CF that referenced this issue Jun 17, 2022
@skliper
Fix nasa#215, Update workflow for full coverage enforcement
a930392
skliper added a commit to skliper/CF that referenced this issue Jun 17, 2022
@skliper
Fix nasa#215, Improve unit test coverage and simplify tests
b6ab955
skliper added a commit to skliper/CF that referenced this issue Jun 17, 2022
@skliper
Fix nasa#215, Update workflow for full coverage enforcement
022c496
@skliper skliper mentioned this issue Jun 17, 2022
Merged
2 tasks
astrogeco added a commit that referenced this issue Jun 22, 2022
@astrogeco
Merge pull request #267 from skliper/fix215-improve_coverage
bff67f8 
Fix #194 #108 #215 #27 #266, Coverage improvements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
draco-rc2 unit-test Related to coverage or functional tests
Projects
None yet
Milestone
Draco
Development

Successfully merging a pull request may close this issue.

Fix #194 #108 #215 #27 #266, Coverage improvements skliper/CF
1 participant
@skliper