Skip to content

Conversation

@ngrozae
Copy link
Collaborator

@ngrozae ngrozae commented Jan 19, 2026

🛠 Summary

CVS-169508

Changes:

  1. Preparations for updates in testing flow
  2. Redundant tests removed
  3. Not supported target devices removed from testing

🧪 Checklist

  • Unit tests added.
  • The documentation updated.
  • Change follows security best practices.
    ``

Copilot AI review requested due to automatic review settings January 19, 2026 12:07
Copy link
Contributor

Copilot AI left a comment

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 updates the functional test suite for OpenVINO Model Server (OVMS) to prepare for changes in the testing flow. The changes include removing redundant tests, updating import statements to use absolute paths, and removing support for deprecated target devices.

Changes:

  • Updated all import statements across test files to use fully qualified paths (e.g., from configfrom tests.functional.config)
  • Removed several test files for redundant test scenarios (batching, mapping, multi-models, update flows, and specific model types)
  • Removed support for MYRIAD, HDDL, and CUDA target devices, consolidating to CPU, GPU, and NPU
  • Added new helper functions and pytest markers for test organization

Reviewed changes

Copilot reviewed 43 out of 47 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tests/functional/utils/*.py Updated imports to use absolute paths from tests.functional namespace
tests/functional/test_*.py Removed redundant test files and updated remaining tests with new imports and device constraints
tests/functional/object_model/*.py Updated imports to use absolute paths
tests/functional/fixtures/*.py Updated imports and fixture dependencies to use absolute paths
tests/functional/constants/*.py Introduced TargetDevice class to replace string constants, removed deprecated device types
tests/functional/config.py Updated to use new TargetDevice class and removed deprecated device timeout configurations
tests/functional/conftest.py Restructured pytest plugin loading and moved fixtures to separate file
Makefile Changed virtualenv creation from python3 -m virtualenv to python3 -m venv

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

@ngrozae ngrozae force-pushed the CVS-169508_enable_ovms_functional_tests branch 2 times, most recently from a14d39f to 76e53d7 Compare January 19, 2026 16:57
@ngrozae ngrozae requested review from pgladkows and porlows1 January 20, 2026 09:18
@ngrozae ngrozae force-pushed the CVS-169508_enable_ovms_functional_tests branch from 76e53d7 to 777633e Compare January 20, 2026 09:23
Copy link
Collaborator

@porlows1 porlows1 left a comment

Choose a reason for hiding this comment

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

Approved

@ngrozae ngrozae requested a review from dtrawins January 20, 2026 10:50
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

Successfully merging this pull request may close these issues.

4 participants