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

Migrate all unittests to pytest #1252

Open
20 tasks
Taniya-Das opened this issue Jun 12, 2023 · 1 comment
Open
20 tasks

Migrate all unittests to pytest #1252

Taniya-Das opened this issue Jun 12, 2023 · 1 comment
Assignees
@Taniya-Das
Labels
testing

Comments

@Taniya-Das
Copy link
Member

Taniya-Das commented Jun 12, 2023 •
edited by PGijsbers

[edited by @PGijsbers] Currently a lot of unit tests are written with the unittest module, but we want to use pytest instead. Generally speaking, this means extracting the test methods into pytest functions, optionally adding things like pytest fixtures to replace setup/tear-down logic and using plain assert instead of the unittest-based assert methods. Below is an overview of which files have already been migrated:

openml-python/tests/

  • test_datasets/
    • *.py (added as we go)
  • test_evaluations/
    • *.py (added as we go)
  • test_extensions/
    • *.py (added as we go)
  • test_flows/
    • *.py (added as we go)
  • test_openml/
    • *.py (added as we go)
  • test_runs/
    • *.py (added as we go)
  • test_setups/
    • *.py (added as we go)
  • test_study/
    • *.py (added as we go)
  • test_tasks/
    • *.py (added as we go)
  • test_utils/
    • *.py (added as we go)
@Taniya-Das Taniya-Das self-assigned this Jul 6, 2023
@PGijsbers
Copy link
Collaborator

Working on test_utils: #1269

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Taniya-Das Taniya-Das

Labels
testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@PGijsbers @Taniya-Das