openmicroscopy icon indicating copy to clipboard operation
openmicroscopy copied to clipboard

Duplication tests: extend coverage for all shapes and external info

Open sbesson opened this issue 2 years ago • 0 comments

This PR extends the scope of the omero.cmd.Duplicate integration tests to:

  • test the duplication of all OMERO shapes rather a handful of them
    • Label, Mask, Polygon and Polyline are being added to the testDuplicateShapeProperties
    • Path is being ignored as this shape is deprecated but can easily be added if desired
  • test the duplication of objects annotated with ExternalInfo
    • the assertions are updated to test both existence of the details.externalInfo as well as the underlying properties
    • initially, this PR amends existing tests duplicating XmlAnnotation, ROI and all Shape objects to cover ExternalInfo

Note the ExternalInfo tests will fail with the current release of omero-blitz 5.5.12 and will require the implementation changes proposed in https://github.com/ome/omero-blitz/pull/129 in order to pass.

/cc @emilroz @muhanadz

sbesson avatar Oct 11 '22 11:10 sbesson