Fix bug that set-like types aren't serialized

dahlia · dahlia · commit 098b1bd6538c · 2017-07-06T00:44:27.000+09:00
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -1,6 +1,17 @@
 Changelog
 =========
 
+Version 0.4.2
+-------------
+
+To be released.
+
+- Fixed a serialization bug that other set-like (i.e. ``collections.Set``) types
+  than Python built-in ``set`` hadn't been reduced to simpler forms so that they
+  can be encoded to JSON.
+
+
+
 Version 0.4.1
 -------------
 
diff --git a/nirum/serialize.py b/nirum/serialize.py
@@ -66,7 +66,7 @@ def serialize_meta(data):
         d = data.isoformat()
     elif isinstance(data, decimal.Decimal) or isinstance(data, uuid.UUID):
         d = str(data)
-    elif isinstance(data, set) or isinstance(data, list):
+    elif isinstance(data, collections.Set) or isinstance(data, list):
         d = [serialize_meta(e) for e in data]
     elif isinstance(data, collections.Mapping):
         d = [
diff --git a/tests/serialize_test.py b/tests/serialize_test.py
@@ -101,6 +101,8 @@ def test_serialize_meta_set_of_record(fx_record_type, fx_unboxed_type,
     serialize_result = serialize_meta({record, record2})
     assert record.__nirum_serialize__() in serialize_result
     assert record2.__nirum_serialize__() in serialize_result
+    assert (sorted(serialize_meta(frozenset([record, record2])), key=repr) ==
+            sorted(serialize_result, key=repr))
 
 
 def test_serialize_meta_map(fx_point):
