Skip to content

feat(tests): run inference tests only on selected model type#626

Open
tybulewicz wants to merge 2 commits into
masterfrom
ttybulew/filter_tests_by_model_class
Open

feat(tests): run inference tests only on selected model type#626
tybulewicz wants to merge 2 commits into
masterfrom
ttybulew/filter_tests_by_model_class

Conversation

@tybulewicz

@tybulewicz tybulewicz commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

This PR adds a pytest CLI filter to the functional inference tests so contributors can run the inference suite only for a selected wrapper/model type (and its subclasses), reducing runtime and focusing validation.

Fixes #598

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@tybulewicz tybulewicz requested a review from Copilot July 7, 2026 07:42
@tybulewicz tybulewicz requested a review from a team as a code owner July 7, 2026 07:42
@github-actions github-actions Bot added the tests Related to tests label Jul 7, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a pytest CLI filter to the functional inference tests so contributors can run the inference suite only for a selected wrapper/model type (and its subclasses), reducing runtime and focusing validation.

Changes:

  • Adds a new pytest CLI option to select which model type(s) to run inference tests for.
  • Introduces a session-scoped fixture that resolves the requested model class and skips non-matching parametrized cases.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
model_api/tests/functional/test_inference.py Adds a fixture-driven model-type filter and applies it to test_image_models.
model_api/tests/functional/conftest.py Registers the new CLI option used to select the model type filter.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread model_api/tests/functional/conftest.py
Comment thread model_api/tests/functional/test_inference.py Outdated
@tybulewicz tybulewicz requested a review from mgumowsk July 7, 2026 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run inference tests on selected models

2 participants