mkv-episode-matcher 0.9.4__tar.gz → 0.9.6__tar.gz

mkv_episode_matcher-0.9.6/.claude/settings.local.json

@@ -0,0 +1,20 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(gh issue view:*)",
+      "Bash(git checkout:*)",
+      "Bash(python -m pytest tests/ -v)",
+      "Bash(uv run pytest:*)",
+      "Bash(gh pr view:*)",
+      "Bash(gh pr checks:*)",
+      "WebFetch(domain:github.com)",
+      "Bash(gh run view:*)",
+      "Bash(uv run:*)",
+      "Bash(git add:*)",
+      "Bash(git commit:*)",
+      "Bash(git push:*)",
+      "Bash(rm:*)"
+    ],
+    "deny": []
+  }
+}

mkv_episode_matcher-0.9.6/.github/workflows/claude-code-review.yml

@@ -0,0 +1,78 @@
+name: Claude Code Review
+
+on:
+  pull_request:
+    types: [opened, synchronize]
+    # Optional: Only run on specific file changes
+    # paths:
+    #   - "src/**/*.ts"
+    #   - "src/**/*.tsx"
+    #   - "src/**/*.js"
+    #   - "src/**/*.jsx"
+
+jobs:
+  claude-review:
+    # Optional: Filter by PR author
+    # if: |
+    #   github.event.pull_request.user.login == 'external-contributor' ||
+    #   github.event.pull_request.user.login == 'new-developer' ||
+    #   github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR'
+    
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      pull-requests: read
+      issues: read
+      id-token: write
+    
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 1
+
+      - name: Run Claude Code Review
+        id: claude-review
+        uses: anthropics/claude-code-action@beta
+        with:
+          claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
+
+          # Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
+          # model: "claude-opus-4-20250514"
+          
+          # Direct prompt for automated review (no @claude mention needed)
+          direct_prompt: |
+            Please review this pull request and provide feedback on:
+            - Code quality and best practices
+            - Potential bugs or issues
+            - Performance considerations
+            - Security concerns
+            - Test coverage
+            
+            Be constructive and helpful in your feedback.
+
+          # Optional: Use sticky comments to make Claude reuse the same comment on subsequent pushes to the same PR
+          # use_sticky_comment: true
+          
+          # Optional: Customize review based on file types
+          # direct_prompt: |
+          #   Review this PR focusing on:
+          #   - For TypeScript files: Type safety and proper interface usage
+          #   - For API endpoints: Security, input validation, and error handling
+          #   - For React components: Performance, accessibility, and best practices
+          #   - For tests: Coverage, edge cases, and test quality
+          
+          # Optional: Different prompts for different authors
+          # direct_prompt: |
+          #   ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' && 
+          #   'Welcome! Please review this PR from a first-time contributor. Be encouraging and provide detailed explanations for any suggestions.' ||
+          #   'Please provide a thorough code review focusing on our coding standards and best practices.' }}
+          
+          # Optional: Add specific tools for running tests or linting
+          # allowed_tools: "Bash(npm run test),Bash(npm run lint),Bash(npm run typecheck)"
+          
+          # Optional: Skip review for certain conditions
+          # if: |
+          #   !contains(github.event.pull_request.title, '[skip-review]') &&
+          #   !contains(github.event.pull_request.title, '[WIP]')
+

mkv_episode_matcher-0.9.6/.github/workflows/claude.yml

