fix: isolate additional test files from production asset registry
Some checks failed
Test Suite / unit-tests (3.11) (push) Has been cancelled
Test Suite / unit-tests (3.12) (push) Has been cancelled
Test Suite / integration-tests (push) Has been cancelled
Test Suite / e2e-tests (push) Has been cancelled
Test Suite / performance-tests (push) Has been cancelled
Test Suite / code-quality (push) Has been cancelled
Test Suite / security-scan (push) Has been cancelled
Test Suite / test-summary (push) Has been cancelled

- Fix test_issue_144_auto_discovery_workspace.py to use isolated test workspace
- Fix test_issue_144_asset_optimization.py to use isolated test workspace
- Ensure all AssetManager instances use test-specific registry paths
- Prevent additional test artifacts from contaminating production registry

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

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
2025-10-24 19:58:18 +02:00
parent 04a9173503
commit 9d3c6f3c81
2 changed files with 17 additions and 13 deletions

View File

@@ -8,8 +8,6 @@ Issue #144: Phase 3 - Advanced Features and Performance
""" """
import pytest import pytest
import tempfile
import shutil
from pathlib import Path from pathlib import Path
from unittest.mock import Mock, patch, MagicMock from unittest.mock import Mock, patch, MagicMock
import json import json
@@ -21,6 +19,7 @@ from markitect.assets.optimizer import AssetOptimizer, OptimizationProfile, Opti
from markitect.assets.optimizer import AssetTransformer as OptimizerTransformer from markitect.assets.optimizer import AssetTransformer as OptimizerTransformer
from markitect.assets.transformer import AssetTransformer, ThumbnailGenerator from markitect.assets.transformer import AssetTransformer, ThumbnailGenerator
from markitect.assets.analyzer import ContentAnalyzer, SimilarityDetector, AssetMetricsCollector from markitect.assets.analyzer import ContentAnalyzer, SimilarityDetector, AssetMetricsCollector
from tests.test_utils import create_test_workspace, get_test_asset_config
class TestAssetOptimizationAndProcessing: class TestAssetOptimizationAndProcessing:
@@ -28,9 +27,9 @@ class TestAssetOptimizationAndProcessing:
def setup_method(self): def setup_method(self):
"""Set up test environment with sample assets.""" """Set up test environment with sample assets."""
self.temp_dir = tempfile.mkdtemp() self.temp_dir = create_test_workspace("asset_optimization")
self.assets_dir = Path(self.temp_dir) / "assets" self.assets_dir = self.temp_dir / "assets"
self.test_files_dir = Path(self.temp_dir) / "test_files" self.test_files_dir = self.temp_dir / "test_files"
self.assets_dir.mkdir() self.assets_dir.mkdir()
self.test_files_dir.mkdir() self.test_files_dir.mkdir()
@@ -39,11 +38,14 @@ class TestAssetOptimizationAndProcessing:
self.create_test_images() self.create_test_images()
self.create_test_documents() self.create_test_documents()
self.asset_manager = AssetManager(storage_path=self.assets_dir) # Use test asset configuration to ensure isolated registry
config = get_test_asset_config(self.temp_dir)
self.asset_manager = AssetManager(config)
def teardown_method(self): def teardown_method(self):
"""Clean up temporary directories.""" """Clean up temporary directories."""
shutil.rmtree(self.temp_dir) import shutil
shutil.rmtree(self.temp_dir, ignore_errors=True)
def create_test_images(self): def create_test_images(self):
"""Create test images with various properties.""" """Create test images with various properties."""

View File

@@ -8,8 +8,6 @@ Issue #144: Phase 3 - Advanced Features and Performance
""" """
import pytest import pytest
import tempfile
import shutil
from pathlib import Path from pathlib import Path
from unittest.mock import Mock, patch, MagicMock from unittest.mock import Mock, patch, MagicMock
import json import json
@@ -19,6 +17,7 @@ from markitect.assets import AssetManager
from markitect.assets.discovery import AssetDiscoveryEngine, MarkdownScanner, AssetReference from markitect.assets.discovery import AssetDiscoveryEngine, MarkdownScanner, AssetReference
from markitect.workspace import WorkspaceManager, WorkspaceTemplate from markitect.workspace import WorkspaceManager, WorkspaceTemplate
from markitect.assets.analytics import AssetAnalytics, UsageReport from markitect.assets.analytics import AssetAnalytics, UsageReport
from tests.test_utils import create_test_workspace, get_test_asset_config
class TestAutoDiscoveryAndWorkspace: class TestAutoDiscoveryAndWorkspace:
@@ -26,8 +25,8 @@ class TestAutoDiscoveryAndWorkspace:
def setup_method(self): def setup_method(self):
"""Set up test environment with sample markdown files and workspace.""" """Set up test environment with sample markdown files and workspace."""
self.temp_dir = tempfile.mkdtemp() self.temp_dir = create_test_workspace("auto_discovery")
self.project_dir = Path(self.temp_dir) / "test_project" self.project_dir = self.temp_dir / "test_project"
self.assets_dir = self.project_dir / "assets" self.assets_dir = self.project_dir / "assets"
self.docs_dir = self.project_dir / "docs" self.docs_dir = self.project_dir / "docs"
@@ -38,11 +37,14 @@ class TestAutoDiscoveryAndWorkspace:
self.create_test_markdown_files() self.create_test_markdown_files()
self.create_test_assets() self.create_test_assets()
self.asset_manager = AssetManager(storage_path=self.assets_dir) # Use test asset configuration to ensure isolated registry
config = get_test_asset_config(self.temp_dir)
self.asset_manager = AssetManager(config)
def teardown_method(self): def teardown_method(self):
"""Clean up temporary directories.""" """Clean up temporary directories."""
shutil.rmtree(self.temp_dir) import shutil
shutil.rmtree(self.temp_dir, ignore_errors=True)
def create_test_markdown_files(self): def create_test_markdown_files(self):
"""Create test markdown files with various asset references.""" """Create test markdown files with various asset references."""