This is an automated email from the ASF dual-hosted git repository.

garydgregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-csv.git

commit d4d4154454b43c46aff65ed75c721605eafb142b
Author: Gary Gregory <[email protected]>
AuthorDate: Sat Jun 13 12:43:32 2026 +0000

    Refactor some magic strings in
    CSVPrinterTest.testQuoteCharEscapedWithQuoteModeNone()
---
 src/test/java/org/apache/commons/csv/CSVPrinterTest.java | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/test/java/org/apache/commons/csv/CSVPrinterTest.java 
b/src/test/java/org/apache/commons/csv/CSVPrinterTest.java
index b5878221..16901c4e 100644
--- a/src/test/java/org/apache/commons/csv/CSVPrinterTest.java
+++ b/src/test/java/org/apache/commons/csv/CSVPrinterTest.java
@@ -1802,9 +1802,11 @@ class CSVPrinterTest {
     void testQuoteCharEscapedWithQuoteModeNone() throws IOException {
         final CSVFormat format = 
CSVFormat.DEFAULT.builder().setQuote('"').setEscape('?').setQuoteMode(QuoteMode.NONE).get();
         final StringWriter sw = new StringWriter();
+        final String col1 = "\"abc";
+        final String col2 = "x\"y";
         try (CSVPrinter printer = new CSVPrinter(sw, format)) {
-            printer.printRecord("\"abc", "x\"y");
-            printer.printRecord(new StringReader("\"abc"), new 
StringReader("x\"y"));
+            printer.printRecord(col1, col2);
+            printer.printRecord(new StringReader(col1), new 
StringReader(col2));
         }
         assertEquals("?\"abc,x?\"y" + RECORD_SEPARATOR + "?\"abc,x?\"y" + 
RECORD_SEPARATOR, sw.toString());
         // The emitted records must read back as the original values.
@@ -1812,8 +1814,8 @@ class CSVPrinterTest {
             final List<CSVRecord> records = parser.getRecords();
             assertEquals(2, records.size());
             for (final CSVRecord record : records) {
-                assertEquals("\"abc", record.get(0));
-                assertEquals("x\"y", record.get(1));
+                assertEquals(col1, record.get(0));
+                assertEquals(col2, record.get(1));
             }
         }
     }

Reply via email to