test: add unit tests for summary helpers

[Anwesha-codes-10]

Summary

Added unit tests for the helper functions in stargraph.cli._summary.

This PR adds isolated tests for _fmt_duration, _is_default, and _is_artifact_field to improve unit test coverage without changing runtime behavior.

Changes

• Added unit tests for _fmt_duration.
• Added unit tests for _is_default.
• Added unit tests for _is_artifact_field.
• Covered boundary cases and valid/invalid inputs.

Testing

• uv run pytest tests/unit/cli/test_summary.py -o addopts="" -q
• uv run ruff check tests/unit/cli/test_summary.py
• uv run pyright
• New tests added for new functionality
• Documentation updated (not applicable)

Related Issues

Closes #136

[se-jo-ma]

Code looks good.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[se-jo-ma]

Thanks for this, @Anwesha-codes-10 — the tests themselves look good. The only thing blocking the merge is the DCO sign-off on your commits.

Good news: we just migrated DCO enforcement to the DCO GitHub App (merged in #161), so you no longer have to rewrite history to fix this. Two ways to unblock:

Option 1 — rebase onto the latest main and sign off (recommended; also picks up the new CI):

git fetch origin
git rebase --signoff origin/main
git push --force-with-lease

Option 2 — keep your history, add a remediation commit: push any update to the branch and the DCO check will comment back the exact one-line command to sign off your existing commits without a rebase — just run it and push.

Either way, once each commit carries a Signed-off-by: Your Name <you@example.com> line, the DCO check goes green and this is good to merge. Shout if you hit any snags!

[se-jo-ma]

It looks like you submitted the file with git diff markings, please choose the diffs you want to keep and recommit.