Skip to content

test: add comprehensive test coverage for scoring and collateral modules#191

Closed
amussara wants to merge 1 commit intoentrius:mainfrom
amussara:add-scoring-tests
Closed

test: add comprehensive test coverage for scoring and collateral modules#191
amussara wants to merge 1 commit intoentrius:mainfrom
amussara:add-scoring-tests

Conversation

@amussara
Copy link

Add dedicated test suites for the core scoring logic:

  • test_scoring.py: Tests for score_pull_request, calculate_base_score, calculate_pr_multipliers, time_decay, issue_multiplier, uniqueness, spam penalty, and finalize_miner_scores
  • test_collateral.py: Tests for open PR collateral score calculation including edge cases and tier configuration handling

These tests cover critical scoring paths that previously had no dedicated test coverage, improving reliability of the reward system.

Summary

Related Issues

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other (describe below)

Testing

  • Tests added/updated
  • Manually tested

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Changes are documented (if applicable)

Add dedicated test suites for the core scoring logic:
- test_scoring.py: Tests for score_pull_request, calculate_base_score,
  calculate_pr_multipliers, time_decay, issue_multiplier, uniqueness,
  spam penalty, and finalize_miner_scores
- test_collateral.py: Tests for open PR collateral score calculation
  including edge cases and tier configuration handling

These tests cover critical scoring paths that previously had no
dedicated test coverage, improving reliability of the reward system.
@anderdc anderdc closed this Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants