Skip to content

Commit

Permalink
Remove failing test.
Browse files Browse the repository at this point in the history
After modifying the amplifier, there may be shifts between it and the
camera-derived amplifier.  As this isn't simply changing a assertTrue
to assertFalse (as the first amp in the axis that is modified will
actually have shift=0), removing the test seems the only solution.
  • Loading branch information
czwa committed Sep 13, 2021
1 parent d19d98a commit b386cb6
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions tests/test_ingest.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,10 +83,8 @@ def checkRepo(self, files=None):
# comes straight from the camera) won't have those patches, so
# we can't compare it to the amp attached to
# unassembled_subimage (which does have those patches).
# Instead we test that these two have the same
# assembly/orientation:
comparison2 = unassembled_subimage.getDetector()[0].compareGeometry(unassembled_amp)
self.assertFalse(comparison2 & AmplifierGeometryComparison.ASSEMBLY_DIFFERS)

self.assertTrue(comparison2 & AmplifierGeometryComparison.REGIONS_DIFFER)
# ...and that unassembled_subimage's amp has the same regions
# (after accounting for assembly/orientation) as assembled_amp.
Expand Down

0 comments on commit b386cb6

Please sign in to comment.