ryankert01 opened a new pull request, #925: URL: https://github.com/apache/mahout/pull/925
### Purpose of PR <!-- Describe what this PR does. --> Now when _qdp is not built, the tests will show as skipped with a helpful message instead of being silently ignored. before: all test pass no skip <img width="1946" height="538" alt="image" src="https://github.com/user-attachments/assets/50cd2886-5d3c-48b2-ac6c-57f7368876be"; /> now: ```bash $ uv run pytest testing/qdp -vrs ========================================================== test session starts =========================================================== platform linux -- Python 3.12.11, pytest-9.0.2, pluggy-1.6.0 -- /home/ryankert/mahout/.venv/bin/python cachedir: .pytest_cache rootdir: /home/ryankert/mahout configfile: pyproject.toml collected 0 items / 3 skipped ======================================================== short test summary info ========================================================= SKIPPED [1] testing/qdp/test_bindings.py:21: QDP extension not built. Run: uv run --active maturin develop --manifest-path qdp/qdp-python/Cargo.toml SKIPPED [1] testing/qdp/test_high_fidelity.py:27: QDP extension not built. Run: uv run --active maturin develop --manifest-path qdp/qdp-python/Cargo.toml SKIPPED [1] testing/qdp/test_numpy.py:26: QDP extension not built. Run: uv run --active maturin develop --manifest-path qdp/qdp-python/Cargo.toml =========================================================== 3 skipped in 1.59s ========================================================== ``` ### Related Issues or PRs <!-- Add links to related issues or PRs. --> <!-- - Closes #123 --> <!-- - Related to #123 --> ### Changes Made <!-- Please mark one with an "x" --> - [ ] Bug fix - [ ] New feature - [ ] Refactoring - [ ] Documentation - [ ] Test - [ ] CI/CD pipeline - [ ] Other ### Breaking Changes <!-- Does this PR introduce a breaking change? --> - [ ] Yes - [ ] No ### Checklist <!-- Please mark each item with an "x" when complete --> <!-- If not all items are complete, please open this as a **Draft PR**. Once all requirements are met, mark as ready for review. --> - [ ] Added or updated unit tests for all changes - [ ] Added or updated documentation for all changes - [ ] Successfully built and ran all unit tests or manual tests locally - [ ] PR title follows "MAHOUT-XXX: Brief Description" format (if related to an issue) - [ ] Code follows ASF guidelines -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
