generic fromJson

fixes #32

CHANGELOG.md

@@ -1,3 +1,7 @@
+# 0.2.5
+
+Fixed a bug where generic json deserialization didn't apply generic parameters (https://github.com/rrousselGit/freezed/issues/32)
+
 # 0.2.4
 
 Make the generated interface a mixin to fix `prefer_mixin` lints (https://github.com/rrousselGit/freezed/issues/28)

lib/src/templates/concrete_template.dart

@@ -76,7 +76,7 @@ $copyWithPrototype
 
   String get redirectedFromJsonConstructor {
     if (!shouldGenerateJson) return '';
-    return 'factory ${constructor.redirectedName}.fromJson(Map<String, dynamic> json) = $concreteName.fromJson;';
+    return 'factory ${constructor.redirectedName}.fromJson(Map<String, dynamic> json) = $concreteName$genericsParameter.fromJson;';
   }
 
   String get concreteFromJsonConstructor {

pubspec.yaml

@@ -2,7 +2,7 @@ name: freezed
 description: >
   Code generation for immutable classes that has a simple syntax/API without
   compromising on the features.
-version: 0.2.4
+version: 0.2.5
 homepage: https://github.com/rrousselGit/freezed
 authors:
   - Remi Rousselet <darky12s@gmail.com>

test/integration/json.dart

@@ -33,3 +33,24 @@ abstract class Decorator with _$Decorator {
 
   factory Decorator.fromJson(Map<String, dynamic> json) => _$DecoratorFromJson(json);
 }
+
+@immutable
+abstract class Generic<T> with _$Generic<T> {
+  factory Generic(@DataConverter() T a) = _Generic<T>;
+
+  factory Generic.fromJson(Map<String, dynamic> json) => _$GenericFromJson<T>(json);
+}
+
+class DataConverter<T> implements JsonConverter<T, Object> {
+  const DataConverter();
+
+  @override
+  T fromJson(Object json) {
+    return json as T;
+  }
+
+  @override
+  Object toJson(T object) {
+    return object;
+  }
+}

test/json_test.dart

@@ -8,6 +8,12 @@ import 'common.dart';
 import 'integration/json.dart';
 
 Future<void> main() async {
+  test('generic json', () {
+    expect(
+      Generic<int>.fromJson(<String, dynamic>{'a': 42}),
+      Generic(42),
+    );
+  });
   test('single ctor + json can access properties/copyWith', () {
     final value = Single(42);