test: cover remaining contributor paths

[vchrombie]

how much coverage is too much coverage

Pull Request

Proposed Changes

Readiness Checklist

Author/Contributor

• If documentation is needed for this change, has that been included in this pull request
• run make lint and fix any issues that you have introduced
• run make test and ensure you have test coverage for the lines you are introducing
• If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

• Label as either bug, documentation, enhancement, infrastructure, maintenance or breaking

[Copilot]

Pull request overview

This PR expands unit test coverage for remaining contributor-related execution paths and a couple of supporting utilities/docs.

Changes:

• Add tests for get_contributors() edge cases (no commits in range, exception handling) and main() execution paths.
• Add tests for env.get_int_env_var() invalid parsing and contributor_stats (__repr__, sponsor info error path).
• Adjust README badges formatting to be on separate lines.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
test_env.py	Adds coverage for invalid integer env var parsing.
test_contributors.py	Adds coverage for additional contributor filtering/error paths and script/main execution.
test_contributor_stats.py	Adds coverage for __repr__ and sponsor info error handling.
README.md	Splits badges onto separate lines for readability.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

[zkoppert]

how much coverage is too much coverage

anything over 100% 🤣

[zkoppert]

@vcrombie Would you be able to take a look at the merge conflicts and update the base branch for this PR? Updating the base branch should resolve the linter issues failing in the checks section for you.

[vchrombie]

Sure, I'll rebase it later in the evening.

[vchrombie]

@zkoppert I rebased the branch, please let me know what do you think about the PR?

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (1)

test_contributors.py:187

• The docstring says this test verifies commit searching "only with start date", but get_contributors() only checks commits when both start_date and end_date are provided. Update the docstring/test intent accordingly (e.g., assert repo.commits is not called when end_date is empty).

    def test_get_contributors_no_commit_end_date(self, mock_contributor_stats):
        """
        Test the get_contributors does the search of commits only with start date
        """

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

[vchrombie]

Guess I am really pushing the boundaries of copilot here.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

[zkoppert]

🚀