Reject comment records from @CsvSource.value

[mpkorstanje]

We want to deprecate @CsvSource.textBlock in favour of .value[] (#5887).

Functionally value[] and textBlock are not identical. textBlock skips comment lines while value[] treats these as regular records. To make them identical, comment lines in value[] should be skipped too in v7. To ensure that enabling skipping comment lines does not result in lost test cases from value[] we preemptively reject comment lines in v6. That way when wen upgrading a passing suite from the latest v6.x to v7 there will be no comment lines in value[] making the upgrade safe.

Consequently, this also means that delimiter, quoteCharacter and commentCharacter must be distinct for both textBlock and value[].

Contributes to: #5017

---

I hereby agree to the terms of the JUnit Contributor License Agreement.

---

Definition of Done

• There are no TODOs left in the code
• Method preconditions are checked and documented in the method's Javadoc
• Coding conventions (e.g. for logging) have been followed
• Change is covered by automated tests including corner cases, errors, and exception handling
• Public API has Javadoc and @API annotations
• Change is documented in the User Guide and Release Notes

[testlens-app]

✅ All tests passed ✅

🏷️ Commit: cc1fbe3
▶️ Tests: 4515 executed
⚪️ Checks: 18/18 completed

---

Learn more about TestLens at testlens.app.