audiometa-python 0.6.0__py3-none-any.whl

audiometa/test/tests/integration/writing/test_forced_format.py

@@ -0,0 +1,224 @@
+"""Tests for forced format parameter functionality.
+
+This module tests the behavior when a specific metadata format is forced using the metadata_format parameter in
+update_metadata.
+"""
+
+import pytest
+
+from audiometa import get_unified_metadata, update_metadata
+from audiometa.exceptions import (
+    MetadataFieldNotSupportedByMetadataFormatError,
+    MetadataFormatNotSupportedByAudioFormatError,
+    MetadataWritingConflictParametersError,
+)
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.utils.metadata_format import MetadataFormat
+from audiometa.utils.metadata_writing_strategy import MetadataWritingStrategy
+from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
+
+
+@pytest.mark.integration
+class TestForcedFormat:
+    def test_forced_format_writes_only_to_specified_format(self):
+        # Create WAV file with initial RIFF metadata
+        initial_metadata = {"title": "Original RIFF Title", "artist": "Original RIFF Artist"}
+        with temp_file_with_metadata(initial_metadata, "wav") as test_file:
+            # Add ID3v2 metadata using the library directly
+            id3v2_metadata = {
+                UnifiedMetadataKey.TITLE: "Original ID3v2 Title",
+                UnifiedMetadataKey.ARTISTS: ["Original ID3v2 Artist"],
+            }
+            update_metadata(test_file, id3v2_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify both formats have their respective metadata
+            riff_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert riff_before.get(UnifiedMetadataKey.TITLE) == "Original RIFF Title"
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) == "Original ID3v2 Title"
+
+            # Force ID3v2 format (should write only to ID3v2, leave RIFF untouched)
+            new_metadata = {
+                UnifiedMetadataKey.TITLE: "New ID3v2 Title",
+                UnifiedMetadataKey.ARTISTS: ["New ID3v2 Artist"],
+            }
+
+            # This should write only to ID3v2 format, leaving RIFF unchanged
+            update_metadata(test_file, new_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify ID3v2 has new metadata
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "New ID3v2 Title"
+            assert id3v2_after.get(UnifiedMetadataKey.ARTISTS) == ["New ID3v2 Artist"]
+
+            # Verify RIFF still has original metadata (forced format doesn't affect other formats)
+            riff_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            assert riff_after.get(UnifiedMetadataKey.TITLE) == "Original RIFF Title"
+            assert riff_after.get(UnifiedMetadataKey.ARTISTS) == ["Original RIFF Artist"]
+
+    def test_forced_format_fails_fast_on_unsupported_fields(self):
+        # Create WAV file with basic metadata
+        initial_metadata = {"title": "Test Title", "artist": "Test Artist"}
+        with temp_file_with_metadata(initial_metadata, "wav") as test_file:
+            # Try to write ARCHIVAL_LOCATION to RIFF format (not supported)
+            unsupported_metadata = {
+                UnifiedMetadataKey.TITLE: "Test Title",
+                UnifiedMetadataKey.ARCHIVAL_LOCATION: "some location",  # ARCHIVAL_LOCATION not supported by RIFF format
+            }
+
+            # This should raise MetadataFieldNotSupportedByMetadataFormatError because format is forced
+            with pytest.raises(
+                MetadataFieldNotSupportedByMetadataFormatError,
+                match="UnifiedMetadataKey.ARCHIVAL_LOCATION metadata not supported by RIFF format",
+            ):
+                update_metadata(test_file, unsupported_metadata, metadata_format=MetadataFormat.RIFF)
+
+    def test_forced_format_succeeds_with_supported_fields(self):
+        # Create WAV file with basic metadata
+        initial_metadata = {"title": "Test Title", "artist": "Test Artist"}
+        with temp_file_with_metadata(initial_metadata, "wav") as test_file:
+            # Write supported fields to RIFF format
+            supported_metadata = {
+                UnifiedMetadataKey.TITLE: "New RIFF Title",
+                UnifiedMetadataKey.ARTISTS: ["New RIFF Artist"],
+                UnifiedMetadataKey.ALBUM: "New RIFF Album",
+            }
+
+            # This should succeed because all fields are supported by RIFF
+            update_metadata(test_file, supported_metadata, metadata_format=MetadataFormat.RIFF)
+
+            # Verify RIFF has new metadata
+            riff_metadata = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            assert riff_metadata.get(UnifiedMetadataKey.TITLE) == "New RIFF Title"
+            assert riff_metadata.get(UnifiedMetadataKey.ARTISTS) == ["New RIFF Artist"]
+            assert riff_metadata.get(UnifiedMetadataKey.ALBUM) == "New RIFF Album"
+
+    def test_forced_format_parameter_conflict_error(self):
+        # Create WAV file with basic metadata
+        initial_metadata = {"title": "Test Title", "artist": "Test Artist"}
+        with temp_file_with_metadata(initial_metadata, "wav") as test_file:
+            metadata = {UnifiedMetadataKey.TITLE: "Test Title", UnifiedMetadataKey.ARTISTS: ["Test Artist"]}
+
+            # This should raise error because both parameters are specified
+            with pytest.raises(
+                MetadataWritingConflictParametersError,
+                match="Cannot specify both metadata_strategy and metadata_format",
+            ):
+                update_metadata(
+                    test_file,
+                    metadata,
+                    metadata_format=MetadataFormat.RIFF,
+                    metadata_strategy=MetadataWritingStrategy.SYNC,
+                )
+
+    def test_forced_format_with_unsupported_file_type(self):
+        # Create MP3 file
+        initial_metadata = {"title": "Test Title", "artist": "Test Artist"}
+        with temp_file_with_metadata(initial_metadata, "mp3") as test_file:
+            metadata = {UnifiedMetadataKey.TITLE: "Test Title", UnifiedMetadataKey.ARTISTS: ["Test Artist"]}
+
+            # Try to force Vorbis format on MP3 file (not supported)
+            with pytest.raises(
+                MetadataFormatNotSupportedByAudioFormatError,
+                match="Tag format MetadataFormat.VORBIS not supported for file extension .mp3",
+            ):
+                update_metadata(test_file, metadata, metadata_format=MetadataFormat.VORBIS)
+
+    def test_forced_format_id3v1_writing_support(self):
+        # Create MP3 file with basic metadata
+        initial_metadata = {"title": "Test Title", "artist": "Test Artist"}
+        with temp_file_with_metadata(initial_metadata, "mp3") as test_file:
+            metadata = {UnifiedMetadataKey.TITLE: "New ID3v1 Title", UnifiedMetadataKey.ARTISTS: ["New ID3v1 Artist"]}
+
+            update_metadata(test_file, metadata, metadata_format=MetadataFormat.ID3V1)
+
+            # Verify the metadata was written
+            from audiometa import get_unified_metadata
+
+            result = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert result.get(UnifiedMetadataKey.TITLE) == "New ID3v1 Title"
+            assert result.get(UnifiedMetadataKey.ARTISTS) == ["New ID3v1 Artist"]
+
+    def test_forced_format_multiple_formats_present(self):
+        # Create WAV file with both RIFF and ID3v2 metadata
+        initial_metadata = {"title": "Original RIFF Title", "artist": "Original RIFF Artist"}
+        with temp_file_with_metadata(initial_metadata, "wav") as test_file:
+            # Add ID3v2 metadata using the library directly
+            id3v2_metadata = {
+                UnifiedMetadataKey.TITLE: "Original ID3v2 Title",
+                UnifiedMetadataKey.ARTISTS: ["Original ID3v2 Artist"],
+            }
+            update_metadata(test_file, id3v2_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify both formats have original metadata
+            riff_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert riff_before.get(UnifiedMetadataKey.TITLE) == "Original RIFF Title"
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) == "Original ID3v2 Title"
+
+            # Force ID3v2 format - should only update ID3v2
+            new_metadata = {
+                UnifiedMetadataKey.TITLE: "New ID3v2 Title",
+                UnifiedMetadataKey.ARTISTS: ["New ID3v2 Artist"],
+            }
+
+            update_metadata(test_file, new_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify ID3v2 was updated
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "New ID3v2 Title"
+            assert id3v2_after.get(UnifiedMetadataKey.ARTISTS) == ["New ID3v2 Artist"]
+
+            # Verify RIFF was NOT updated (forced format only affects specified format)
+            riff_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            assert riff_after.get(UnifiedMetadataKey.TITLE) == "Original RIFF Title"
+            assert riff_after.get(UnifiedMetadataKey.ARTISTS) == ["Original RIFF Artist"]
+
+    def test_forced_format_with_rating_field(self):
+        # Create MP3 file with basic metadata
+        initial_metadata = {"title": "Test Title", "artist": "Test Artist"}
+        with temp_file_with_metadata(initial_metadata, "mp3") as test_file:
+            # Write rating to ID3v2 format with proper configuration
+            metadata_with_rating = {UnifiedMetadataKey.TITLE: "Test Title", UnifiedMetadataKey.RATING: 80}
+
+            # This should succeed with proper configuration
+            update_metadata(
+                test_file,
+                metadata_with_rating,
+                metadata_format=MetadataFormat.ID3V2,
+                normalized_rating_max_value=100,
+            )
+
+            # Verify rating was written (converted from 0-100 to 0-255 scale)
+            id3v2_metadata = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v2_metadata.get(UnifiedMetadataKey.TITLE) == "Test Title"
+            # Rating 85 on 0-100 scale becomes 196 on 0-255 scale (85 * 255 / 100 = 216.75, rounded to 196)
+            assert id3v2_metadata.get(UnifiedMetadataKey.RATING) == 196
+
+    # Note: delete_all_metadata tests have been moved to test_delete_all_metadata.py
+
+    def test_forced_format_validation_before_writing(self):
+        # Create WAV file with initial metadata
+        initial_metadata = {"title": "Original Title", "artist": "Original Artist"}
+        with temp_file_with_metadata(initial_metadata, "wav") as test_file:
+            # Try to write mixed supported/unsupported fields to RIFF format
+            mixed_metadata = {
+                UnifiedMetadataKey.TITLE: "New Title",  # Supported
+                UnifiedMetadataKey.ARTISTS: ["New Artist"],  # Supported
+                UnifiedMetadataKey.ARCHIVAL_LOCATION: "some location",  # NOT supported by RIFF
+                UnifiedMetadataKey.ALBUM: "New Album",  # Supported
+            }
+
+            # This should fail because ARCHIVAL_LOCATION is not supported by RIFF
+            with pytest.raises(
+                MetadataFieldNotSupportedByMetadataFormatError,
+                match="UnifiedMetadataKey.ARCHIVAL_LOCATION metadata not supported by RIFF format",
+            ):
+                update_metadata(test_file, mixed_metadata, metadata_format=MetadataFormat.RIFF)
+
+            # Verify NO changes were made to the file (validation happens before writing)
+            final_metadata = get_unified_metadata(test_file)
+            assert final_metadata.get(UnifiedMetadataKey.TITLE) == "Original Title"
+            assert final_metadata.get(UnifiedMetadataKey.ARTISTS) == ["Original Artist"]
+            assert final_metadata.get(UnifiedMetadataKey.BPM) is None
+            assert final_metadata.get(UnifiedMetadataKey.ALBUM) is None

audiometa/test/tests/integration/writing/test_multiple_format_preservation.py

@@ -0,0 +1,223 @@
+"""Tests for multiple format preservation when updating metadata.
+
+This module tests that updating fields in one format with the PRESERVE strategy does not affect other formats. It covers
+the main working combinations.
+"""
+
+import pytest
+
+from audiometa import get_unified_metadata, update_metadata
+from audiometa.test.helpers.id3v1 import ID3v1MetadataSetter
+from audiometa.test.helpers.id3v2 import ID3v2MetadataSetter
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.utils.metadata_format import MetadataFormat
+from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
+
+
+@pytest.mark.integration
+class TestMultipleFormatPreservation:
+    def test_id3v1_preserves_id3v2(self):
+        with temp_file_with_metadata({"title": "ID3v2 Title", "artist": "ID3v2 Artist"}, "mp3") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"title": "ID3v2 Title"})
+            ID3v2MetadataSetter.set_metadata(test_file, {"artist": "ID3v2 Artist"})
+
+            # Verify initial state
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            id3v1_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+            assert id3v1_before.get(UnifiedMetadataKey.TITLE) is None
+
+            # Update ID3v1 with PRESERVE strategy (specify ID3v1 format)
+            id3v1_metadata = {UnifiedMetadataKey.TITLE: "ID3v1 Title", UnifiedMetadataKey.ARTISTS: ["ID3v1 Artist"]}
+            update_metadata(test_file, id3v1_metadata, metadata_format=MetadataFormat.ID3V1)
+
+            # Verify ID3v1 was updated and ID3v2 was preserved
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            id3v1_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+            assert id3v1_after.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+
+    def test_id3v2_preserves_id3v1(self):
+        with temp_file_with_metadata({"title": "ID3v1 Title", "artist": "ID3v1 Artist"}, "id3v1") as test_file:
+            # Verify initial state
+            id3v1_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v1_before.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) is None
+
+            id3v2_metadata = {UnifiedMetadataKey.TITLE: "ID3v2 Title", UnifiedMetadataKey.ARTISTS: ["ID3v2 Artist"]}
+            update_metadata(test_file, id3v2_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify ID3v2 was updated and ID3v1 was preserved
+            id3v1_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v1_after.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+
+    def test_riff_preserves_id3v1(self):
+        with temp_file_with_metadata({}, "wav") as test_file:
+            ID3v1MetadataSetter.set_title(test_file, "ID3v1 Title")
+            ID3v1MetadataSetter.set_artist(test_file, "ID3v1 Artist")
+
+            # Verify initial state
+            id3v1_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            riff_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            assert id3v1_before.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+            assert riff_before.get(UnifiedMetadataKey.TITLE) is None
+
+            riff_metadata = {UnifiedMetadataKey.TITLE: "RIFF Title", UnifiedMetadataKey.ARTISTS: ["RIFF Artist"]}
+            update_metadata(test_file, riff_metadata, metadata_format=MetadataFormat.RIFF)
+
+            # Verify RIFF was updated and ID3v1 was preserved
+            id3v1_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            riff_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            assert id3v1_after.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+            assert riff_after.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+
+    def test_id3v1_preserves_riff(self):
+        with temp_file_with_metadata({"title": "RIFF Title", "artist": "RIFF Artist"}, "wav") as test_file:
+            # Verify initial state
+            riff_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            id3v1_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert riff_before.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+            assert id3v1_before.get(UnifiedMetadataKey.TITLE) is None
+
+            # Update ID3v1 with PRESERVE strategy (specify ID3v1 format)
+            id3v1_metadata = {UnifiedMetadataKey.TITLE: "ID3v1 Title", UnifiedMetadataKey.ARTISTS: ["ID3v1 Artist"]}
+            update_metadata(test_file, id3v1_metadata, metadata_format=MetadataFormat.ID3V1)
+
+            # Verify ID3v1 was updated and RIFF was preserved
+            riff_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            id3v1_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert riff_after.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+            assert id3v1_after.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+
+    def test_riff_preserves_id3v2(self):
+        with temp_file_with_metadata({}, "wav") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"title": "ID3v2 Title"})
+            ID3v2MetadataSetter.set_metadata(test_file, {"artist": "ID3v2 Artist"})
+
+            # Verify initial state
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            riff_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+            assert riff_before.get(UnifiedMetadataKey.TITLE) is None
+
+            # Update RIFF with PRESERVE strategy (specify RIFF format)
+            riff_metadata = {UnifiedMetadataKey.TITLE: "RIFF Title", UnifiedMetadataKey.ARTISTS: ["RIFF Artist"]}
+            update_metadata(test_file, riff_metadata, metadata_format=MetadataFormat.RIFF)
+
+            # Verify RIFF was updated and ID3v2 was preserved
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            riff_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+            assert riff_after.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+
+    def test_id3v2_preserves_riff(self):
+        with temp_file_with_metadata({"title": "RIFF Title", "artist": "RIFF Artist"}, "wav") as test_file:
+            # Verify initial state
+            riff_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert riff_before.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) is None
+
+            # Update ID3v2 with PRESERVE strategy (specify ID3v2 format)
+            id3v2_metadata = {UnifiedMetadataKey.TITLE: "ID3v2 Title", UnifiedMetadataKey.ARTISTS: ["ID3v2 Artist"]}
+            update_metadata(test_file, id3v2_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify ID3v2 was updated and RIFF was preserved
+            riff_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert riff_after.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+
+    def test_vorbis_preserves_id3v2(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"title": "ID3v2 Title"})
+            ID3v2MetadataSetter.set_metadata(test_file, {"artist": "ID3v2 Artist"})
+
+            # Verify initial state
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            vorbis_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.VORBIS)
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+            assert vorbis_before.get(UnifiedMetadataKey.TITLE) is None
+
+            # Update Vorbis with PRESERVE strategy (specify Vorbis format)
+            vorbis_metadata = {UnifiedMetadataKey.TITLE: "Vorbis Title", UnifiedMetadataKey.ARTISTS: ["Vorbis Artist"]}
+            update_metadata(test_file, vorbis_metadata, metadata_format=MetadataFormat.VORBIS)
+
+            # Verify Vorbis was updated and ID3v2 was preserved
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            vorbis_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.VORBIS)
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+            assert vorbis_after.get(UnifiedMetadataKey.TITLE) == "Vorbis Title"
+
+    def test_id3v2_preserves_vorbis(self):
+        with temp_file_with_metadata({"title": "Vorbis Title", "artist": "Vorbis Artist"}, "flac") as test_file:
+            # Verify initial state
+            vorbis_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.VORBIS)
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert vorbis_before.get(UnifiedMetadataKey.TITLE) == "Vorbis Title"
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) is None
+
+            # Update ID3v2 with PRESERVE strategy (specify ID3v2 format)
+            id3v2_metadata = {UnifiedMetadataKey.TITLE: "ID3v2 Title", UnifiedMetadataKey.ARTISTS: ["ID3v2 Artist"]}
+            update_metadata(test_file, id3v2_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify ID3v2 was updated and Vorbis was preserved
+            vorbis_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.VORBIS)
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert vorbis_after.get(UnifiedMetadataKey.TITLE) == "Vorbis Title"
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+
+    def test_multiple_fields_preservation(self):
+        with temp_file_with_metadata({"title": "Original Title", "artist": "Original Artist"}, "mp3") as test_file:
+            ID3v1MetadataSetter.set_title(test_file, "ID3v1 Title")
+            ID3v1MetadataSetter.set_artist(test_file, "ID3v1 Artist")
+            # Use ID3v2.3 instead of ID3v2.4 to preserve ID3v1 metadata
+            ID3v2MetadataSetter.set_metadata(test_file, {"title": "ID3v2 Title"}, version="2.3")
+            ID3v2MetadataSetter.set_metadata(test_file, {"artist": "ID3v2 Artist"}, version="2.3")
+
+            # Verify initial state
+            id3v1_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            id3v2_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v1_before.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+            assert id3v2_before.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+
+            # Update only ID3v2 with multiple fields
+            id3v2_metadata = {
+                UnifiedMetadataKey.TITLE: "Updated ID3v2 Title",
+                UnifiedMetadataKey.ARTISTS: ["Updated ID3v2 Artist"],
+                UnifiedMetadataKey.ALBUM: "ID3v2 Album",
+            }
+            update_metadata(test_file, id3v2_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify ID3v2 was updated and ID3v1 was preserved
+            id3v1_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v1_after.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+            assert id3v1_after.get(UnifiedMetadataKey.ARTISTS) == ["ID3v1 Artist"]
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "Updated ID3v2 Title"
+            assert id3v2_after.get(UnifiedMetadataKey.ARTISTS) == ["Updated ID3v2 Artist"]
+            assert id3v2_after.get(UnifiedMetadataKey.ALBUM) == "ID3v2 Album"
+
+    def test_preserve_strategy_with_none_values(self):
+        with temp_file_with_metadata({"title": "Original Title", "artist": "Original Artist"}, "mp3") as test_file:
+            ID3v1MetadataSetter.set_title(test_file, "ID3v1 Title")
+            ID3v1MetadataSetter.set_artist(test_file, "ID3v1 Artist")
+
+            # Verify initial state
+            id3v1_before = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert id3v1_before.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+
+            # Update ID3v2 with None values (should not affect ID3v1)
+            id3v2_metadata = {
+                UnifiedMetadataKey.TITLE: None,  # This should not affect ID3v1
+                UnifiedMetadataKey.ARTISTS: ["ID3v2 Artist"],
+            }
+            update_metadata(test_file, id3v2_metadata, metadata_format=MetadataFormat.ID3V2)
+
+            # Verify ID3v1 was preserved
+            id3v1_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert id3v1_after.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+            assert id3v1_after.get(UnifiedMetadataKey.ARTISTS) == ["ID3v1 Artist"]

audiometa/test/tests/integration/writing/test_partial_update.py

@@ -0,0 +1,36 @@
+"""Tests for general metadata writing functionality using external scripts.
+
+This refactored version uses external scripts to set up test data instead of the app's update functions, preventing
+circular dependencies.
+"""
+
+import pytest
+
+from audiometa import get_unified_metadata, update_metadata
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
+
+
+@pytest.mark.integration
+class TestMetadataWriting:
+    # Note: delete_all_metadata tests have been moved to test_delete_all_metadata.py
+
+    def test_write_metadata_partial_update(self):
+        # Use external script to set initial metadata
+        initial_metadata = {"title": "Original Title", "artist": "Original Artist", "album": "Original Album"}
+
+        with temp_file_with_metadata(initial_metadata, "mp3") as test_file:
+            # Get original metadata
+            original_metadata = get_unified_metadata(test_file)
+            original_album = original_metadata.get(UnifiedMetadataKey.ALBUM)
+
+            # Update only title using app's function (this is what we're testing)
+            test_metadata = {UnifiedMetadataKey.TITLE: "Partial Update Title"}
+
+            update_metadata(test_file, test_metadata)
+            updated_metadata = get_unified_metadata(test_file)
+
+            # Title should be updated
+            assert updated_metadata.get(UnifiedMetadataKey.TITLE) == "Partial Update Title"
+            # Other fields should remain unchanged
+            assert updated_metadata.get(UnifiedMetadataKey.ALBUM) == original_album

audiometa/test/tests/integration/writing/writing_strategies/test_cleanup_strategy.py

@@ -0,0 +1,79 @@
+"""Tests for CLEANUP metadata writing strategy.
+
+This module tests the CLEANUP strategy which writes to native format and removes all non-native metadata formats.
+"""
+
+import pytest
+
+from audiometa import get_unified_metadata, update_metadata
+from audiometa.test.helpers.id3v1.id3v1_metadata_setter import ID3v1MetadataSetter
+from audiometa.test.helpers.id3v2.id3v2_metadata_setter import ID3v2MetadataSetter
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.utils.metadata_format import MetadataFormat
+from audiometa.utils.metadata_writing_strategy import MetadataWritingStrategy
+from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
+
+
+@pytest.mark.integration
+class TestCleanupStrategy:
+    def test_cleanup_strategy_wav(self):
+        with temp_file_with_metadata(
+            {"title": "Basic Title", "artist": "Basic Artist", "album": "Basic Album"}, "wav"
+        ) as test_file:
+            ID3v2MetadataSetter.set_metadata(
+                test_file,
+                {"title": "ID3v2 Title", "artist": "ID3v2 Artist", "album": "ID3v2 Album"},
+                version="2.3",
+            )
+
+            metadata = {
+                UnifiedMetadataKey.TITLE: "RIFF Title",
+                UnifiedMetadataKey.ARTISTS: ["RIFF Artist"],
+                UnifiedMetadataKey.ALBUM: "RIFF Album",
+            }
+            update_metadata(test_file, metadata, metadata_strategy=MetadataWritingStrategy.CLEANUP)
+
+            # Verify ID3v2 was removed
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) is None
+
+            # Verify RIFF has new metadata
+            riff_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.RIFF)
+            assert riff_after.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+
+            # Merged metadata should only have RIFF (ID3v2 was cleaned up)
+            merged = get_unified_metadata(test_file)
+            assert merged.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+
+    def test_cleanup_strategy_mp3_id3v1_not_preserved(self):
+        # Create test file with ID3v1 metadata using external script
+        with temp_file_with_metadata({}, "mp3") as test_file:
+            # Add ID3v1 metadata using temp_file_with_metadata
+            ID3v1MetadataSetter.set_metadata(
+                test_file, {"title": "ID3v1 Title", "artist": "ID3v1 Artist", "album": "ID3v1 Album"}
+            )
+
+            # Verify ID3v1 metadata was written
+            id3v1_result = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert id3v1_result.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+
+            # Now write ID3v2 metadata with CLEANUP strategy
+            id3v2_metadata = {
+                UnifiedMetadataKey.TITLE: "ID3v2 Title",
+                UnifiedMetadataKey.ARTISTS: ["ID3v2 Artist"],
+                UnifiedMetadataKey.ALBUM: "ID3v2 Album",
+            }
+            update_metadata(test_file, id3v2_metadata, metadata_strategy=MetadataWritingStrategy.CLEANUP)
+
+            # Verify ID3v1 metadata behavior with CLEANUP strategy
+            # ID3v1 should be removed (cleaned up) with CLEANUP strategy
+            id3v1_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert id3v1_after.get(UnifiedMetadataKey.TITLE) is None  # ID3v1 was cleaned up
+
+            # Verify ID3v2 metadata was written
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+
+            # Merged metadata should prefer ID3v2 (higher precedence)
+            merged = get_unified_metadata(test_file)
+            assert merged.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"

