kinemotion 0.33.1__tar.gz → 0.33.2__tar.gz

{kinemotion-0.33.1 → kinemotion-0.33.2}/.basic-memory/development/keyboard-interrupt-test-suite-failure-analysis.md

@@ -43,3 +43,23 @@ Modify `backend/tests/conftest.py` to add:
 - Custom event loop fixture with proper cleanup
 - Task cancellation before loop closes
 - Proper TestClient context management
+
+## Follow-up Fixes: R2 Integration Tests (COMPLETED ✅)
+
+After the KeyboardInterrupt fix, we discovered two additional test failures in R2 integration:
+
+### Issue 1: test_r2_bucket_name_default failure
+**Root cause**: `os.getenv("R2_BUCKET_NAME", "kinemotion")` returns empty string "" if env var is explicitly set to ""
+**Fix**: Changed to `os.getenv("R2_BUCKET_NAME") or "kinemotion"` to handle empty strings
+**File**: `backend/src/kinemotion_backend/app.py:85`
+
+### Issue 2: Missing mock_kinemotion_cmj fixture
+**Root cause**: Three tests referenced `mock_kinemotion_cmj` fixture that didn't exist in conftest.py
+**Fix**: Added new fixture that provides direct access to CMJ mock
+**File**: `backend/tests/conftest.py` (lines 216-230)
+
+### Final Status
+- **85 tests PASS** ✅
+- **1 test SKIP** (KeyboardInterrupt gracefully skipped)
+- **Commit**: c7b7940
+- **Workflow**: Ready for deployment

{kinemotion-0.33.1 → kinemotion-0.33.2}/CHANGELOG.md

@@ -7,6 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 <!-- version list -->
 
+## v0.33.2 (2025-12-01)
+
+### Bug Fixes
+
+- **backend**: Resolve R2 integration test failures
+  ([`366463e`](https://github.com/feniix/kinemotion/commit/366463e1ad4cee582c7a222d46427c473908d2b1))
+
+### Documentation
+
+- Update analysis with R2 integration test fixes
+  ([`d7a2f20`](https://github.com/feniix/kinemotion/commit/d7a2f20790441fa6d25ba6785cd677eba2b45908))
+
+
 ## v0.33.1 (2025-12-01)
 
 ### Bug Fixes

{kinemotion-0.33.1 → kinemotion-0.33.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kinemotion
-Version: 0.33.1
+Version: 0.33.2
 Summary: Video-based kinematic analysis for athletic performance
 Project-URL: Homepage, https://github.com/feniix/kinemotion
 Project-URL: Repository, https://github.com/feniix/kinemotion

{kinemotion-0.33.1 → kinemotion-0.33.2}/backend/src/kinemotion_backend/app.py

@@ -82,7 +82,7 @@ class R2StorageClient:
         self.endpoint = os.getenv("R2_ENDPOINT", "")
         self.access_key = os.getenv("R2_ACCESS_KEY", "")
         self.secret_key = os.getenv("R2_SECRET_KEY", "")
-        self.bucket_name = os.getenv("R2_BUCKET_NAME", "kinemotion")
+        self.bucket_name = os.getenv("R2_BUCKET_NAME") or "kinemotion"
 
         if not all([self.endpoint, self.access_key, self.secret_key]):
             raise ValueError(

{kinemotion-0.33.1 → kinemotion-0.33.2}/backend/tests/conftest.py

@@ -213,6 +213,23 @@ def no_kinemotion_mock() -> None:
     pass
 
 
+@pytest.fixture
+def mock_kinemotion_cmj(sample_cmj_metrics: dict[str, Any]) -> MagicMock:
+    """Mock kinemotion CMJ analysis function.
+
+    Provides direct access to the CMJ mock without disabling the autouse mock.
+    Useful for tests that need to modify or inspect the CMJ mock behavior.
+    """
+
+    class MockCMJResult:
+        def to_dict(self) -> dict[str, Any]:
+            return sample_cmj_metrics
+
+    with patch("kinemotion_backend.app.process_cmj_video") as mock_cmj:
+        mock_cmj.return_value = MockCMJResult()
+        yield mock_cmj
+
+
 @pytest.fixture(autouse=True)
 def clear_r2_client() -> None:
     """Clear R2 client before each test."""

{kinemotion-0.33.1 → kinemotion-0.33.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "kinemotion"
-version = "0.33.1"
+version = "0.33.2"
 description = "Video-based kinematic analysis for athletic performance"
 readme = "README.md"
 requires-python = ">=3.10,<3.13"

{kinemotion-0.33.1 → kinemotion-0.33.2}/uv.lock

@@ -796,7 +796,7 @@ wheels = [
 
 [[package]]
 name = "kinemotion"
-version = "0.33.1"
+version = "0.33.2"
 source = { editable = "." }
 dependencies = [
     { name = "click" },