add summary comment

[tgehrs]

Fix for #442 to create a separate summary comment from the truncated PR comment

[github-actions]

End-to-end public repo

Admin commands cheatsheet:

• /e2e (in approved PR review body): Trigger end-to-end tests on external contributions
• /invite (in comment): Invite the author & admins to the end-to-end private repo

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
coverage_comment
main.py
Project Total

This report was generated by python-coverage-comment-action

[ewjoachim]

Can you look at existing tests and tell me if it's easy to add a new one for this case ?

[tgehrs]

From my reading of the tests, this one seems to confirm that this functionality should work. It seems that mocks are rarely used, so I'm not sure how else to verify that it is called with None for max_files. If you have a place in mind you would like me to add a test please let me know and I will gladly do my best.

python-coverage-comment-action/tests/unit/test_template.py

Lines 327 to 357 in fafc0ae

	def test_template__no_max_files(coverage_obj_more_files, diff_coverage_obj_more_files):
	files, total = template.select_files(
	coverage=coverage_obj_more_files,
	diff_coverage=diff_coverage_obj_more_files,
	previous_coverage=None,
	max_files=25,
	)
	result = template.get_comment_markdown(
	coverage=coverage_obj_more_files,
	diff_coverage=diff_coverage_obj_more_files,
	previous_coverage=None,
	files=files,
	count_files=total,
	previous_coverage_rate=decimal.Decimal("0.92"),
	minimum_green=decimal.Decimal("79"),
	minimum_orange=decimal.Decimal("40"),
	repo_name="org/repo",
	pr_number=5,
	max_files=None,
	base_template=template.read_template_file("comment.md.j2"),
	marker="<!-- foo -->",
	subproject_id="foo",
	custom_template="""{% extends "base" %}
	{% block emoji_coverage_down %}:sob:{% endblock emoji_coverage_down %}
	""",
	)
	print(result)
	assert "The report is truncated" not in result
	assert "code.py" in result
	assert "other.py" in result

[ewjoachim]

(Ah sorry, missed you message).

I agree with you, adding a test there will probably mean more mocks, and limited value. Let's go !

[ewjoachim]

Released in https://github.com/py-cov-action/python-coverage-comment-action/releases/tag/v3.26 :)

Thank you very much for your contribution !