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

Improve NEXRAD tests #249

Open
dopplershift opened this issue Nov 4, 2016 · 0 comments
Open

Improve NEXRAD tests #249

dopplershift opened this issue Nov 4, 2016 · 0 comments
Labels
Area: IO Pertains to reading data Area: Tests Affects tests Type: Maintenance Updates and clean ups (but not wrong)

Comments

@dopplershift
Copy link
Member

Most of the NEXRAD tests are just smoke tests and aren't validating the results of the parsing. We should add parameterized tests (like GINI has) to validate all of the structs being parsed--I sense this will likely reveal at least a couple issues.
@dopplershift dopplershift added Type: Maintenance Updates and clean ups (but not wrong) Area: Tests Affects tests Area: IO Pertains to reading data labels Nov 4, 2016
@dopplershift dopplershift modified the milestone: Fall 2017 Mar 10, 2017
@jrleeman jrleeman modified the milestones: Fall 2017, Winter 2017 Oct 26, 2017
@jrleeman jrleeman removed this from the 0.7 milestone Nov 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: IO Pertains to reading data Area: Tests Affects tests Type: Maintenance Updates and clean ups (but not wrong)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dopplershift @jrleeman