Add bit more support to hopefully/eventually solve issue #15

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/dataformat/csv/CsvFactory.java

@@ -100,7 +100,7 @@ public String getFormatName() {
     @Override
     public MatchStrength hasFormat(InputAccessor acc) throws IOException
     {
-        // !!! TBI -- but how?
+        // !!! TBI [Issue#2] -- but how?
         return MatchStrength.INCONCLUSIVE;
     }
 

src/main/java/com/fasterxml/jackson/dataformat/csv/CsvMapper.java

@@ -202,12 +202,15 @@ protected CsvSchema _schemaFor(JavaType pojoType, LRUMap<JavaType,CsvSchema> sch
         CsvSchema.Builder builder = CsvSchema.builder();
         for (BeanPropertyDefinition prop : beanDesc.findProperties()) {
             // ignore setter-only properties:
-            if (prop.couldSerialize()) {
-                if (typed) {
-                    builder.addColumn(prop.getName(), _determineType(prop.getAccessor().getRawType()));
-                } else {
-                    builder.addColumn(prop.getName());
-                }
+            if (!prop.couldSerialize()) {
+                continue;
+            }
+            // TODO: [Issue#15]: need to handle unwrapped props?
+
+            if (typed) {
+                builder.addColumn(prop.getName(), _determineType(prop.getAccessor().getRawType()));
+            } else {
+                builder.addColumn(prop.getName());
             }
         }
         CsvSchema result = builder.build();

src/test/java/com/fasterxml/jackson/dataformat/csv/TestUnwrappingWithCSV.java

@@ -51,8 +51,7 @@ public void testSimpleUnwrappingRoundtrip() throws Exception
             .addColumn("x")
             .addColumn("y")
             .build();
-        Unwrapping wrapper = mapper.reader(schema).withType(Unwrapping.class).readValue(
-                CSV);
+        Unwrapping wrapper = mapper.reader(schema).withType(Unwrapping.class).readValue(CSV);
         assertNotNull(wrapper);
         assertNotNull(wrapper.location);
         assertEquals(15, wrapper.location.x);
@@ -67,12 +66,19 @@ public void testSimpleUnwrappingRoundtrip() throws Exception
      * Another simple test, but this time auto-generating Schema from
      * POJO.
      */
+    /* NOTE: the problem here is that Unwrapped properties should be further
+     * detected to find sub-properties -- but that information is not yet
+     * available via BeanProperty/POJOPropertyBuilder. But it needs to be
+     * made; and when this occurs, we can handle this case reasonably well.
+     */
     /*
     public void testSimpleWithAutoSchema() throws Exception
     {
         final String CSV = "Henry,1,2\n";
         CsvMapper mapper = mapperForCsv();
         CsvSchema schema = mapper.schemaFor(Unwrapping.class);
+System.err.println("SChema/auto -> "+schema);        
+        
         Unwrapping wrapper = mapper.reader(schema).withType(Unwrapping.class).readValue(CSV);
         assertNotNull(wrapper);
         assertNotNull(wrapper.location);