added more comments, fixed imports

Author: MK2112

tests/test_cli.py

@@ -1,20 +1,18 @@
-import pytest
 import sys
-from tests.test_fixtures import any_to_any_instance
+import subprocess
+import os
 
 def test_cli_help_output(tmp_path):
-    import subprocess
-    import os
     script_path = os.path.abspath(os.path.join(os.path.dirname(__file__), '..', 'any_to_any.py'))
     result = subprocess.run([sys.executable, script_path, "-h"], capture_output=True, text=True)
     assert "usage" in result.stdout.lower() or "usage" in result.stderr.lower()
 
 def test_cli_invalid_format(tmp_path):
-    import subprocess
     result = subprocess.run([sys.executable, "any_to_any.py", "-i", str(tmp_path), "-f", "xyz"], cwd=str(tmp_path.parent), capture_output=True, text=True)
     assert "unsupported format" in result.stdout.lower() or "unsupported format" in result.stderr.lower() or result.returncode != 0
 
 def test_blank_start_no_files_in_cli_output(any_to_any_instance, caplog):
     with caplog.at_level("INFO"):
+        # Jesus Christ, centralize this; maybe make a central factory for this
         any_to_any_instance.run([], None, None, None, None, False, False, False, False, False, False, "en_US")
     assert "No convertible media files" in caplog.text

tests/test_conversion_methods.py

@@ -1,7 +1,6 @@
 import pytest
 import warnings
 from any_to_any import Category
-from tests.test_fixtures import any_to_any_instance
 
 def test_to_audio_invalid_format(any_to_any_instance, tmp_path):
     invalid_file = tmp_path / "invalid_file.mp3"
@@ -17,6 +16,7 @@ def test_to_audio_invalid_format(any_to_any_instance, tmp_path):
 
 def test_to_gif_handles_invalid_file(any_to_any_instance, tmp_path):
     fake_file = tmp_path / "fake.mp3"
+    # This is funny, I like this pattern, need to suppress any rambling from ffmpeg though
     fake_file.write_bytes(b"\x00" * 128)
     with pytest.raises(Exception):
         any_to_any_instance.to_gif({'audio': [(str(tmp_path) + "/", 'fake', 'mp3')]}, 'gif')

tests/test_core.py

@@ -1,13 +1,11 @@
-import pytest
-from any_to_any import AnyToAny
-from tests.test_fixtures import any_to_any_instance, test_input_folder
 from modules.category import Category
 
 def test_supported_formats(any_to_any_instance):
     assert isinstance(any_to_any_instance.supported_formats, list)
     assert len(any_to_any_instance.supported_formats) > 0
 
 def test_audio_bitrate(any_to_any_instance):
+    # Kind of nonsensical, I know, but it is a low-level structural test anyway
     assert any_to_any_instance._audio_bitrate('mp3', 'high') == '320k'
     assert any_to_any_instance._audio_bitrate('flac', 'medium') == '320k'
     assert any_to_any_instance._audio_bitrate('ogg', 'low') == '128k'

tests/test_edge_cases.py

@@ -1,5 +1,6 @@
 import pytest
-from tests.test_fixtures import any_to_any_instance
+import random
+import string
 from unittest.mock import patch
 
 def test_empty_directory(any_to_any_instance, tmp_path):
@@ -10,6 +11,7 @@ def test_empty_directory(any_to_any_instance, tmp_path):
 
 def test_permission_error_on_output(any_to_any_instance, tmp_path):
     file_path = tmp_path / "test.mp4"
+    # Building a tiny fake file
     file_path.write_bytes(b"\x00" * 128)
     with patch("os.remove", side_effect=PermissionError):
         with pytest.raises(PermissionError):
@@ -20,10 +22,10 @@ def test_get_file_paths_invalid_directory(any_to_any_instance):
         any_to_any_instance._get_file_paths(input="nonexistent_directory")
 
 def test_fuzz_random_file_names(any_to_any_instance, tmp_path):
-    import random
-    import string
     for _ in range(10):
         name = ''.join(random.choices(string.ascii_letters, k=8))
