audiometa-python 0.7.1__tar.gz → 0.8.1__tar.gz

{audiometa_python-0.7.1/audiometa_python.egg-info → audiometa_python-0.8.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: audiometa-python
-Version: 0.7.1
+Version: 0.8.1
 Summary: A comprehensive Python library for reading and writing audio metadata across multiple formats
 Author: AudioMeta Python Contributors
 Author-email: Andreas Garcia <garcia.andreas.1991@gmail.com>
@@ -41,29 +41,29 @@ Requires-Dist: PyYAML==6.0; extra == "yaml"
 Dynamic: license-file
 
 <div align="center">
-  <img src="https://raw.githubusercontent.com/Andreas-Garcia/audiometa/main/assets/logo.png" alt="AudioMeta Logo" width="200"/>
+  <img src="https://raw.githubusercontent.com/BehindTheMusicTree/audiometa/main/assets/logo.png" alt="AudioMeta Logo" width="200"/>
 </div>
 
 # AudioMeta Python
 
-[![CI](https://github.com/Andreas-Garcia/audiometa/actions/workflows/ci.yml/badge.svg)](https://github.com/Andreas-Garcia/audiometa/actions/workflows/ci.yml)
+[![CI](https://github.com/BehindTheMusicTree/audiometa/actions/workflows/ci.yml/badge.svg)](https://github.com/BehindTheMusicTree/audiometa/actions/workflows/ci.yml)
 [![Python](https://img.shields.io/badge/python-3.12%20%7C%203.13%20%7C%203.14-blue)](https://www.python.org/)
 [![License](https://img.shields.io/badge/license-Apache%202.0-green.svg)](LICENSE)
 [![PyPI version](https://img.shields.io/pypi/v/audiometa-python)](https://pypi.org/project/audiometa-python/)
 [![Downloads](https://img.shields.io/pepy/dt/audiometa-python)](https://pepy.tech/project/audiometa-python)
-[![GitHub stars](https://img.shields.io/github/stars/Andreas-Garcia/audiometa?style=social)](https://github.com/Andreas-Garcia/audiometa/stargazers)
+[![GitHub stars](https://img.shields.io/github/stars/BehindTheMusicTree/audiometa?style=social)](https://github.com/BehindTheMusicTree/audiometa/stargazers)
 
 A powerful, unified Python library for reading and writing audio metadata across multiple formats. AudioMeta supports MP3, FLAC, and WAV audio files, working seamlessly with ID3v1, ID3v2, Vorbis, and RIFF metadata formats through a single, consistent API.
 
-**Author**: [Andreas Garcia](https://github.com/Andreas-Garcia)
+**Author**: [Andreas Garcia](https://github.com/BehindTheMusicTree)
 
 ## ⭐ Show Your Support
 
 If you find AudioMeta Python useful, please consider:
 
 - ⭐ **Starring this repository** - It helps others discover the project
-- 🐛 **Reporting bugs** - Help improve the library by [opening an issue](https://github.com/Andreas-Garcia/audiometa/issues)
-- 💡 **Suggesting features** - Share your ideas via [GitHub Discussions](https://github.com/Andreas-Garcia/audiometa/discussions) or [feature requests](https://github.com/Andreas-Garcia/audiometa/issues)
+- 🐛 **Reporting bugs** - Help improve the library by [opening an issue](https://github.com/BehindTheMusicTree/audiometa/issues)
+- 💡 **Suggesting features** - Share your ideas via [GitHub Discussions](https://github.com/BehindTheMusicTree/audiometa/discussions) or [feature requests](https://github.com/BehindTheMusicTree/audiometa/issues)
 - 🤝 **Contributing** - See [CONTRIBUTING.md](CONTRIBUTING.md) for ways to help
 - 📢 **Sharing** - Tell others about AudioMeta Python
 
@@ -111,7 +111,7 @@ Your support helps make this project better for everyone! 🎵
 - [📝 Changelog](#-changelog)
 - [🤝 Contributing](#-contributing)
 - [Code of Conduct](CODE_OF_CONDUCT.md)
-- [Security Policy](https://github.com/Andreas-Garcia/audiometa/security/policy)
+- [Security Policy](https://github.com/BehindTheMusicTree/audiometa/security/policy)
 - [License](#license)
 
 ## ✨ Features
@@ -1561,11 +1561,11 @@ Whether you're fixing bugs, adding features, improving documentation, or sharing
 
 **Ways to contribute:**
 
-- 🐛 **Report bugs** - Use the [bug report template](https://github.com/Andreas-Garcia/audiometa/issues/new?template=bug_report.yml)
-- 💡 **Suggest features** - Use the [feature request template](https://github.com/Andreas-Garcia/audiometa/issues/new?template=feature_request.yml)
+- 🐛 **Report bugs** - Use the [bug report template](https://github.com/BehindTheMusicTree/audiometa/issues/new?template=bug_report.yml)
+- 💡 **Suggest features** - Use the [feature request template](https://github.com/BehindTheMusicTree/audiometa/issues/new?template=feature_request.yml)
 - 🔧 **Submit pull requests** - See [CONTRIBUTING.md](CONTRIBUTING.md) for guidelines
 - 📝 **Improve documentation** - Fix typos, clarify explanations, add examples
-- 💬 **Join discussions** - Share ideas and help others in [GitHub Discussions](https://github.com/Andreas-Garcia/audiometa/discussions)
+- 💬 **Join discussions** - Share ideas and help others in [GitHub Discussions](https://github.com/BehindTheMusicTree/audiometa/discussions)
 - ⭐ **Star the repo** - Help others discover the project
 
 For detailed contribution guidelines, see [CONTRIBUTING.md](CONTRIBUTING.md).

{audiometa_python-0.7.1 → audiometa_python-0.8.1}/README.md

@@ -1,27 +1,27 @@
 <div align="center">
-  <img src="https://raw.githubusercontent.com/Andreas-Garcia/audiometa/main/assets/logo.png" alt="AudioMeta Logo" width="200"/>
+  <img src="https://raw.githubusercontent.com/BehindTheMusicTree/audiometa/main/assets/logo.png" alt="AudioMeta Logo" width="200"/>
 </div>
 
 # AudioMeta Python
 
-[![CI](https://github.com/Andreas-Garcia/audiometa/actions/workflows/ci.yml/badge.svg)](https://github.com/Andreas-Garcia/audiometa/actions/workflows/ci.yml)
+[![CI](https://github.com/BehindTheMusicTree/audiometa/actions/workflows/ci.yml/badge.svg)](https://github.com/BehindTheMusicTree/audiometa/actions/workflows/ci.yml)
 [![Python](https://img.shields.io/badge/python-3.12%20%7C%203.13%20%7C%203.14-blue)](https://www.python.org/)
 [![License](https://img.shields.io/badge/license-Apache%202.0-green.svg)](LICENSE)
 [![PyPI version](https://img.shields.io/pypi/v/audiometa-python)](https://pypi.org/project/audiometa-python/)
 [![Downloads](https://img.shields.io/pepy/dt/audiometa-python)](https://pepy.tech/project/audiometa-python)
-[![GitHub stars](https://img.shields.io/github/stars/Andreas-Garcia/audiometa?style=social)](https://github.com/Andreas-Garcia/audiometa/stargazers)
+[![GitHub stars](https://img.shields.io/github/stars/BehindTheMusicTree/audiometa?style=social)](https://github.com/BehindTheMusicTree/audiometa/stargazers)
 
 A powerful, unified Python library for reading and writing audio metadata across multiple formats. AudioMeta supports MP3, FLAC, and WAV audio files, working seamlessly with ID3v1, ID3v2, Vorbis, and RIFF metadata formats through a single, consistent API.
 
-**Author**: [Andreas Garcia](https://github.com/Andreas-Garcia)
+**Author**: [Andreas Garcia](https://github.com/BehindTheMusicTree)
 
 ## ⭐ Show Your Support
 
 If you find AudioMeta Python useful, please consider:
 
 - ⭐ **Starring this repository** - It helps others discover the project
-- 🐛 **Reporting bugs** - Help improve the library by [opening an issue](https://github.com/Andreas-Garcia/audiometa/issues)
-- 💡 **Suggesting features** - Share your ideas via [GitHub Discussions](https://github.com/Andreas-Garcia/audiometa/discussions) or [feature requests](https://github.com/Andreas-Garcia/audiometa/issues)
+- 🐛 **Reporting bugs** - Help improve the library by [opening an issue](https://github.com/BehindTheMusicTree/audiometa/issues)
+- 💡 **Suggesting features** - Share your ideas via [GitHub Discussions](https://github.com/BehindTheMusicTree/audiometa/discussions) or [feature requests](https://github.com/BehindTheMusicTree/audiometa/issues)
 - 🤝 **Contributing** - See [CONTRIBUTING.md](CONTRIBUTING.md) for ways to help
 - 📢 **Sharing** - Tell others about AudioMeta Python
 
@@ -69,7 +69,7 @@ Your support helps make this project better for everyone! 🎵
 - [📝 Changelog](#-changelog)
 - [🤝 Contributing](#-contributing)
 - [Code of Conduct](CODE_OF_CONDUCT.md)
-- [Security Policy](https://github.com/Andreas-Garcia/audiometa/security/policy)
+- [Security Policy](https://github.com/BehindTheMusicTree/audiometa/security/policy)
 - [License](#license)
 
 ## ✨ Features
@@ -1519,11 +1519,11 @@ Whether you're fixing bugs, adding features, improving documentation, or sharing
 
 **Ways to contribute:**
 
-- 🐛 **Report bugs** - Use the [bug report template](https://github.com/Andreas-Garcia/audiometa/issues/new?template=bug_report.yml)
-- 💡 **Suggest features** - Use the [feature request template](https://github.com/Andreas-Garcia/audiometa/issues/new?template=feature_request.yml)
+- 🐛 **Report bugs** - Use the [bug report template](https://github.com/BehindTheMusicTree/audiometa/issues/new?template=bug_report.yml)
+- 💡 **Suggest features** - Use the [feature request template](https://github.com/BehindTheMusicTree/audiometa/issues/new?template=feature_request.yml)
 - 🔧 **Submit pull requests** - See [CONTRIBUTING.md](CONTRIBUTING.md) for guidelines
 - 📝 **Improve documentation** - Fix typos, clarify explanations, add examples
-- 💬 **Join discussions** - Share ideas and help others in [GitHub Discussions](https://github.com/Andreas-Garcia/audiometa/discussions)
+- 💬 **Join discussions** - Share ideas and help others in [GitHub Discussions](https://github.com/BehindTheMusicTree/audiometa/discussions)
 - ⭐ **Star the repo** - Help others discover the project
 
 For detailed contribution guidelines, see [CONTRIBUTING.md](CONTRIBUTING.md).

{audiometa_python-0.7.1 → audiometa_python-0.8.1}/audiometa/__init__.py

@@ -553,6 +553,7 @@ def update_metadata(
     metadata_strategy: MetadataWritingStrategy | None = None,
     metadata_format: MetadataFormat | None = None,
     fail_on_unsupported_field: bool = False,
+    warn_on_unsupported_field: bool = True,
 ) -> None:
     """Update metadata in an audio file.
 
@@ -575,6 +576,9 @@ def update_metadata(
             If False (default), unsupported fields are filtered out with individual warnings for each field.
             For SYNC strategy, this applies per-format: unsupported fields are skipped for each format that
             doesn't support them, while still syncing all supported fields.
+        warn_on_unsupported_field: If True (default), issues warnings when unsupported fields are encountered.
+            If False, suppresses warnings about unsupported fields. Automatically set to False when
+            fail_on_unsupported_field is True.
 
     Returns:
         None
@@ -600,8 +604,9 @@ def update_metadata(
 
         When metadata_strategy is used, unsupported metadata fields are handled based on the
         fail_on_unsupported_field parameter: True raises MetadataFieldNotSupportedByMetadataFormatError, False (default)
-        filters unsupported fields and warns about each one individually. For SYNC strategy, unsupported fields
-        are skipped per-format, allowing supported fields to sync while warning about unsupported ones.
+        filters unsupported fields and warns about each one individually (unless warn_on_unsupported_field is False).
+        For SYNC strategy, unsupported fields are skipped per-format, allowing supported fields to sync while warning
+        about unsupported ones.
 
         Data Filtering:
         For list-type metadata fields (e.g., ARTISTS, GENRES), empty strings and None values
@@ -642,6 +647,9 @@ def update_metadata(
         }
         # Results in: ["Artist 1", "Artist 2"] - empty strings and None filtered out
         update_metadata("song.mp3", metadata)
+
+        # Suppress warnings about unsupported fields
+        update_metadata("song.mp3", metadata, warn_on_unsupported_field=False)
     """
     audio_file = _AudioFile(file)
 
@@ -655,6 +663,11 @@ def update_metadata(
         )
         raise MetadataWritingConflictParametersError(msg)
 
+    # Automatically disable warnings when failing on unsupported fields
+    # This provides a more intuitive API where fail takes precedence over warn
+    if fail_on_unsupported_field:
+        warn_on_unsupported_field = False
+
     # Default to SYNC strategy if not specified
     if metadata_strategy is None:
         metadata_strategy = MetadataWritingStrategy.SYNC
@@ -677,6 +690,7 @@ def update_metadata(
         id3v2_version,
         metadata_format,
         fail_on_unsupported_field,
+        warn_on_unsupported_field,
     )
 
 
@@ -688,6 +702,7 @@ def _handle_metadata_strategy(
     id3v2_version: tuple[int, int, int] | None,
     target_format: MetadataFormat | None = None,
     fail_on_unsupported_field: bool = False,
+    warn_on_unsupported_field: bool = True,
 ) -> None:
     """Handle metadata strategy-specific behavior for all strategies."""
 
@@ -743,11 +758,12 @@ def _handle_metadata_strategy(
                 msg = f"Fields not supported by {target_format_actual.value} format: {unsupported_fields}"
                 raise MetadataFieldNotSupportedByMetadataFormatError(msg)
             # Warn about each unsupported field individually
-            for unsupported_field in unsupported_fields:
-                field_warn_msg = (
-                    f"Field {unsupported_field} not supported by {target_format_actual.value} format, skipped"
-                )
-                warnings.warn(field_warn_msg, stacklevel=2)
+            if warn_on_unsupported_field:
+                for unsupported_field in unsupported_fields:
+                    field_warn_msg = (
+                        f"Field {unsupported_field} not supported by {target_format_actual.value} format, skipped"
+                    )
+                    warnings.warn(field_warn_msg, stacklevel=2)
             # Create filtered metadata without unsupported fields
             filtered_metadata = {k: v for k, v in unified_metadata.items() if k not in unsupported_fields}
             unified_metadata = filtered_metadata
@@ -784,11 +800,12 @@ def _handle_metadata_strategy(
                     unsupported_fields.append(field)
             if unsupported_fields:
                 # Warn about each unsupported field individually for target format
-                for unsupported_field in unsupported_fields:
-                    field_warn_msg = (
-                        f"Field {unsupported_field} not supported by {target_format_actual.value} format, skipped"
-                    )
-                    warnings.warn(field_warn_msg, stacklevel=2)
+                if warn_on_unsupported_field:
+                    for unsupported_field in unsupported_fields:
+                        field_warn_msg = (
+                            f"Field {unsupported_field} not supported by {target_format_actual.value} format, skipped"
+                        )
+                        warnings.warn(field_warn_msg, stacklevel=2)
                 # Create filtered metadata without unsupported fields
                 filtered_metadata = {k: v for k, v in unified_metadata.items() if k not in unsupported_fields}
                 unified_metadata = filtered_metadata
@@ -799,8 +816,9 @@ def _handle_metadata_strategy(
             target_manager.update_metadata(unified_metadata)
         except MetadataFieldNotSupportedByMetadataFormatError as e:
             # For SYNC strategy, log warning but continue with other formats
-            format_warn_msg = f"Format {target_format_actual} doesn't support some metadata fields: {e}"
-            warnings.warn(format_warn_msg, stacklevel=2)
+            if warn_on_unsupported_field:
+                format_warn_msg = f"Format {target_format_actual} doesn't support some metadata fields: {e}"
+                warnings.warn(format_warn_msg, stacklevel=2)
         except Exception as e:
             # Re-raise user errors (like InvalidRatingValueError) immediately
             from .exceptions import ConfigurationError, InvalidRatingValueError
@@ -812,6 +830,28 @@ def _handle_metadata_strategy(
         # Then sync all other available formats
         # Note: We need to be careful about the order to avoid conflicts
         for fmt_name, manager in other_managers.items():
+            # Check if this format has existing metadata (for SYNC strategy)
+            has_existing = False
+            if fmt_name == "id3v1":
+                has_existing = audio_file._has_id3v1_tags()
+            elif fmt_name == "id3v2":
+                # Check if file starts with ID3
+                try:
+                    from pathlib import Path
+
+                    with Path(audio_file.file_path).open("rb") as f:
+                        header = f.read(3)
+                        has_existing = header == b"ID3"
+                except Exception:
+                    has_existing = False
+            elif fmt_name == "vorbis":
+                has_existing = audio_file.file_extension == ".flac"  # Vorbis is native for FLAC
+            elif fmt_name == "riff":
+                has_existing = audio_file.file_extension == ".wav"  # RIFF is native for WAV
+
+            if not has_existing:
+                continue
+
             # For non-target formats, filter out unsupported fields and warn about them
             # This allows syncing supported fields even when some fields are not supported
             unsupported_fields = []
@@ -828,9 +868,10 @@ def _handle_metadata_strategy(
             )
 
             # Warn about each unsupported field individually for non-target formats
-            for unsupported_field in unsupported_fields:
-                field_warn_msg = f"Field {unsupported_field} not supported by {fmt_name} format, skipped"
-                warnings.warn(field_warn_msg, stacklevel=2)
+            if warn_on_unsupported_field:
+                for unsupported_field in unsupported_fields:
+                    field_warn_msg = f"Field {unsupported_field} not supported by {fmt_name} format, skipped"
+                    warnings.warn(field_warn_msg, stacklevel=2)
 
             # Try to update with supported fields only
             if format_metadata:  # Only update if there are supported fields
@@ -868,7 +909,8 @@ def _handle_metadata_strategy(
             unsupported_warn_msg = (
                 f"Fields not supported by {target_format_actual.value} format will be skipped: {unsupported_fields}"
             )
-            warnings.warn(unsupported_warn_msg, stacklevel=2)
+            if warn_on_unsupported_field:
+                warnings.warn(unsupported_warn_msg, stacklevel=2)
             # Create filtered metadata without unsupported fields
             filtered_metadata = {k: v for k, v in unified_metadata.items() if k not in unsupported_fields}
             unified_metadata = filtered_metadata

{audiometa_python-0.7.1 → audiometa_python-0.8.1}/audiometa/_audio_file.py

@@ -5,6 +5,7 @@ import json
 import subprocess
 import tempfile
 import types
+import warnings
 from pathlib import Path
 from typing import cast
 
@@ -356,6 +357,16 @@ class _AudioFile:
             msg = "The file is not a FLAC file"
             raise FileTypeNotSupportedError(msg)
 
+        # Warn if ID3v1 tags will be removed during re-encoding
+        if self._has_id3v1_tags():
+            warnings.warn(
+                "ID3v1 tags detected in FLAC file. These tags will be removed during MD5 repair "
+                "as they are non-standard in FLAC format and interfere with integrity validation. "
+                "Consider backing up ID3v1 metadata before repair if you need to preserve it.",
+                UserWarning,
+                stacklevel=3,
+            )
+
         # Create a temporary file to store the corrected FLAC content
         temp_file = tempfile.NamedTemporaryFile(delete=False, suffix=".flac")
         temp_path = temp_file.name

{audiometa_python-0.7.1 → audiometa_python-0.8.1}/audiometa/test/tests/e2e/workflows/test_format_specific_workflows.py

@@ -6,8 +6,18 @@ capabilities and limitations.
 
 import pytest
 
-from audiometa import delete_all_metadata, get_bitrate, get_duration_in_sec, get_unified_metadata, update_metadata
+from audiometa import (
+    FlacMd5State,
+    delete_all_metadata,
+    fix_md5_checking,
+    get_bitrate,
+    get_duration_in_sec,
+    get_unified_metadata,
+    is_flac_md5_valid,
+    update_metadata,
+)
 from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.test.tests.integration.technical_info.flac_md5.conftest import corrupt_md5, ensure_flac_has_md5
 from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
 
 
@@ -97,6 +107,54 @@ class TestFormatSpecificWorkflows:
             assert bitrate > 0
             assert duration > 0
 
+    def test_flac_md5_validity_workflow(self):
+        """E2E test for FLAC MD5 validity checking and correction workflow."""
+        # Create a FLAC file with initial metadata
+        initial_metadata = {
+            "title": "MD5 Test FLAC Title",
+            "artist": "MD5 Test FLAC Artist",
+            "album": "MD5 Test FLAC Album",
+        }
+        with temp_file_with_metadata(initial_metadata, "flac") as test_file:
+            # Ensure the file has a valid MD5 checksum
+            ensure_flac_has_md5(test_file)
+
+            # 1. Verify initial MD5 is valid
+            initial_md5_state = is_flac_md5_valid(test_file)
+            assert initial_md5_state == FlacMd5State.VALID, "Initial FLAC file should have valid MD5"
+
+            # 2. Corrupt the MD5 checksum
+            corrupt_md5(test_file, "random")
+
+            # 3. Verify MD5 is now invalid
+            corrupted_md5_state = is_flac_md5_valid(test_file)
+            assert corrupted_md5_state == FlacMd5State.INVALID, "Corrupted MD5 should be detected as invalid"
+
+            # 4. Correct the MD5 checksum
+            corrected_file_path = fix_md5_checking(test_file)
+
+            # 5. Verify the corrected file has valid MD5
+            corrected_md5_state = is_flac_md5_valid(corrected_file_path)
+            assert corrected_md5_state == FlacMd5State.VALID, "Corrected file should have valid MD5"
+
+            # 6. Update metadata on the corrected file
+            test_metadata = {
+                UnifiedMetadataKey.TITLE: "Updated MD5 Test Title",
+                UnifiedMetadataKey.ARTISTS: ["Updated MD5 Test Artist"],
+                UnifiedMetadataKey.ALBUM: "Updated MD5 Test Album",
+            }
+            update_metadata(corrected_file_path, test_metadata)
+
+            # 7. Verify metadata was updated
+            updated_metadata = get_unified_metadata(corrected_file_path)
+            assert updated_metadata.get(UnifiedMetadataKey.TITLE) == "Updated MD5 Test Title"
+            assert updated_metadata.get(UnifiedMetadataKey.ARTISTS) == ["Updated MD5 Test Artist"]
+            assert updated_metadata.get(UnifiedMetadataKey.ALBUM) == "Updated MD5 Test Album"
+
+            # 8. Verify MD5 is still valid after metadata update
+            final_md5_state = is_flac_md5_valid(corrected_file_path)
+            assert final_md5_state == FlacMd5State.VALID, "MD5 should remain valid after metadata update"
+
     def test_complete_metadata_workflow_wav(self):
         # Use external script to set initial metadata
         initial_metadata = {"title": "Initial WAV Title", "artist": "Initial WAV Artist", "album": "Initial WAV Album"}

audiometa_python-0.8.1/audiometa/test/tests/integration/technical_info/flac_md5/md5_checking/test_invalid_md5.py

@@ -0,0 +1,35 @@
+"""Test invalid MD5 validation."""
+
+import pytest
+
+from audiometa import FlacMd5State, is_flac_md5_valid
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.test.tests.integration.technical_info.flac_md5.conftest import corrupt_md5, ensure_flac_has_md5
+
+
+@pytest.mark.integration
+class TestInvalidMd5:
+    def test_is_flac_md5_valid_detects_random_md5_corruption(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            ensure_flac_has_md5(test_file)
+            corrupt_md5(test_file, "random")
+
+            assert (
+                is_flac_md5_valid(test_file) == FlacMd5State.INVALID
+            ), "Random MD5 corruption should be detected as INVALID"
+
+    def test_is_flac_md5_valid_detects_partial_md5_corruption(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            ensure_flac_has_md5(test_file)
+            corrupt_md5(test_file, "partial")
+
+            assert (
+                is_flac_md5_valid(test_file) == FlacMd5State.INVALID
+            ), "Partially corrupted MD5 should be detected as INVALID"
+
+    def test_is_flac_md5_valid_detects_flipped_md5(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            ensure_flac_has_md5(test_file)
+            corrupt_md5(test_file, "flip_all")
+
+            assert is_flac_md5_valid(test_file) == FlacMd5State.INVALID, "Flipped MD5 should be detected as INVALID"

audiometa_python-0.8.1/audiometa/test/tests/integration/technical_info/flac_md5/md5_checking/test_unset_md5.py

@@ -0,0 +1,32 @@
+"""Test unset MD5 validation."""
+
+import pytest
+
+from audiometa import FlacMd5State, is_flac_md5_valid
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.test.tests.integration.technical_info.flac_md5.conftest import (
+    corrupt_md5,
+    create_flac_without_md5,
+    ensure_flac_has_md5,
+)
+
+
+@pytest.mark.integration
+class TestUnsetMd5:
+    def test_is_flac_md5_valid_with_naturally_unset_md5(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            # Create FLAC file without MD5 (naturally unset, not corrupted)
+            create_flac_without_md5(test_file)
+
+            # Unset MD5 (all zeros) should be detected as UNSET
+            result = is_flac_md5_valid(test_file)
+            assert result == FlacMd5State.UNSET, "Unset MD5 should be detected as UNSET"
+
+    def test_is_flac_md5_valid_with_corrupted_to_unset_md5(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            ensure_flac_has_md5(test_file)
+            corrupt_md5(test_file, "zeros")
+
+            # Unset MD5 (all zeros) should be detected as UNSET
+            result = is_flac_md5_valid(test_file)
+            assert result == FlacMd5State.UNSET, "Unset MD5 should be detected as UNSET"

audiometa_python-0.8.1/audiometa/test/tests/integration/technical_info/flac_md5/md5_repair/test_delete_original.py

@@ -0,0 +1,90 @@
+"""Test delete_original parameter behavior in get_file_with_corrected_md5."""
+
+from pathlib import Path
+from unittest.mock import patch
+
+import pytest
+
+from audiometa import FlacMd5State, is_flac_md5_valid
+from audiometa._audio_file import _AudioFile
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.test.tests.integration.technical_info.flac_md5.conftest import corrupt_md5, ensure_flac_has_md5
+
+
+@pytest.mark.integration
+class TestDeleteOriginalParameter:
+    """Test the delete_original parameter in get_file_with_corrected_md5."""
+
+    def test_delete_original_false_preserves_original_file(self):
+        """Test that delete_original=False (default) preserves the original file."""
+        with temp_file_with_metadata({}, "flac") as test_file:
+            ensure_flac_has_md5(test_file)
+            corrupt_md5(test_file, "flip_all")
+
+            # Verify initial state
+            assert Path(test_file).exists(), "Original file should exist before repair"
+            initial_state = is_flac_md5_valid(test_file)
+            assert initial_state == FlacMd5State.INVALID, "File should have invalid MD5 needing repair"
+
+            # Repair with delete_original=False (default)
+            audio_file = _AudioFile(test_file)
+            corrected_file_path = audio_file.get_file_with_corrected_md5(delete_original=False)
+
+            # Verify results
+            assert Path(test_file).exists(), "Original file should still exist after repair"
+            assert Path(corrected_file_path).exists(), "Corrected file should exist"
+
+            # Verify corrected file has valid MD5
+            final_state = is_flac_md5_valid(corrected_file_path)
+            assert final_state == FlacMd5State.VALID, "Corrected file should have valid MD5"
+
+    def test_delete_original_true_deletes_original_file(self):
+        """Test that delete_original=True deletes the original file."""
+        with temp_file_with_metadata({}, "flac") as test_file:
+            ensure_flac_has_md5(test_file)
+            corrupt_md5(test_file, "flip_all")
+
+            # Verify initial state
+            assert Path(test_file).exists(), "Original file should exist before repair"
+            initial_state = is_flac_md5_valid(test_file)
+            assert initial_state == FlacMd5State.INVALID, "File should have invalid MD5 needing repair"
+
+            # Repair with delete_original=True
+            audio_file = _AudioFile(test_file)
+            corrected_file_path = audio_file.get_file_with_corrected_md5(delete_original=True)
+
+            # Verify results
+            assert not Path(test_file).exists(), "Original file should be deleted after repair"
+            assert Path(corrected_file_path).exists(), "Corrected file should exist"
+
+            # Verify corrected file has valid MD5
+            final_state = is_flac_md5_valid(corrected_file_path)
+            assert final_state == FlacMd5State.VALID, "Corrected file should have valid MD5"
+
+    def test_delete_original_handles_deletion_error(self):
+        """Test that OSError is raised when original file deletion fails."""
+        with temp_file_with_metadata({}, "flac") as test_file:
+            ensure_flac_has_md5(test_file)
+            corrupt_md5(test_file, "flip_all")
+
+            # Verify initial state
+            assert Path(test_file).exists(), "Original file should exist before repair"
+
+            # Mock Path.unlink to raise OSError
+            with patch.object(Path, "unlink") as mock_unlink:
+                mock_unlink.side_effect = OSError("Permission denied")
+
+                # Attempt repair with delete_original=True
+                audio_file = _AudioFile(test_file)
+                with pytest.raises(RuntimeError) as exc_info:
+                    audio_file.get_file_with_corrected_md5(delete_original=True)
+
+                assert "Failed to execute FLAC command" in str(exc_info.value)
+                assert "Failed to delete original file" in str(exc_info.value)
+                assert "Permission denied" in str(exc_info.value)
+
+                # Verify unlink was called (indicating the deletion attempt was made)
+                mock_unlink.assert_called_once()
+
+                # Original file should still exist since deletion failed (mocked)
+                assert Path(test_file).exists(), "Original file should still exist when deletion fails"

{audiometa_python-0.7.1/audiometa/test/tests/integration/technical_info/flac_md5 → audiometa_python-0.8.1/audiometa/test/tests/integration/technical_info/flac_md5/md5_repair}/test_invalid_md5/test_flipped_md5.py

@@ -9,13 +9,6 @@ from audiometa.test.tests.integration.technical_info.flac_md5.conftest import co
 
 @pytest.mark.integration
 class TestFlippedMd5:
-    def test_is_flac_md5_valid_detects_flipped_md5(self):
-        with temp_file_with_metadata({}, "flac") as test_file:
-            ensure_flac_has_md5(test_file)
-            corrupt_md5(test_file, "flip_all")
-
-            assert is_flac_md5_valid(test_file) == FlacMd5State.INVALID, "Flipped MD5 should be detected as INVALID"
-
     def test_fix_md5_checking_flac(self):
         with temp_file_with_metadata({}, "flac") as test_file:
             ensure_flac_has_md5(test_file)

{audiometa_python-0.7.1/audiometa/test/tests/integration/technical_info/flac_md5 → audiometa_python-0.8.1/audiometa/test/tests/integration/technical_info/flac_md5/md5_repair}/test_invalid_md5/test_partial_md5.py

@@ -9,15 +9,6 @@ from audiometa.test.tests.integration.technical_info.flac_md5.conftest import co
 
 @pytest.mark.integration
 class TestPartialMd5:
-    def test_is_flac_md5_valid_detects_partial_md5_corruption(self):
-        with temp_file_with_metadata({}, "flac") as test_file:
-            ensure_flac_has_md5(test_file)
-            corrupt_md5(test_file, "partial")
-
-            assert (
-                is_flac_md5_valid(test_file) == FlacMd5State.INVALID
-            ), "Partially corrupted MD5 should be detected as INVALID"
-
     def test_fix_md5_checking_with_partial_corruption(self):
         with temp_file_with_metadata({}, "flac") as test_file:
             ensure_flac_has_md5(test_file)

{audiometa_python-0.7.1/audiometa/test/tests/integration/technical_info/flac_md5 → audiometa_python-0.8.1/audiometa/test/tests/integration/technical_info/flac_md5/md5_repair}/test_invalid_md5/test_random_md5.py

@@ -9,15 +9,6 @@ from audiometa.test.tests.integration.technical_info.flac_md5.conftest import co
 
 @pytest.mark.integration
 class TestRandomMd5:
-    def test_is_flac_md5_valid_detects_random_md5_corruption(self):
-        with temp_file_with_metadata({}, "flac") as test_file:
-            ensure_flac_has_md5(test_file)
-            corrupt_md5(test_file, "random")
-
-            assert (
-                is_flac_md5_valid(test_file) == FlacMd5State.INVALID
-            ), "Random MD5 corruption should be detected as INVALID"
-
     def test_fix_md5_checking_with_random_corruption(self):
         with temp_file_with_metadata({}, "flac") as test_file:
             ensure_flac_has_md5(test_file)