guan404ming opened a new pull request, #566:
URL: https://github.com/apache/mahout/pull/566

   ### Purpose of PR
     Update testing setup to be more closely aligned with pytest conventions 
and Poetry dependency management.
   
   ### Linked Issues
   Add links to related issues.
   
   ### Changes Made
   - [ ] Bug fix
   - [x] New feature
   - [ ] Documentation update
   
   ### Important ToDos
   Please mark each with an "x"
   
   A GitHub issue exists (if not, please create one) 
[https://github.com/apache/mahout/issues]
   - [ ] Title of PR is "Issue #XXXX: Brief Description of Changes" where XXXX 
is the GitHub issue number.
   - [x] Created unit tests where appropriate (enhanced existing tests with 
pytest conventions)
   - [x] Added correct licenses on newly added files
   - [ ] Assigned GitHub issue to self
   - [x] Added documentation in ScalaDocs/JavaDocs and to the website (updated 
testing README)
   - [x] Successfully built and ran all unit tests, verified that all tests 
pass locally
   
   If all of these items are not yet complete, but you still feel it is 
appropriate to open a PR, please open it as a **Draft PR**
    instead.
   Once all requirements are met, you can mark it as ready for review.
   
   ### Breaking Changes
   Does this PR introduce a breaking change?
   - [ ] Yes
   - [x] No
   
   ### Testing & Verification
   Describe how you tested the changes.
   - [x] Unit tests added (enhanced existing tests with pytest features)
   - [x] Manually tested (verified all test commands work correctly)
   
   **Testing performed:**
   - All tests pass with `poetry run pytest`
   - Parametrized tests work correctly for multiple quantum states
   - Test markers (`unit`, `integration`) function properly
   - Test collection and execution work with new structure
   
   ### Checklist
   - [ ] The title follows the format "MAHOUT-XXXX Brief Description"
   - [ ] GitHub issue is created
   - [x] Code follows ASF guidelines
   
   **Summary of Changes:**
   - Enhanced test structure with pytest class-based organization
   - Added parametrized tests for multiple initial quantum states
   - Created proper test fixtures and markers
   - Updated testing README


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