+        # Specifically checking the 'malformed' case of 'jpg' instead of 'jpeg'
+        # because people use it so often, this has to be handled/supported
         ext = random.choice(['mp4', 'mp3', 'jpg', 'wav', 'pdf'])
         file = tmp_path / f"{name}.{ext}"
         file.write_bytes(b"\x00" * 128)

tests/test_fixtures.py

@@ -1,15 +1,15 @@
-import sys
 import os
-sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
+import sys
 import pytest
-import tempfile
-import shutil
+
+sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from any_to_any import AnyToAny
 
 @pytest.fixture
 def temp_media_dir(tmp_path):
     media_dir = tmp_path / "media"
     media_dir.mkdir()
+    # Fake File Creation
     (media_dir / "test.mp4").write_bytes(b"\x00" * 128)
     (media_dir / "test.mp3").write_bytes(b"\x00" * 128)
     (media_dir / "test.jpg").write_bytes(b"\x00" * 128)

tests/test_integration_and_routing.py

@@ -1,20 +1,14 @@
 import pytest
-import sys
-import os
 import argparse
 from unittest import mock
 from any_to_any import AnyToAny
-from modules.category import Category
-import types
 
 @pytest.fixture
 def instance():
     return AnyToAny()
 
 def test_routing_supported_formats(instance):
-    """
-    Ensure all supported formats are routed to the correct handler or codec.
-    """
+    # Ensure all supported formats are routed to the correct handler or codec.
     for cat, formats in instance._supported_formats.items():
         for fmt, handler in formats.items():
             if callable(handler):
@@ -27,28 +21,22 @@ def test_routing_supported_formats(instance):
                 assert isinstance(handler, (str, list))
 
 def test_run_web_flag_starts_web():
-    """
-    Test that the CLI parser recognizes the -w/--web flag.
-    """
+    # Test that the CLI parser recognizes the -w/--web flag.
     parser = argparse.ArgumentParser()
     parser.add_argument('-w', '--web', action='store_true')
     args = parser.parse_args(['-w'])
     assert args.web
 
 
 def test_end_with_msg_logs_and_exits(instance, caplog):
-    """
-    Test _end_with_msg logs error and raises SystemExit.
-    """
+    # Test _end_with_msg logs error and raises SystemExit.
     with caplog.at_level('WARNING'):
         with pytest.raises(SystemExit):
             instance._end_with_msg(SystemExit, 'fail message')
     assert 'fail message' in caplog.text
 
 def test_recursive_file_discovery(instance, tmp_path):
-    """
-    _get_file_paths does not recurse; only top-level files are found.
-    """
+    # _get_file_paths does not recurse; only top-level files are found.
     d1 = tmp_path / "a"
     d2 = d1 / "b"
     d2.mkdir(parents=True)
@@ -60,13 +48,11 @@ def test_recursive_file_discovery(instance, tmp_path):
         str(f.parent) in path[0] and path[1] == 'test' and path[2] == 'mp4'
         for paths in file_paths.values() for path in paths
     )
-    assert not found  # Documented limitation
+    assert not found  # Documented limitation, this is intentional
 
 
 def test_weird_filenames(instance, tmp_path):
-    """
-    Test handling of files with unicode and special chars in names.
-    """
+    # Test handling of files with unicode and special chars in names.
     fname = "weird_名字_#@!.mp3"
     f = tmp_path / fname
     f.write_bytes(b"\x00"*128)
@@ -75,9 +61,7 @@ def test_weird_filenames(instance, tmp_path):
     assert found
 
 def test_post_process_permission_error(instance, tmp_path):
-    """
-    Test _post_process logs and raises on permission error during delete.
-    """
+    # Test _post_process logs and raises on permission error during delete.
     f = tmp_path / "test.mp4"
     f.write_bytes(b"\x00"*128)
     with mock.patch("os.remove", side_effect=PermissionError):

tests/test_logging_progress.py

@@ -1,5 +1,3 @@
-import pytest
-from tests.test_fixtures import any_to_any_instance
 
 def test_logging_and_progress(any_to_any_instance, caplog):
     with caplog.at_level("INFO"):

tests/test_merge_concat.py

@@ -1,5 +1,4 @@
 import pytest
-from tests.test_fixtures import any_to_any_instance, test_input_folder, test_output_folder
 from modules.category import Category
 
 def test_merging_method(any_to_any_instance, test_input_folder, test_output_folder):