Skip to content

Commit 93d3ab3

Browse files
committed
Add tests for focal planes
1 parent 6fc0bae commit 93d3ab3

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

src/highdicom/image.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3064,8 +3064,6 @@ def _prepare_spatial_metadata(
30643064
)
30653065

30663066
if not positions_are_copied:
3067-
# Plane positions may not match source images
3068-
# TODO in this else block
30693067
if number_of_planes != len(plane_positions):
30703068
raise ValueError(
30713069
'Number of PlanePositionSequence items provided '

tests/test_seg.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1139,6 +1139,7 @@ def test_construction_2(self):
11391139
self._sm_image.TotalPixelMatrixRows
11401140
assert instance.TotalPixelMatrixColumns == \
11411141
self._sm_image.TotalPixelMatrixColumns
1142+
assert instance.TotalPixelMatrixFocalPlanes == 1
11421143
assert len(instance.SharedFunctionalGroupsSequence) == 1
11431144
shared_item = instance.SharedFunctionalGroupsSequence[0]
11441145
assert len(shared_item.PixelMeasuresSequence) == 1
@@ -1638,6 +1639,8 @@ def test_construction_7(self):
16381639
instance.TotalPixelMatrixRows # noqa: B018
16391640
with pytest.raises(AttributeError):
16401641
instance.TotalPixelMatrixColumns # noqa: B018
1642+
with pytest.raises(AttributeError):
1643+
instance.TotalPixelMatrixFocalPlanes # noqa: B018
16411644
assert len(instance.SegmentSequence) == 2
16421645
assert instance.SegmentSequence[0].SegmentNumber == 1
16431646
assert instance.SegmentSequence[1].SegmentNumber == 2

0 commit comments

Comments
 (0)