common_test: empty heading resulting with no heading

[u3s]

• allow user to actually specify no heading in print output

[github-actions]

CT Test Results

  2 files   57 suites   1h 15m 23s ⏱️
451 tests 436 ✅ 15 💤 0 ❌
485 runs  467 ✅ 18 💤 0 ❌

Results for commit 09ea2f4.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

• Complete CT logs (Download Logs)
• HTML Documentation (Download HTML Docs)
• Windows Installer

// Erlang/OTP Github Action Bot

[Copilot]

Pull Request Overview

This PR modifies the tc_print function in Common Test to allow users to specify no heading in print output. When an empty string is provided as the heading, the output will exclude the header and use single line break instead of double.

• Updates heading handling logic to treat empty strings as "no heading" requests
• Changes line break formatting based on whether a heading is present

Comments suppressed due to low confidence (1)

lib/common_test/src/ct_logs.erl:551

• [nitpick] The variable name 'Parts' is somewhat generic. Consider a more descriptive name like 'FormatParts' or 'OutputParts' to clarify its purpose as components of the formatted output string.

            Parts =

[Copilot]

The hardcoded line break pattern creates inconsistency. Consider defining constants or using a more systematic approach to handle line break variations, as this logic may need to be maintained consistently across similar formatting functions.

Suggested change

	"" -> [Format, "\n"];
	_ -> [get_header(Heading),Format,"\n\n"]
	"" -> [Format, ?LINE_BREAK];
	_ -> [get_header(Heading),Format,?LINE_BREAK, ?LINE_BREAK]