fix: Bindings tests

Author: tmathern

tests/test_unit_tests.py

@@ -342,31 +342,6 @@ def test_reader_state_with_invalid_native_pointer(self):
         with self.assertRaises(Error):
             reader.json()
 
-    # TODO: Unskip once fixed configuration to read data is clarified
-    # def test_read_cawg_data_file(self):
-    #     """Test reading C2PA metadata from C_with_CAWG_data.jpg file."""
-    #     file_path = os.path.join(self.data_dir, "C_with_CAWG_data.jpg")
-
-    #     with open(file_path, "rb") as file:
-    #         reader = Reader("image/jpeg", file)
-    #         json_data = reader.json()
-    #         self.assertIsInstance(json_data, str)
-
-    #         # Parse the JSON and verify specific fields
-    #         manifest_data = json.loads(json_data)
-
-    #         # Verify basic manifest structure
-    #         self.assertIn("manifests", manifest_data)
-    #         self.assertIn("active_manifest", manifest_data)
-
-    #         # Get the active manifest
-    #         active_manifest_id = manifest_data["active_manifest"]
-    #         active_manifest = manifest_data["manifests"][active_manifest_id]
-
-    #         # Verify manifest is not null or empty
-    #         assert active_manifest is not None, "Active manifest should not be null"
-    #         assert len(active_manifest) > 0, "Active manifest should not be empty"
-
     def test_reader_is_embedded(self):
         """Test the is_embedded method returns correct values for embedded and remote manifests."""
 
@@ -415,7 +390,6 @@ def test_reader_is_embedded(self):
             ]
         }
 
-        # Test remote manifest
         with open(self.testPath, "rb") as file:
             builder = Builder(manifest_definition)
             builder.set_no_embed()
@@ -431,17 +405,42 @@ def test_reader_is_embedded(self):
     def test_reader_get_remote_url(self):
         """Test the get_remote_url method returns correct values for embedded and remote manifests."""
 
-        # Test embedded manifest (should return None)
+        # Test get_remote_url for file with embedded manifest (should return None)
         with open(self.testPath, "rb") as file:
             reader = Reader("image/jpeg", file)
-            self.assertIsNone(reader.get_remote_url(), "Embedded manifest should return None for remote URL")
+            self.assertIsNone(reader.get_remote_url())
             reader.close()
 
         # Test remote manifest using cloud.jpg fixture which has a remote URL
-        cloud_fixture_path = os.path.join(self.data_dir, "files-for-reading-tests", "cloud.jpg")
+        cloud_fixture_path = os.path.join(self.data_dir, "cloud.jpg")
         with Reader("image/jpeg", cloud_fixture_path) as reader:
             remote_url = reader.get_remote_url()
-            self.assertFalse(reader.is_embedded(), "This should be a remote manifest")
+            self.assertFalse(reader.is_embedded())
+
+    # TODO: Unskip once fixed configuration to read data is clarified
+    # def test_read_cawg_data_file(self):
+    #     """Test reading C2PA metadata from C_with_CAWG_data.jpg file."""
+    #     file_path = os.path.join(self.data_dir, "C_with_CAWG_data.jpg")
+
+    #     with open(file_path, "rb") as file:
+    #         reader = Reader("image/jpeg", file)
+    #         json_data = reader.json()
+    #         self.assertIsInstance(json_data, str)
+
+    #         # Parse the JSON and verify specific fields
+    #         manifest_data = json.loads(json_data)
+
+    #         # Verify basic manifest structure
+    #         self.assertIn("manifests", manifest_data)
+    #         self.assertIn("active_manifest", manifest_data)
+
+    #         # Get the active manifest
+    #         active_manifest_id = manifest_data["active_manifest"]
+    #         active_manifest = manifest_data["manifests"][active_manifest_id]
+
+    #         # Verify manifest is not null or empty
+    #         assert active_manifest is not None, "Active manifest should not be null"
+    #         assert len(active_manifest) > 0, "Active manifest should not be empty"
 
 
 class TestBuilderWithSigner(unittest.TestCase):