@@ -0,0 +1,64 @@
+name: Claude Code
+
+on:
+  issue_comment:
+    types: [created]
+  pull_request_review_comment:
+    types: [created]
+  issues:
+    types: [opened, assigned]
+  pull_request_review:
+    types: [submitted]
+
+jobs:
+  claude:
+    if: |
+      (github.event_name == 'issue_comment' && contains(github.event.comment.body, '@claude')) ||
+      (github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, '@claude')) ||
+      (github.event_name == 'pull_request_review' && contains(github.event.review.body, '@claude')) ||
+      (github.event_name == 'issues' && (contains(github.event.issue.body, '@claude') || contains(github.event.issue.title, '@claude')))
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      pull-requests: read
+      issues: read
+      id-token: write
+      actions: read # Required for Claude to read CI results on PRs
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 1
+
+      - name: Run Claude Code
+        id: claude
+        uses: anthropics/claude-code-action@beta
+        with:
+          claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
+
+          # This is an optional setting that allows Claude to read CI results on PRs
+          additional_permissions: |
+            actions: read
+          
+          # Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
+          # model: "claude-opus-4-20250514"
+          
+          # Optional: Customize the trigger phrase (default: @claude)
+          # trigger_phrase: "/claude"
+          
+          # Optional: Trigger when specific user is assigned to an issue
+          # assignee_trigger: "claude-bot"
+          
+          # Optional: Allow Claude to run specific commands
+          # allowed_tools: "Bash(npm install),Bash(npm run build),Bash(npm run test:*),Bash(npm run lint:*)"
+          
+          # Optional: Add custom instructions for Claude to customize its behavior for your project
+          # custom_instructions: |
+          #   Follow our coding standards
+          #   Ensure all new code has tests
+          #   Use TypeScript for new files
+          
+          # Optional: Custom environment variables for Claude
+          # claude_env: |
+          #   NODE_ENV: test
+

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mkv-episode-matcher
-Version: 0.9.4
+Version: 0.9.6
 Summary: The MKV Episode Matcher is a tool for identifying TV series episodes from MKV files and renaming the files accordingly.
 Home-page: https://github.com/Jsakkos/mkv-episode-matcher
 Author: Jonathan Sakkos

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.6}/mkv_episode_matcher/config.py

@@ -43,7 +43,7 @@ def set_config(
     Returns:
         None
     """
-    config = configparser.ConfigParser()
+    config = configparser.ConfigParser(interpolation=None)
     config["Config"] = {
         "tmdb_api_key": str(tmdb_api_key),
         "show_dir": show_dir,
@@ -56,7 +56,7 @@ def set_config(
     logger.info(
         f"Setting config with API:{tmdb_api_key}, show_dir: {show_dir}, and max_threads: {MAX_THREADS}"
     )
-    with open(file, "w") as configfile:
+    with open(file, "w", encoding="utf-8") as configfile:
         config.write(configfile)
 
 
@@ -72,8 +72,8 @@ def get_config(file):
 
     """
     logger.info(f"Loading config from {file}")
-    config = configparser.ConfigParser()
+    config = configparser.ConfigParser(interpolation=None)
     if Path(file).exists():
-        config.read(file)
+        config.read(file, encoding="utf-8")
         return config["Config"] if "Config" in config else None
     return {}

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.6}/mkv_episode_matcher/episode_identification.py

@@ -155,11 +155,13 @@ class EpisodeMatcher:
         ]
 
         reference_files = []
-        for _pattern in patterns:
+        for pattern in patterns:
+            # Use case-insensitive file extension matching by checking both .srt and .SRT
+            srt_files = list(reference_dir.glob("*.srt")) + list(reference_dir.glob("*.SRT"))
             files = [
                 f
-                for f in reference_dir.glob("*.srt")
-                if any(re.search(f"{p}\\d+", f.name, re.IGNORECASE) for p in patterns)
+                for f in srt_files
+                if re.search(f"{pattern}\\d+", f.name, re.IGNORECASE)
             ]
             reference_files.extend(files)
 

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.6}/mkv_episode_matcher.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mkv-episode-matcher
-Version: 0.9.4
+Version: 0.9.6
 Summary: The MKV Episode Matcher is a tool for identifying TV series episodes from MKV files and renaming the files accordingly.
 Home-page: https://github.com/Jsakkos/mkv-episode-matcher
 Author: Jonathan Sakkos

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.6}/mkv_episode_matcher.egg-info/SOURCES.txt

@@ -11,7 +11,10 @@ pyproject.toml
 setup.cfg
 setup.py
 uv.lock
+.claude/settings.local.json
 .github/funding.yml
