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

Re-organise tests #63

Merged
merged 9 commits into from
Mar 24, 2021
Merged

Re-organise tests #63

merged 9 commits into from
Mar 24, 2021

Conversation

adriangonz
Copy link

Fixes #15

Changelog

  • Move tests to top-level folder (so that they are out of the main tempo package)
  • Move Docker and K8s specific fixtures to tests/seldon/k8s and tests/seldon/docker respectively
  • Simplify fixtures so that they are hierarchical (i.e. runtime-agnostic under tests/conftest and specialised for K8s and Docker on each tests subpackage).

@ukclivecox ukclivecox merged commit 7eb6ca2 into SeldonIO:master Mar 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ukclivecox ukclivecox Awaiting requested review from ukclivecox

@axsaucedo axsaucedo Awaiting requested review from axsaucedo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Re-organise test structure and fixtures
2 participants
@adriangonz @ukclivecox