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.

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

Tests need upgrade/completion #388

Open
ardunn opened this issue Sep 23, 2021 · 0 comments
Open

Tests need upgrade/completion #388

ardunn opened this issue Sep 23, 2021 · 0 comments

Comments

@ardunn
Copy link
Collaborator

ardunn commented Sep 23, 2021

  • Tests should be organized on a module-by-module basis; they are currently like halfway there
  • Many test files bloating the repo that are never used
  • Test files are not separated by module
  • Some module need need more testing:
    • structure.validation: tests were disabled to accommodate changes in validation from [mat-2116] Rework CLI using Click, remove JSON requirements for use #107 and no longer apply to integration of validation with structuring
    • CLI needs an end-to-end test on a minimal set of small files
    • CLI needs a few more test cases for structure, featurize , train, predict
    • CLI inspect test needs to be debugged on CI although passing locally
    • auto_load and auto_load_processed need to run thru at least one of each file, plus a legacy file for auto_loaded_processed
  • Re-enable windows tests?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant