From fc2431332b73c1b139ae8bef65519507882be901 Mon Sep 17 00:00:00 2001 From: Michael Minella Date: Thu, 18 Sep 2014 10:41:35 -0500 Subject: [PATCH] Added test to DelimitedLineTokenizer based on StackOverflow Question A question was asked on StackOverflow (http://stackoverflow.com/questions/25914945/spring-batch-delimitedlinetokenizer-with-line-breaks-newline-in-csv) in which a row wasn't being parsed as expected. We didn't have a test for that specific scenario so this commit adds one. --- .../file/transform/DelimitedLineTokenizerTests.java | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/spring-batch-infrastructure/src/test/java/org/springframework/batch/item/file/transform/DelimitedLineTokenizerTests.java b/spring-batch-infrastructure/src/test/java/org/springframework/batch/item/file/transform/DelimitedLineTokenizerTests.java index b4fa1068a..110dc8cb9 100644 --- a/spring-batch-infrastructure/src/test/java/org/springframework/batch/item/file/transform/DelimitedLineTokenizerTests.java +++ b/spring-batch-infrastructure/src/test/java/org/springframework/batch/item/file/transform/DelimitedLineTokenizerTests.java @@ -16,12 +16,12 @@ package org.springframework.batch.item.file.transform; +import org.junit.Test; + import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; -import org.junit.Test; - public class DelimitedLineTokenizerTests { @@ -365,4 +365,12 @@ public class DelimitedLineTokenizerTests { assertEquals("c", line.readString("bar")); } + @Test + public void testTokenizeOverMultipleLines() { + tokenizer = new DelimitedLineTokenizer(";"); + FieldSet line = tokenizer.tokenize("value1;\"value2\nvalue2cont\";value3;value4"); + assertEquals(4, line.getFieldCount()); + assertEquals("value2\nvalue2cont", line.readString(1)); + } + } \ No newline at end of file