Merge pull request #328 from ImagingDataCommons/bug/enforce_series_description_length

Enforce series description length

Author: CPBridge

src/highdicom/base.py

@@ -191,6 +191,7 @@ def __init__(
         self.SeriesNumber = series_number
         self.Modality = modality
         if series_description is not None:
+            _check_long_string(series_description)
             self.SeriesDescription = series_description
 
         # Equipment

tests/test_base.py

@@ -1,3 +1,4 @@
+import re
 import pytest
 import unittest
 
@@ -99,6 +100,24 @@ def test_implicit_vr(self):
             transfer_syntax_uid=ImplicitVRLittleEndian,
         )
 
+    def test_series_description_too_long(self):
+        msg = (
+            "Values of DICOM value representation Long "
+            "String (LO) must not exceed 64 characters."
+        )
+        with pytest.raises(ValueError, match=re.escape(msg)):
+            SOPClass(
+                study_instance_uid=UID(),
+                series_instance_uid=UID(),
+                series_number=1,
+                sop_instance_uid=UID(),
+                sop_class_uid='1.2.840.10008.5.1.4.1.1.88.33',
+                instance_number=1,
+                modality='SR',
+                manufacturer='highdicom',
+                series_description="abc" * 100,
+            )
+
 
 class TestEndianCheck(unittest.TestCase):