Skip to content

Remove mox3 from the test requirements#60

Merged
jburel merged 2 commits intoome:masterfrom
sbesson:mox3_removal
Jan 17, 2024
Merged

Remove mox3 from the test requirements#60
jburel merged 2 commits intoome:masterfrom
sbesson:mox3_removal

Conversation

@sbesson
Copy link
Member

@sbesson sbesson commented Nov 16, 2023

See ome/omero-py#385

  • Update tests requirements to require OMERO.py 5.17.0 which deprecates the usage of mox3 for running integration tests
  • Consume omero.testlib.cli.AbstractCLITest

Update tests requirements to require OMERO.py 5.17.0 which
deprecates the usage of mox3 for running integration tests
Consume omero.testlib.cli.AbstractCLITest
@sbesson sbesson requested a review from jburel November 16, 2023 10:26
Copy link
Member

@jburel jburel left a comment

Choose a reason for hiding this comment

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

Changes make sense.
Note that this has not tested with newer Python version since it is based on test-infra which has not been updated yet.

@sbesson sbesson closed this Jan 17, 2024
@sbesson sbesson reopened this Jan 17, 2024
@sbesson sbesson closed this Jan 17, 2024
@sbesson sbesson reopened this Jan 17, 2024
@sbesson
Copy link
Member Author

sbesson commented Jan 17, 2024

Re-executed the GitHub workflow with the updated Docker image using Rocky Linux 9 / Python 3.9. No new issue was detected. Should we get this in to avoid future conflicts?

@jburel jburel merged commit eb785c6 into ome:master Jan 17, 2024
@sbesson sbesson deleted the mox3_removal branch January 17, 2024 17:19
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.

2 participants