fix: Wrap stream passed to reader in C2paStream, add check to tests for validation_status

Author: barnjamin

c2pa/c2pa_api/c2pa_api.py

@@ -38,7 +38,7 @@ class Reader(api.Reader):
     def __init__(self, format, stream, manifest_data=None):
         super().__init__()
         if manifest_data is not None:
-            self.from_manifest_data_and_stream(manifest_data, format, stream)
+            self.from_manifest_data_and_stream(manifest_data, format, C2paStream(stream))
         else:
             self.from_stream(format, C2paStream(stream))
 

tests/test_unit_tests.py

@@ -98,19 +98,23 @@ def test_streams_sign(self):
             builder.sign(TestBuilder.signer, "image/jpeg", file, output)
             output.seek(0)
             reader = Reader("image/jpeg", output)
-            self.assertIn("Python Test", reader.json())
+            json_data = reader.json()
+            self.assertIn("Python Test", json_data)
+            self.assertNotIn("validation_status", json_data)
 
     def test_archive_sign(self):
         with open(testPath, "rb") as file:
             builder = Builder(TestBuilder.manifestDefinition)
-            archive = byte_array = io.BytesIO(bytearray())
+            archive = io.BytesIO(bytearray())
             builder.to_archive(archive)
             builder = Builder.from_archive(archive)
-            output = byte_array = io.BytesIO(bytearray())
+            output = io.BytesIO(bytearray())
             builder.sign(TestBuilder.signer, "image/jpeg", file, output)
             output.seek(0)
             reader = Reader("image/jpeg", output)
-            self.assertIn("Python Test", reader.json())
+            json_data = reader.json()
+            self.assertIn("Python Test", json_data)
+            self.assertNotIn("validation_status", json_data)
 
     def test_remote_sign(self):
         with open(testPath, "rb") as file:
@@ -120,7 +124,9 @@ def test_remote_sign(self):
             manifest_data = builder.sign(TestBuilder.signer, "image/jpeg", file, output)
             output.seek(0)
             reader = Reader("image/jpeg", output, manifest_data)
-            self.assertIn("Python Test", reader.json())
+            json_data = reader.json()
+            self.assertIn("Python Test", json_data)
+            self.assertNotIn("validation_status", json_data)
 
 if __name__ == '__main__':
-    unittest.main()
+    unittest.main()