fix:signfile

Author: Tania Mathern

src/c2pa/c2pa.py

@@ -3749,7 +3749,7 @@ def sign_file(
                 if signer is not None:
                     return self.sign(signer, mime_type, source_file, dest_file)
                 # else:
-                return self.sign(mime_type, source_file, dest_file)
+                return self.sign_with_context(mime_type, source_file, dest_file)
         except Exception as e:
             raise C2paError(f"Error signing file: {str(e)}") from e
 

tests/test_unit_tests.py

@@ -5606,6 +5606,42 @@ def test_contextual_builder_sign_no_signer_raises(self):
         builder.close()
         context.close()
 
+    def test_sign_file_with_context_signer_no_explicit_signer(self):
+        signer = self._ctx_make_signer()
+        context = Context(signer=signer)
+        builder = Builder(
+            self.test_manifest, context=context,
+        )
+        with tempfile.TemporaryDirectory() as temp_dir:
+            dest_path = os.path.join(temp_dir, "out.jpg")
+            manifest_bytes = builder.sign_file(
+                source_path=DEFAULT_TEST_FILE,
+                dest_path=dest_path,
+            )
+            self.assertIsNotNone(manifest_bytes)
+            self.assertGreater(len(manifest_bytes), 0)
+            reader = Reader(dest_path)
+            data = reader.json()
+            self.assertIsNotNone(data)
+            reader.close()
+        builder.close()
+        context.close()
+
+    def test_sign_file_no_signer_raises(self):
+        context = Context()
+        builder = Builder(
+            self.test_manifest, context=context,
+        )
+        with tempfile.TemporaryDirectory() as temp_dir:
+            dest_path = os.path.join(temp_dir, "out.jpg")
+            with self.assertRaises(Error):
+                builder.sign_file(
+                    source_path=DEFAULT_TEST_FILE,
+                    dest_path=dest_path,
+                )
+        builder.close()
+        context.close()
+
 
 class TestContextIntegration(TestContextAPIs):