+.github/workflows/claude-code-review.yml
+.github/workflows/claude.yml
 .github/workflows/documentation.yml
 .github/workflows/python-publish.yml
 .github/workflows/tests.yml
@@ -39,6 +42,8 @@ mkv_episode_matcher.egg-info/entry_points.txt
 mkv_episode_matcher.egg-info/requires.txt
 mkv_episode_matcher.egg-info/top_level.txt
 tests/__init__.py
+tests/test_config_special_characters.py
+tests/test_episode_identification.py
 tests/test_main.py
 tests/test_path_handling.py
 tests/test_trailing_slash.py

{mkv_episode_matcher-0.9.4 → mkv_episode_matcher-0.9.6}/setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = mkv_episode_matcher
-version = 0.9.4
+version = 0.9.6
 author = Jonathan Sakkos
 author_email = jonathansakkos@gmail.com
 description = The MKV Episode Matcher is a tool for identifying TV series episodes from MKV files and renaming the files accordingly.

mkv_episode_matcher-0.9.6/tests/test_config_special_characters.py

@@ -0,0 +1,174 @@
+"""Test cases for config.py handling of special characters in passwords."""
+
+import tempfile
+from pathlib import Path
+
+import pytest
+
+from mkv_episode_matcher.config import get_config, set_config
+
+
+class TestConfigSpecialCharacters:
+    """Test that config handling works correctly with special characters in passwords."""
+
+    @pytest.fixture
+    def temp_config_file(self):
+        """Create a temporary config file for testing."""
+        with tempfile.NamedTemporaryFile(mode='w', suffix='.ini', delete=False) as f:
+            yield f.name
+        Path(f.name).unlink(missing_ok=True)
+
+    @pytest.fixture
+    def mock_config_data(self):
+        """Mock configuration data for testing."""
+        return {
+            "tmdb_api_key": "test_tmdb_api_key",
+            "open_subtitles_api_key": "test_os_api_key",
+            "open_subtitles_user_agent": "test_user_agent",
+            "open_subtitles_username": "test_username",
+            "show_dir": "/test/path"
+        }
+
+    def test_password_with_percent_symbol(self, temp_config_file, mock_config_data):
+        """Test that passwords containing % symbol don't cause interpolation errors."""
+        password_with_percent = "H7z*X$X29JdJ^#%Q"  # gitguardian:ignore
+        
+        # This should not raise a ValueError
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            password_with_percent,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        # Verify the config was written successfully
+        config = get_config(temp_config_file)
+        assert config is not None
+        assert config["open_subtitles_password"] == password_with_percent
+
+    def test_password_with_multiple_percent_symbols(self, temp_config_file, mock_config_data):
+        """Test that passwords with multiple % symbols work correctly."""
+        password_with_percents = "password%with%multiple%percent%signs"
+        
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            password_with_percents,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        config = get_config(temp_config_file)
+        assert config["open_subtitles_password"] == password_with_percents
+
+    def test_password_with_interpolation_like_syntax(self, temp_config_file, mock_config_data):
+        """Test that passwords resembling interpolation syntax are handled correctly."""
+        # This resembles ConfigParser interpolation syntax but should be treated literally
+        password_with_interpolation = "%(section)s_password_%(option)s"
+        
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            password_with_interpolation,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        config = get_config(temp_config_file)
+        assert config["open_subtitles_password"] == password_with_interpolation
+
+    def test_password_with_various_special_characters(self, temp_config_file, mock_config_data):
+        """Test that passwords with various special characters work correctly."""
+        special_passwords = [
+            "pass@word!123",
+            "my$ecret#key*",
+            "complex&password^with()brackets[]",
+            "unicode_测试_password",
+            "spaces in password",
+            "tabs\tand\nnewlines",
+        ]
+        
+        for password in special_passwords:
+            set_config(
+                mock_config_data["tmdb_api_key"],
+                mock_config_data["open_subtitles_api_key"],
+                mock_config_data["open_subtitles_user_agent"],
+                mock_config_data["open_subtitles_username"],
+                password,
+                mock_config_data["show_dir"],
+                temp_config_file,
+            )
+            
+            config = get_config(temp_config_file)
+            assert config["open_subtitles_password"] == password, f"Failed for password: {password}"
+
+    def test_original_bug_case(self, temp_config_file, mock_config_data):
+        """Test the specific password from the original bug report."""
+        # This is the exact password that caused the original issue
+        problematic_password = "H7z*X$X29JdJ^#%Q"  # gitguardian:ignore
+        
+        # Before the fix, this would raise:
+        # ValueError: invalid interpolation syntax in 'H7z*X$X29JdJ^#%Q' at position 14
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            problematic_password,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        # Verify we can read it back correctly
+        config = get_config(temp_config_file)
+        assert config is not None
+        assert config["open_subtitles_password"] == problematic_password
+        
+        # Verify all other fields are preserved
+        assert config["tmdb_api_key"] == mock_config_data["tmdb_api_key"]
+        assert config["open_subtitles_username"] == mock_config_data["open_subtitles_username"]
+        assert config["show_dir"] == mock_config_data["show_dir"]
+
+    def test_empty_password(self, temp_config_file, mock_config_data):
+        """Test that empty passwords are handled correctly."""
+        empty_password = ""
+        
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            empty_password,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        config = get_config(temp_config_file)
+        assert config["open_subtitles_password"] == empty_password
+
+    def test_config_persistence(self, temp_config_file, mock_config_data):
+        """Test that config values persist correctly across multiple operations."""
+        password = "persistent%password#123"
+        
+        # Set config
+        set_config(
+            mock_config_data["tmdb_api_key"],
+            mock_config_data["open_subtitles_api_key"],
+            mock_config_data["open_subtitles_user_agent"],
+            mock_config_data["open_subtitles_username"],
+            password,
+            mock_config_data["show_dir"],
+            temp_config_file,
+        )
+        
+        # Read config multiple times to ensure consistency
+        for _ in range(3):
+            config = get_config(temp_config_file)
+            assert config["open_subtitles_password"] == password

