Apache/commons-csv
1
0
Fork 0
mirror of https://github.com/apache/commons-csv.git synced 2025-02-27 13:34:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

changing the exception message to be more meaningful

Browse Source
This commit is contained in:
gbidsilva 2023-09-06 18:42:13 +05:30
parent 710459843c
commit eb95e6dff7
2 changed files with 14 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/apache/commons/csv/CSVFormat.java
View File

@ -2449,7 +2449,7 @@ public final class CSVFormat implements Serializable {
}
if (escapeCharacter == null && quoteMode == QuoteMode.NONE) {
throw new IllegalArgumentException("No quotes mode set but no escape character is set");
throw new IllegalArgumentException("Quote mode set to NONE but no escape character is set");
}
// Validate headers

13
src/test/java/org/apache/commons/csv/CSVFormatTest.java
View File

@ -1477,4 +1477,17 @@ public class CSVFormatTest {
final CSVFormat formatWithRecordSeparator = CSVFormat.DEFAULT.withSystemRecordSeparator();
assertEquals(System.lineSeparator(), formatWithRecordSeparator.getRecordSeparator());
}
@Test
public void testQuoteModeNoneShouldReturnMeaningfulExceptionMessage() {
Exception exception = assertThrows(IllegalArgumentException.class, () -> {
CSVFormat.DEFAULT.builder()
.setHeader("Col1", "Col2", "Col3", "Col4")
.setQuoteMode(QuoteMode.NONE)
.build();
});
String actualMessage = exception.getMessage();
String expectedMessage = "Quote mode set to NONE but no escape character is set";
assertEquals(expectedMessage, actualMessage);
}
}