From 9d3c6f3c81c2658a507807f245b37dc848532b74 Mon Sep 17 00:00:00 2001 From: tegwick Date: Fri, 24 Oct 2025 19:58:18 +0200 Subject: [PATCH] fix: isolate additional test files from production asset registry MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 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 --- tests/test_issue_144_asset_optimization.py | 16 +++++++++------- tests/test_issue_144_auto_discovery_workspace.py | 14 ++++++++------ 2 files changed, 17 insertions(+), 13 deletions(-) diff --git a/tests/test_issue_144_asset_optimization.py b/tests/test_issue_144_asset_optimization.py index 548d9a02..3e03e60e 100644 --- a/tests/test_issue_144_asset_optimization.py +++ b/tests/test_issue_144_asset_optimization.py @@ -8,8 +8,6 @@ Issue #144: Phase 3 - Advanced Features and Performance """ import pytest -import tempfile -import shutil from pathlib import Path from unittest.mock import Mock, patch, MagicMock 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.transformer import AssetTransformer, ThumbnailGenerator from markitect.assets.analyzer import ContentAnalyzer, SimilarityDetector, AssetMetricsCollector +from tests.test_utils import create_test_workspace, get_test_asset_config class TestAssetOptimizationAndProcessing: @@ -28,9 +27,9 @@ class TestAssetOptimizationAndProcessing: def setup_method(self): """Set up test environment with sample assets.""" - self.temp_dir = tempfile.mkdtemp() - self.assets_dir = Path(self.temp_dir) / "assets" - self.test_files_dir = Path(self.temp_dir) / "test_files" + self.temp_dir = create_test_workspace("asset_optimization") + self.assets_dir = self.temp_dir / "assets" + self.test_files_dir = self.temp_dir / "test_files" self.assets_dir.mkdir() self.test_files_dir.mkdir() @@ -39,11 +38,14 @@ class TestAssetOptimizationAndProcessing: self.create_test_images() 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): """Clean up temporary directories.""" - shutil.rmtree(self.temp_dir) + import shutil + shutil.rmtree(self.temp_dir, ignore_errors=True) def create_test_images(self): """Create test images with various properties.""" diff --git a/tests/test_issue_144_auto_discovery_workspace.py b/tests/test_issue_144_auto_discovery_workspace.py index feb4c686..522ca768 100644 --- a/tests/test_issue_144_auto_discovery_workspace.py +++ b/tests/test_issue_144_auto_discovery_workspace.py @@ -8,8 +8,6 @@ Issue #144: Phase 3 - Advanced Features and Performance """ import pytest -import tempfile -import shutil from pathlib import Path from unittest.mock import Mock, patch, MagicMock import json @@ -19,6 +17,7 @@ from markitect.assets import AssetManager from markitect.assets.discovery import AssetDiscoveryEngine, MarkdownScanner, AssetReference from markitect.workspace import WorkspaceManager, WorkspaceTemplate from markitect.assets.analytics import AssetAnalytics, UsageReport +from tests.test_utils import create_test_workspace, get_test_asset_config class TestAutoDiscoveryAndWorkspace: @@ -26,8 +25,8 @@ class TestAutoDiscoveryAndWorkspace: def setup_method(self): """Set up test environment with sample markdown files and workspace.""" - self.temp_dir = tempfile.mkdtemp() - self.project_dir = Path(self.temp_dir) / "test_project" + self.temp_dir = create_test_workspace("auto_discovery") + self.project_dir = self.temp_dir / "test_project" self.assets_dir = self.project_dir / "assets" self.docs_dir = self.project_dir / "docs" @@ -38,11 +37,14 @@ class TestAutoDiscoveryAndWorkspace: self.create_test_markdown_files() 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): """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): """Create test markdown files with various asset references."""