mkv_episode_matcher-0.9.6/tests/test_episode_identification.py

@@ -0,0 +1,286 @@
+import unittest
+import tempfile
+import re
+from pathlib import Path
+from unittest.mock import Mock, patch
+
+from mkv_episode_matcher.episode_identification import EpisodeMatcher
+
+
+class TestEpisodeIdentificationPatternMatching(unittest.TestCase):
+    """Test the pattern matching functionality in episode identification."""
+
+    def setUp(self):
+        """Set up test fixtures."""
+        self.temp_dir = tempfile.TemporaryDirectory()
+        self.cache_dir = Path(self.temp_dir.name)
+        
+        # Mock subtitle cache
+        self.mock_subtitle_cache = Mock()
+        
+        # Create identifier instance
+        self.identifier = EpisodeMatcher(
+            cache_dir=self.cache_dir,
+            show_name="Test Show (2023)"
+        )
+
+    def tearDown(self):
+        """Clean up test fixtures."""
+        self.temp_dir.cleanup()
+
+    def _create_test_files(self, filenames, show_name=None):
+        """Create test subtitle files in the cache directory."""
+        if show_name is None:
+            show_name = self.identifier.show_name
+        
+        show_dir = self.cache_dir / "data" / show_name
+        show_dir.mkdir(parents=True, exist_ok=True)
+        
+        created_files = []
+        for filename in filenames:
+            file_path = show_dir / filename
+            file_path.write_text("dummy subtitle content")
+            created_files.append(file_path)
+        
+        return created_files
+
+    def test_pattern_matching_s01e_format(self):
+        """Test pattern matching for S01E format files."""
+        # Create test files with S01E format
+        test_files = [
+            "Test Show (2023) - S01E01.srt",
+            "Test Show (2023) - S01E02.srt", 
+            "Test Show (2023) - S01E10.srt",
+            "Test Show (2023) - S02E01.srt",  # Should not match season 1
+        ]
+        
+        self._create_test_files(test_files)
+        
+        # Test season 1 matching
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find 3 files for season 1
+        self.assertEqual(len(reference_files), 3)
+        
+        # Verify correct files are matched
+        matched_names = [f.name for f in reference_files]
+        self.assertIn("Test Show (2023) - S01E01.srt", matched_names)
+        self.assertIn("Test Show (2023) - S01E02.srt", matched_names)
+        self.assertIn("Test Show (2023) - S01E10.srt", matched_names)
+        self.assertNotIn("Test Show (2023) - S02E01.srt", matched_names)
+
+    def test_pattern_matching_s1e_format(self):
+        """Test pattern matching for S1E format files."""
+        test_files = [
+            "Show - S1E01.srt",
+            "Show - S1E05.srt",
+            "Show - S2E01.srt",  # Should not match season 1
+        ]
+        
+        self._create_test_files(test_files)
+        
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find 2 files for season 1
+        self.assertEqual(len(reference_files), 2)
+        
+        matched_names = [f.name for f in reference_files]
+        self.assertIn("Show - S1E01.srt", matched_names)
+        self.assertIn("Show - S1E05.srt", matched_names)
+        self.assertNotIn("Show - S2E01.srt", matched_names)
+
+    def test_pattern_matching_01x_format(self):
+        """Test pattern matching for 01x format files."""
+        test_files = [
+            "Show - 01x01.srt",
+            "Show - 01x15.srt",
+            "Show - 02x01.srt",  # Should not match season 1
+        ]
+        
+        self._create_test_files(test_files)
+        
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find 2 files for season 1
+        self.assertEqual(len(reference_files), 2)
+        
+        matched_names = [f.name for f in reference_files]
+        self.assertIn("Show - 01x01.srt", matched_names)
+        self.assertIn("Show - 01x15.srt", matched_names)
+        self.assertNotIn("Show - 02x01.srt", matched_names)
+
+    def test_pattern_matching_1x_format(self):
+        """Test pattern matching for 1x format files."""
+        test_files = [
+            "Show - 1x01.srt",
+            "Show - 1x08.srt",
+            "Show - 2x01.srt",  # Should not match season 1
+        ]
+        
+        self._create_test_files(test_files)
+        
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find 2 files for season 1
+        self.assertEqual(len(reference_files), 2)
+        
+        matched_names = [f.name for f in reference_files]
+        self.assertIn("Show - 1x01.srt", matched_names)
+        self.assertIn("Show - 1x08.srt", matched_names)
+        self.assertNotIn("Show - 2x01.srt", matched_names)
+
+    def test_pattern_matching_mixed_formats(self):
+        """Test pattern matching with mixed file formats."""
+        test_files = [
+            "Show - S01E01.srt",
+            "Show - S1E02.srt", 
+            "Show - 01x03.srt",
+            "Show - 1x04.srt",
+            "Show - S02E01.srt",  # Different season
+            "Show - episode1.srt",  # No matching pattern
+        ]
+        
+        self._create_test_files(test_files)
+        
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find 4 files for season 1 (all matching patterns)
+        self.assertEqual(len(reference_files), 4)
+        
+        matched_names = [f.name for f in reference_files]
+        self.assertIn("Show - S01E01.srt", matched_names)
+        self.assertIn("Show - S1E02.srt", matched_names)
+        self.assertIn("Show - 01x03.srt", matched_names)
+        self.assertIn("Show - 1x04.srt", matched_names)
+        self.assertNotIn("Show - S02E01.srt", matched_names)
+        self.assertNotIn("Show - episode1.srt", matched_names)
+
+    def test_case_insensitive_matching(self):
+        """Test that pattern matching is case insensitive."""
+        test_files = [
+            "show - s01e01.srt",  # lowercase s and e
+            "SHOW - S01E02.SRT",  # uppercase everything
+            "Show - S01e03.srt",  # mixed case
+        ]
+        
+        self._create_test_files(test_files)
+        
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find all 3 files regardless of case
+        self.assertEqual(len(reference_files), 3)
+
+    def test_dexter_new_blood_issue_reproduction(self):
+        """Test the specific issue from bug report with Dexter New Blood files."""
+        # Create files matching the exact pattern from the bug report
+        test_files = [
+            "Dexter - New Blood (2021) - S01E01.srt",
+            "Dexter - New Blood (2021) - S01E02.srt",
+            "Dexter - New Blood (2021) - S01E03.srt",
+            "Dexter - New Blood (2021) - S01E04.srt",
+            "Dexter - New Blood (2021) - S01E05.srt",
+            "Dexter - New Blood (2021) - S01E06.srt",
+            "Dexter - New Blood (2021) - S01E07.srt",
+            "Dexter - New Blood (2021) - S01E08.srt",
+            "Dexter - New Blood (2021) - S01E09.srt",
+            "Dexter - New Blood (2021) - S01E10.srt",
+        ]
+        
+        # Update the identifier to use the exact show name from the bug report
+        show_name = "Dexter - New Blood (2021)"
+        self.identifier.show_name = show_name
+        
+        self._create_test_files(test_files, show_name)
+        
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find all 10 files - this was the bug that was failing
+        self.assertEqual(len(reference_files), 10, 
+                        f"Expected 10 files but found {len(reference_files)}. "
+                        f"Files found: {[f.name for f in reference_files]}")
+
+    def test_no_duplicate_files(self):
+        """Test that duplicate matches are removed properly."""
+        # Create a file that could match multiple patterns
+        test_files = [
+            "Show - S01E01.srt",  # Could match both S01E and 01x patterns if logic is wrong
+        ]
+        
+        self._create_test_files(test_files)
+        
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find only 1 unique file, not duplicates
+        self.assertEqual(len(reference_files), 1)
+        
+        # Verify it's the correct file
+        self.assertEqual(reference_files[0].name, "Show - S01E01.srt")
+
+    def test_empty_directory(self):
+        """Test behavior when no matching files exist."""
+        # Create the directory but no files
+        show_dir = self.cache_dir / "data" / "Test Show (2023)"
+        show_dir.mkdir(parents=True, exist_ok=True)
+        
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find no files
+        self.assertEqual(len(reference_files), 0)
+
+    def test_directory_does_not_exist(self):
+        """Test behavior when the show directory doesn't exist."""
+        # Don't create the directory
+        reference_files = self.identifier.get_reference_files(1)
+        
+        # Should find no files (glob returns empty on non-existent directory)
+        self.assertEqual(len(reference_files), 0)
+
+    def test_caching_functionality(self):
+        """Test that results are properly cached."""
+        test_files = ["Show - S01E01.srt"]
+        self._create_test_files(test_files)
+        
+        # First call
+        reference_files_1 = self.identifier.get_reference_files(1)
+        
+        # Second call should return cached result
+        reference_files_2 = self.identifier.get_reference_files(1)
+        
+        # Should be the same result
+        self.assertEqual(reference_files_1, reference_files_2)
+        
+        # Verify it's actually using the cache by checking the cache directly
+        cache_key = ("Test Show (2023)", 1)
+        self.assertIn(cache_key, self.identifier.reference_files_cache)
+
+    def test_different_seasons(self):
+        """Test that different seasons are handled correctly."""
+        test_files = [
+            "Show - S01E01.srt",
+            "Show - S01E02.srt",
+            "Show - S02E01.srt",
+            "Show - S02E02.srt",
+            "Show - S03E01.srt",
+        ]
+        
+        self._create_test_files(test_files)
+        
+        # Test season 1
+        season1_files = self.identifier.get_reference_files(1)
+        self.assertEqual(len(season1_files), 2)
+        
+        # Test season 2
+        season2_files = self.identifier.get_reference_files(2)
+        self.assertEqual(len(season2_files), 2)
+        
+        # Test season 3
+        season3_files = self.identifier.get_reference_files(3)
+        self.assertEqual(len(season3_files), 1)
+        
+        # Test non-existent season
+        season4_files = self.identifier.get_reference_files(4)
+        self.assertEqual(len(season4_files), 0)
+
+
+if __name__ == '__main__':
+    unittest.main()