fix: Add_action API

Author: tmathern

tests/test_unit_tests.py

@@ -2499,12 +2499,11 @@ def test_builder_add_action_to_manifest_with_auto_add(self):
 
         builder.close()
 
-        # Reset settings
+        # Reset settings to default
         load_settings('{"builder":{"actions":{"auto_placed_action":{"enabled":true}}}}')
         load_settings('{"builder":{"actions":{"auto_opened_action":{"enabled":true}}}}')
         load_settings('{"builder":{"actions":{"auto_created_action":{"enabled":true}}}}')
 
-
     def test_builder_minimal_manifest_add_actions_and_sign_no_auto_add(self):
         # For testing, remove auto-added actions
         load_settings('{"builder":{"actions":{"auto_placed_action":{"enabled":false}}}}')
@@ -2574,6 +2573,85 @@ def test_builder_minimal_manifest_add_actions_and_sign_no_auto_add(self):
         load_settings('{"builder":{"actions":{"auto_opened_action":{"enabled":true}}}}')
         load_settings('{"builder":{"actions":{"auto_created_action":{"enabled":true}}}}')
 
+    def test_builder_minimal_manifest_add_actions_and_sign_with_auto_add(self):
+        # For testing, remove auto-added actions
+        load_settings('{"builder":{"actions":{"auto_placed_action":{"enabled":true}}}}')
+        load_settings('{"builder":{"actions":{"auto_opened_action":{"enabled":true}}}}')
+        load_settings('{"builder":{"actions":{"auto_created_action":{"enabled":true}}}}')
+
+        initial_manifest_definition = {
+            "claim_generator": "python_test",
+            "claim_generator_info": [{
+                "name": "python_test",
+                "version": "0.0.1",
+            }],
+            "format": "image/jpeg",
+            "title": "Python Test Image V2",
+        }
+
+        builder = Builder.from_json(initial_manifest_definition)
+        action_json = '{"action": "c2pa.color_adjustments", "parameters": {"name": "brightnesscontrast"}}'
+        builder.add_action(action_json)
+
+        with open(self.testPath2, "rb") as file:
+            output = io.BytesIO(bytearray())
+            builder.sign(self.signer, "image/jpeg", file, output)
+            output.seek(0)
+            reader = Reader("image/jpeg", output)
+            json_data = reader.json()
+            manifest_data = json.loads(json_data)
+
+            # Verify active manifest exists
+            self.assertIn("active_manifest", manifest_data)
+            active_manifest_id = manifest_data["active_manifest"]
+
+            # Verify active manifest object exists
+            self.assertIn("manifests", manifest_data)
+            self.assertIn(active_manifest_id, manifest_data["manifests"])
+            active_manifest = manifest_data["manifests"][active_manifest_id]
+
+            # Verify assertions object exists in active manifest
+            self.assertIn("assertions", active_manifest)
+            assertions = active_manifest["assertions"]
+
+            # Find the c2pa.actions.v2 assertion to look for what we added
+            actions_assertion = None
+            for assertion in assertions:
+                if assertion.get("label") == "c2pa.actions.v2":
+                    actions_assertion = assertion
+                    break
+
+            self.assertIsNotNone(actions_assertion)
+            self.assertIn("data", actions_assertion)
+            assertion_data = actions_assertion["data"]
+            # Verify the manifest now contains actions
+            self.assertIn("actions", assertion_data)
+            actions = assertion_data["actions"]
+            # Verify "c2pa.created" action exists anywhere in the actions array
+            created_action_found = False
+            for action in actions:
+                if action.get("action") == "c2pa.created":
+                    created_action_found = True
+                    break
+
+            self.assertTrue(created_action_found)
+
+            # Verify "c2pa.color_adjustments" action also exists in the same actions array
+            color_adjustments_found = False
+            for action in actions:
+                if action.get("action") == "c2pa.color_adjustments":
+                    color_adjustments_found = True
+                    break
+
+            self.assertTrue(color_adjustments_found)
+
+        builder.close()
+
+        # Reset settings
+        load_settings('{"builder":{"actions":{"auto_placed_action":{"enabled":true}}}}')
+        load_settings('{"builder":{"actions":{"auto_opened_action":{"enabled":true}}}}')
+        load_settings('{"builder":{"actions":{"auto_created_action":{"enabled":true}}}}')
+
 
 class TestStream(unittest.TestCase):
     def setUp(self):