img: add _LongIfdEntry._parse_value()

Steve Canny · Steve Canny · commit 02b4a634bcca · 2014-02-27T12:08:36.000-08:00
diff --git a/docx/image/tiff.py b/docx/image/tiff.py
@@ -294,6 +294,16 @@ class _LongIfdEntry(_IfdEntry):
     """
     IFD entry expressed as a long (4-byte) integer
     """
+    @classmethod
+    def _parse_value(cls, stream_rdr, offset, value_count, value_offset):
+        """
+        Return the long int value contained in the *value_offset* field of
+        this entry. Only supports single values at present.
+        """
+        if value_count == 1:
+            return stream_rdr.read_long(offset, 8)
+        else:
+            return 'Multi-value long integer NOT IMPLEMENTED'
 
 
 class _RationalIfdEntry(_IfdEntry):
diff --git a/tests/image/test_tiff.py b/tests/image/test_tiff.py
@@ -14,8 +14,8 @@
 from docx.image.constants import MIME_TYPE, TIFF_TAG
 from docx.image.helpers import BIG_ENDIAN, LITTLE_ENDIAN, StreamReader
 from docx.image.tiff import (
-    _IfdEntries, _IfdEntry, _IfdEntryFactory, _IfdParser, _ShortIfdEntry,
-    Tiff, _TiffParser
+    _IfdEntries, _IfdEntry, _IfdEntryFactory, _IfdParser, _LongIfdEntry,
+    _ShortIfdEntry, Tiff, _TiffParser
 )
 
 from ..unitutil import (
@@ -424,3 +424,12 @@ def it_can_parse_a_short_int_IFD_entry(self):
         stream_rdr = StreamReader(BytesIO(bytes_), BIG_ENDIAN)
         val = _ShortIfdEntry._parse_value(stream_rdr, 0, 1, None)
         assert val == 42
+
+
+class Describe_LongIfdEntry(object):
+
+    def it_can_parse_a_long_int_IFD_entry(self):
+        bytes_ = b'foobaroo\x00\x00\x00\x2A'
+        stream_rdr = StreamReader(BytesIO(bytes_), BIG_ENDIAN)
+        val = _LongIfdEntry._parse_value(stream_rdr, 0, 1, None)
+        assert val == 42
