fix: clean room test for python

Author: yhseanhsu

Makefile

@@ -124,10 +124,27 @@ install-py: uninstall-py clean-py wasm
 	echo "$(BOLD)📦 Phase 3: Installing $$WHEEL_PATH...$(RESET)"; \
 	if [ -n "$(UV)" ]; then $(UV) pip install $$WHEEL_PATH; \
 	else $(PYTHON) -m pip install $$WHEEL_PATH; fi
-	@echo "$(BOLD)🧪 Phase 4: Integration Test (Isolated from Source)...$(RESET)"
-	@cd $(BINDING_PY)/tests && (if [ -n "$(UV)" ]; then $(UV) run python -m pytest . -s; \
-	else $(PYTHON) -m pytest . -s; fi)
-	@echo "$(GREEN)$(BOLD)✅ Success: Python Wheel verified.$(RESET)"
+	@echo "$(BOLD)🧪 Phase 4: Integration Test (Clean-Room)...$(RESET)"
+	@TEMP_DIR=$$(mktemp -d); \
+		echo "Temp Test Directory: $$TEMP_DIR"; \
+		cp -r $(BINDING_PY)/tests $$TEMP_DIR/tests; \
+		mkdir -p $$TEMP_DIR/tests/samples; \
+		cp -r $(PROJECT_ROOT)/j2735codec/core/$(J2735_REV)/samples/jer/* $$TEMP_DIR/tests/samples/; \
+		cd $$TEMP_DIR; \
+		echo "Contents of temp directory:"; \
+		ls -R; \
+		if [ -n "$(UV)" ]; then \
+			$(UV) init; \
+			$(UV) venv; \
+			WHEEL_PATH=$$(realpath $(PROJECT_ROOT)/$(BINDING_PY)/dist/*.whl | head -n1); \
+			$(UV) pip install --upgrade pip pytest psutil $$WHEEL_PATH; \
+			$(UV) run python -m pytest tests -s; \
+		else \
+			$(PYTHON) -m pip install --upgrade pip pytest; \
+			$(PYTHON) -m pytest tests -s; \
+		fi; \
+		rm -rf $$TEMP_DIR; \
+		echo "$(GREEN)$(BOLD)✅ Success: Python Wheel verified in isolation.$(RESET)"
 
 clean-py:
 	rm -rf $(PY_DIST_DIR) $(BINDING_PY)/*.egg-info
@@ -151,6 +168,7 @@ clean-js:
 	@echo "🧹 Cleaning Node artifacts..."
 	@rm -rf $(JS_DIST_DIR)
 	@find $(BINDING_JS) -name "*.tgz" -delete
+	@find . -type d -name "node_modules" -exec rm -rf {} +
 
 install-js: clean-js build-js
 	@echo "$(BOLD)🧪 Phase 1: Local Source Test...$(RESET)"
@@ -167,6 +185,7 @@ install-js: clean-js build-js
 		$(NPM) init -y > /dev/null && \
 		echo "📦 Installing tarball..." && \
 		$(NPM) install $$FULL_TARBALL_PATH > /dev/null && \
+		$(NPM) install vitest --no-save > /dev/null && \
 		cp -r $(PROJECT_ROOT)/$(BINDING_JS)/tests ./tests && \
 		mkdir -p samples && \
 		cp -r $(PROJECT_ROOT)/j2735codec/core/$(J2735_REV)/samples/jer/* ./samples/ && \

etx/examples/python/pyproject.toml

@@ -11,7 +11,6 @@ requires-python = ">=3.9"
 dependencies = [
     "j2735codec",
     "paho-mqtt>=2.1.0",
-    "protobuf>=6.33.4",
     "python-geohash>=0.8.5",
     "requests>=2.32.5",
 ]

j2735codec/bindings/python/pyproject.toml

@@ -10,6 +10,7 @@ authors = [
 requires-python = ">=3.9"
 dependencies = [
     "wasmtime>=25.0.0",
+    "protobuf>=6.33.4",
 ]
 
 [build-system]
@@ -18,12 +19,14 @@ build-backend = "hatchling.build"
 
 [dependency-groups]
 dev = [
-    "protobuf>=6.33.4",
     "psutil>=7.2.1",
     "pytest>=8.3.5",
     "ruff>=0.14.11",
 ]
 
+[tool.hatch.build]
+ignore-vcs=true
+
 [tool.hatch.build.targets.wheel]
 packages = ["src/j2735codec"]
 

j2735codec/bindings/python/tests/test_codec.py

@@ -10,12 +10,29 @@
 from j2735codec import EncodingRules, PDUTypes, Codec, GeoRoutedHeader
 
 def get_sample_dir():
-    """Finds the core samples directory relative to this test file."""
+    """Finds the core samples directory for tests.
+
+    Tries these locations in order:
+    1. ./samples relative to current working directory (clean-room test)
+    2. core/j2735_202409/samples/jer relative to project root (source tree)
+    """
+    # 1. Check if samples exist relative to tests/ (clean-room / packaged test)
+    cwd_samples = Path(__file__).parent / "samples"
+    if cwd_samples.exists():
+        return cwd_samples.resolve()
+
+    # 2. Check relative to project root (source tree / monorepo)
     current_file = Path(__file__).resolve()
-    # Path navigation based on your monorepo structure
+    # Adjust 'parents[3]' if your file depth changes
     project_root = current_file.parents[3] 
-    sample_path = project_root / "core" / "j2735_202409" / "samples" / "jer"
-    return sample_path
+    repo_samples = project_root / "core" / "j2735_202409" / "samples" / "jer"
+
+    if repo_samples.exists():
+        return repo_samples.resolve()
+
+    raise FileNotFoundError(
+        "Samples directory not found in either packaged samples folder or source tree."
+    )
 
 def get_sample_files():
     sample_dir = get_sample_dir()