Support the EXIF SShort type (handled similarly as Short).

This doesn't seem to be standard EXIF but it is mentioned in the TIFF specification.

1 files changed, 25 insertions, 0 deletions

diff --git a/test/exif.py b/test/exif.py
index cd077cd..6cb4f77 100644
--- a/test/exif.py
+++ b/test/exif.py
@@ -162,6 +162,31 @@ class TestExifTag(unittest.TestCase):
         self.failUnlessRaises(ExifValueError, tag._convert_to_string, 'invalid')
         self.failUnlessRaises(ExifValueError, tag._convert_to_string, 3.14)
 
+    def test_convert_to_python_sshort(self):
+        # Valid values
+        tag = ExifTag('Exif.Image.TimeZoneOffset')
+        self.assertEqual(tag.type, 'SShort')
+        self.assertEqual(tag._convert_to_python('8'), 8)
+        self.assertEqual(tag._convert_to_python('+5'), 5)
+        self.assertEqual(tag._convert_to_python('-6'), -6)
+
+        # Invalid values
+        self.failUnlessRaises(ExifValueError, tag._convert_to_python, 'abc')
+        self.failUnlessRaises(ExifValueError, tag._convert_to_python, '5,64')
+        self.failUnlessRaises(ExifValueError, tag._convert_to_python, '47.0001')
+        self.failUnlessRaises(ExifValueError, tag._convert_to_python, '1E3')
+
+    def test_convert_to_string_sshort(self):
+        # Valid values
+        tag = ExifTag('Exif.Image.TimeZoneOffset')
+        self.assertEqual(tag.type, 'SShort')
+        self.assertEqual(tag._convert_to_string(12), '12')
+        self.assertEqual(tag._convert_to_string(-3), '-3')
+
+        # Invalid values
+        self.failUnlessRaises(ExifValueError, tag._convert_to_string, 'invalid')
+        self.failUnlessRaises(ExifValueError, tag._convert_to_string, 3.14)
+
     def test_convert_to_python_long(self):
         # Valid values
         tag = ExifTag('Exif.Image.ImageWidth')