more tuning for #33 fix

FasterXML · Mar 21, 2014 · 785aaba · 785aaba
1 parent 7b02792
commit 785aaba
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/main/java/com/fasterxml/jackson/dataformat/csv/impl/CsvWriter.java b/src/main/java/com/fasterxml/jackson/dataformat/csv/impl/CsvWriter.java
@@ -274,7 +274,7 @@ public void endRow() throws IOException
             _lastBuffered = -1;
             for (int i = _nextColumnToWrite; i <= last; ++i) {
                 BufferedValue value = _buffered[i];
-                if (i > 0) {
+                if (i > _nextColumnToWrite) {
                     appendColumnSeparator();
                 }
                 if (value != null) {

diff --git a/...on/dataformat/csv/failing/TestWriter.java → ...rmat/csv/TestWriterWithMissingValues.java b/...on/dataformat/csv/failing/TestWriter.java → ...rmat/csv/TestWriterWithMissingValues.java
@@ -1,17 +1,16 @@
-package com.fasterxml.jackson.dataformat.csv.failing;
+package com.fasterxml.jackson.dataformat.csv;
 
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.ObjectWriter;
 import com.fasterxml.jackson.dataformat.csv.CsvMapper;
 import com.fasterxml.jackson.dataformat.csv.CsvSchema;
-import com.fasterxml.jackson.dataformat.csv.ModuleTestBase;
 import com.google.common.collect.ImmutableMap;
 import org.junit.Test;
 
 import java.lang.String;
 
 // [Issue#33]
-public class TestWriter extends ModuleTestBase
+public class TestWriterWithMissingValues extends ModuleTestBase
 {
     private final CsvSchema SCHEMA = new CsvSchema.Builder()
         .addColumn("timestamp", CsvSchema.ColumnType.STRING)