audiometa/test/tests/integration/writing/writing_strategies/test_preserve_strategy.py

@@ -0,0 +1,76 @@
+"""Tests for PRESERVE metadata writing strategy.
+
+This module tests the PRESERVE strategy which writes to native format only and preserves existing metadata in other
+formats.
+"""
+
+import pytest
+
+from audiometa import get_unified_metadata, update_metadata
+from audiometa.test.helpers.id3v1.id3v1_metadata_setter import ID3v1MetadataSetter
+from audiometa.test.helpers.id3v2.id3v2_metadata_setter import ID3v2MetadataSetter
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.utils.metadata_format import MetadataFormat
+from audiometa.utils.metadata_writing_strategy import MetadataWritingStrategy
+from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
+
+
+@pytest.mark.integration
+class TestPreserveStrategy:
+    def test_preserve_strategy_wav(self):
+        with temp_file_with_metadata(
+            {"title": "ID3v2 Title", "artist": "ID3v2 Artist", "album": "ID3v2 Album"}, "wav"
+        ) as test_file:
+            ID3v2MetadataSetter.set_metadata(
+                test_file,
+                {"title": "ID3v2 Title", "artist": "ID3v2 Artist", "album": "ID3v2 Album"},
+                version="2.3",
+            )
+
+            # Verify ID3v2 metadata was written
+            id3v2_result = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v2_result.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+
+            # Now write RIFF metadata with PRESERVE strategy (default)
+            # This part still uses the app's function since we're testing the strategy
+            riff_metadata = {
+                UnifiedMetadataKey.TITLE: "RIFF Title",
+                UnifiedMetadataKey.ARTISTS: ["RIFF Artist"],
+                UnifiedMetadataKey.ALBUM: "RIFF Album",
+            }
+            update_metadata(test_file, riff_metadata, metadata_strategy=MetadataWritingStrategy.PRESERVE)
+
+            # Merged metadata should prefer RIFF (WAV native format has higher precedence)
+            merged = get_unified_metadata(test_file)
+            assert merged.get(UnifiedMetadataKey.TITLE) == "RIFF Title"
+
+    def test_preserve_strategy_mp3_id3v1_not_preserved(self):
+        with temp_file_with_metadata({}, "mp3") as test_file:
+            ID3v1MetadataSetter.set_metadata(
+                test_file, {"title": "ID3v1 Title", "artist": "ID3v1 Artist", "album": "ID3v1 Album"}
+            )
+
+            # Verify ID3v1 metadata was written
+            id3v1_result = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert id3v1_result.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"
+
+            # Now write ID3v2 metadata with PRESERVE strategy
+            metadata = {
+                UnifiedMetadataKey.TITLE: "ID3v2 Title",
+                UnifiedMetadataKey.ARTISTS: ["ID3v2 Artist"],
+                UnifiedMetadataKey.ALBUM: "ID3v2 Album",
+            }
+            update_metadata(test_file, metadata, metadata_strategy=MetadataWritingStrategy.PRESERVE)
+
+            # Verify ID3v1 metadata behavior with PRESERVE strategy
+            # ID3v1 should be preserved (not overwritten) with PRESERVE strategy
+            id3v1_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V1)
+            assert id3v1_after.get(UnifiedMetadataKey.TITLE) == "ID3v1 Title"  # ID3v1 was preserved
+
+            # Verify ID3v2 metadata was written
+            id3v2_after = get_unified_metadata(test_file, metadata_format=MetadataFormat.ID3V2)
+            assert id3v2_after.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"
+
+            # Merged metadata should prefer ID3v2 (higher precedence)
+            merged = get_unified_metadata(test_file)
+            assert merged.get(UnifiedMetadataKey.TITLE) == "ID3v2 Title"