Fix Windows file locking issues in logo caching tests

- Replace TemporaryDirectory with manual mkdtemp for Windows compatibility
- Explicitly close PIL Image objects to release file handles
- Add Windows-compatible cleanup with retry logic and error handling
- Prevents PermissionError [WinError 32] on Windows CI

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: jbf302

tests/test_logos.py

@@ -51,7 +51,8 @@ def test_get_available_teams_returns_list(self):
     def test_logo_caching_works(self):
         """Test that logo caching works properly."""
         # Use temporary directory for test cache
-        with tempfile.TemporaryDirectory() as temp_dir:
+        temp_dir = tempfile.mkdtemp()
+        try:
             manager = NFLAssetManager(cache_dir=temp_dir)
 
             # First download should create cache
@@ -65,6 +66,23 @@ def test_logo_caching_works(self):
             self.assertIsInstance(logo1, Image.Image)
             self.assertIsInstance(logo2, Image.Image)
             self.assertEqual(cache_info_1['logos_count'], cache_info_2['logos_count'])
+            
+            # Close images to release file handles on Windows
+            logo1.close()
+            logo2.close()
+            
+        finally:
+            # Windows-compatible cleanup
+            try:
+                shutil.rmtree(temp_dir)
+            except (OSError, PermissionError):
+                # On Windows, wait briefly and try again
+                import time
+                time.sleep(0.1)
+                try:
+                    shutil.rmtree(temp_dir)
+                except (OSError, PermissionError):
+                    pass  # Cleanup failed but test completed successfully
 
 
 class test_matplotlib_integration(TestCase):
@@ -205,25 +223,36 @@ class test_asset_manager(TestCase):
 
     def test_asset_manager_cache_info(self):
         """Test that asset manager provides cache information."""
-        with tempfile.TemporaryDirectory() as temp_dir:
+        temp_dir = tempfile.mkdtemp()
+        try:
             manager = NFLAssetManager(cache_dir=temp_dir)
             cache_info = manager.get_cache_info()
 
             self.assertIsInstance(cache_info, dict)
             self.assertIn('cache_dir', cache_info)
             self.assertIn('logos_count', cache_info)
             self.assertIn('total_size_bytes', cache_info)
+            
+        finally:
+            try:
+                shutil.rmtree(temp_dir)
+            except (OSError, PermissionError):
+                pass
 
     def test_asset_manager_clear_cache(self):
         """Test cache clearing functionality."""
-        with tempfile.TemporaryDirectory() as temp_dir:
+        temp_dir = tempfile.mkdtemp()
+        try:
             manager = NFLAssetManager(cache_dir=temp_dir)
 
             # Download a logo to create cache
             try:
-                manager.get_logo('KC')
+                logo = manager.get_logo('KC')
                 cache_info_before = manager.get_cache_info()
 
+                # Close image to release file handle
+                logo.close()
+                
                 # Clear cache
                 manager.clear_cache('logos')
                 cache_info_after = manager.get_cache_info()
@@ -233,4 +262,10 @@ def test_asset_manager_clear_cache(self):
 
             except Exception:
                 # If logo download fails, that's okay for this test
+                pass
+                
+        finally:
+            try:
+                shutil.rmtree(temp_dir)
+            except (OSError, PermissionError):
                 pass