Skip to content

Deprecated plugins test#6183

Merged
joshmoore merged 1 commit intoome:developfrom
sbesson:deprecated_plugins_test
Nov 27, 2019
Merged

Deprecated plugins test#6183
joshmoore merged 1 commit intoome:developfrom
sbesson:deprecated_plugins_test

Conversation

@sbesson
Copy link
Copy Markdown
Member

@sbesson sbesson commented Nov 21, 2019

Companion PR to ome/omero-py#124

The renaming of the metadata and import plugins in the omero-py repository prevents the import and the execution of all OMERO.py tests - see https://py3-ci.openmicroscopy.org/jenkins/job/OMERO-test-integration/64/testReport/(root)/(empty)/OmeroPy_test_integration_clitest_test_metadata/

This should fix the import to match the renaming done in #124. As the decoupled plugins have now been upgraded to be Python 3 and are tested via omero-test-infra, an alternative would be to remove these tests.

@snoopycrimecop
Copy link
Copy Markdown
Member

snoopycrimecop commented Nov 21, 2019

Conflicting PR. Removed from build OMERO-push#277. See the console output for more details.
Possible conflicts:

  • PR Parallel fixes #6156 joshmoore 'Parallel fixes'
    • components/tools/OmeroPy/test/integration/clitest/test_upload.py
    • components/tools/OmeroPy/test/integration/clitest/test_metadata.py

--conflicts Conflict resolved in build OMERO-push#123. See the console output for more details.

@sbesson sbesson force-pushed the deprecated_plugins_test branch 2 times, most recently from 907dd68 to 0f910e8 Compare November 27, 2019 16:55
@joshmoore
Copy link
Copy Markdown
Member

This looks good to be in terms of 124. I very much assume things aren't being fully tested, so things may go red tomorrow. If so, we can review removing these files completely.

@joshmoore joshmoore merged commit fa05f9c into ome:develop Nov 27, 2019
@sbesson sbesson deleted the deprecated_plugins_test branch November 27, 2019 17:56
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.

3 participants