Skip to content

Add comprehensive tests for dynamic emissions module#1

Open
amussara wants to merge 1 commit intomainfrom
test-dynamic-emissions-advanced
Open

Add comprehensive tests for dynamic emissions module#1
amussara wants to merge 1 commit intomainfrom
test-dynamic-emissions-advanced

Conversation

@amussara
Copy link
Owner

  • 10 tests for _exponential_unlock_scalar (boundaries, monotonicity, production constants)
  • 7 tests for get_network_totals (tiered-only filtering, dedup, edge cases)
  • 12 tests for apply_dynamic_emissions_using_network_contributions (recycling, proportions, zero rewards, many miners)

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)

- 10 tests for _exponential_unlock_scalar (boundaries, monotonicity, production constants)
- 7 tests for get_network_totals (tiered-only filtering, dedup, edge cases)
- 12 tests for apply_dynamic_emissions_using_network_contributions (recycling, proportions, zero rewards, many miners)
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.

1 participant

Comments