SetMultimap should be deserialized to a LinkedHashMultimap by default (#80)

Author: cowtowncoder

guava/src/main/java/com/fasterxml/jackson/datatype/guava/GuavaDeserializers.java

@@ -247,7 +247,7 @@ public JsonDeserializer<?> findMapLikeDeserializer(MapLikeType type,
             }
 
             // TODO: Remove the default fall-through once all implementations are covered.
-            return new HashMultimapDeserializer(type, keyDeserializer, elementTypeDeserializer,
+            return new LinkedHashMultimapDeserializer(type, keyDeserializer, elementTypeDeserializer,
                     elementDeserializer);
         }
 

guava/src/test/java/com/fasterxml/jackson/datatype/guava/TestMultimaps.java

@@ -319,6 +319,13 @@ public void testIssue67() throws IOException
         assertEquals(Maps.immutableEntry("a", 7), iterator.next());
         assertEquals(Maps.immutableEntry("a", 8), iterator.next());
     }
+
+    // [Issue#25]
+    public void testDefaultSetMultiMap() throws IOException {
+        @SuppressWarnings("unchecked")
+        SetMultimap<String, String> map = (SetMultimap<String, String>) MAPPER.readValue(STRING_STRING_MULTIMAP, new TypeReference<SetMultimap>() {});
+        assertTrue(map instanceof LinkedHashMultimap);
+    }
 
     public void testPolymorphicValue() throws IOException {
         ImmutableMultimapWrapper input = new ImmutableMultimapWrapper(ImmutableMultimap.of("add", new AddOp(3, 2), "mul", new MulOp(4, 6)));