Skip to content

Commit c399d79

Browse files
committed
fix: Return sign values
1 parent e9828ea commit c399d79

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

src/c2pa/c2pa.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1928,7 +1928,7 @@ def sign(
19281928
dest_stream = Stream(dest)
19291929

19301930
# Use the internal stream-base signing logic
1931-
self._sign_internal(signer, format, source_stream, dest_stream)
1931+
return self._sign_internal(signer, format, source_stream, dest_stream)
19321932

19331933
def sign_file(self,
19341934
source_path: Union[str,

tests/test_unit_tests.py

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -354,6 +354,21 @@ def test_remote_sign(self):
354354
reader = Reader("image/jpeg", output)
355355
output.close()
356356

357+
def test_remote_sign_using_returned_bytes(self):
358+
with open(self.testPath, "rb") as file:
359+
builder = Builder(self.manifestDefinition)
360+
builder.set_no_embed()
361+
with io.BytesIO() as output_buffer:
362+
_, manifest_data = builder.sign(
363+
self.signer, "image/jpeg", file, output_buffer)
364+
output_buffer.seek(0)
365+
read_buffer = io.BytesIO(output_buffer.getvalue())
366+
367+
with Reader("image/jpeg", read_buffer, manifest_data) as reader:
368+
manifest_data = reader.json()
369+
self.assertIn("Python Test", manifest_data)
370+
self.assertNotIn("validation_status", manifest_data)
371+
357372
def test_sign_all_files(self):
358373
"""Test signing all files in both fixtures directories"""
359374
signing_dir = os.path.join(self.data_dir, "files-for-signing-tests")

0 commit comments

Comments
 (0)