alisha-1000 opened a new pull request, #1084:
URL: https://github.com/apache/mahout/pull/1084

   ### Related Issues
   Closes #1058
   
   ---
   
   ### Changes
   - [ ] Bug fix
   - [ ] New feature
   - [ ] Refactoring
   - [ ] Documentation
   - [x] Test
   - [ ] CI/CD pipeline
   - [ ] Other
   
   ---
   
   ### Why
   
   The current test coverage for `qumat/qdp.py` does not fully validate the
   fallback behavior when the native `_qdp` extension is unavailable.
   
   This module contains import-time logic that dynamically assigns stub
   implementations when the native extension is not installed. Ensuring
   this fallback mechanism works correctly is important for reliability
   in environments where the extension has not been built.
   
   ---
   
   ### How
   
   Added unit tests to explicitly exercise and validate:
   
   - Import fallback behavior when `_qdp` cannot be imported
   - Emission of `ImportWarning` during fallback
   - Stub class generation via `_make_stub`
   - Proper `ImportError` raised on stub instantiation
   - Correct module exports via `__all__`
   
   These tests ensure that all functional code paths in `qumat/qdp.py`
   are executed and validated.
   
   Coverage impact:
   - `qumat/qdp.py` improved to ~93%, with all functional logic paths exercised.
   
   ---
   
   ## Checklist
   
   - [x] Added or updated unit tests for all changes
   - [ ] Added or updated documentation for all changes


-- 
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]

Reply via email to