#151: added duplicate value check

Author: jorre127

example/analysis_options.yaml

@@ -3,7 +3,6 @@ analyzer:
     missing_required_param: error
     missing_return: error
     todo: ignore
-    sdk_version_async_exported_from_core: ignore
   exclude:
     - '**.g.dart'
     - 'lib/util/locale/**'

example/lib/navigator/main_navigator.dart

@@ -30,13 +30,10 @@ class MainNavigatorWidgetState extends State<MainNavigatorWidget> {
 
   @override
   Widget build(BuildContext context) {
-    return WillPopScope(
-      onWillPop: _willPop,
-      child: Navigator(
-        key: navigationKey,
-        initialRoute: HomeScreen.routeName,
-        onGenerateRoute: onGenerateRoute,
-      ),
+    return Navigator(
+      key: navigationKey,
+      initialRoute: HomeScreen.routeName,
+      onGenerateRoute: onGenerateRoute,
     );
   }
 
@@ -50,14 +47,6 @@ class MainNavigatorWidgetState extends State<MainNavigatorWidget> {
     }
   }
 
-  Future<bool> _willPop() async {
-    final currentState = navigationKey.currentState;
-    if (currentState == null) {
-      return true;
-    }
-    return !await currentState.maybePop();
-  }
-
   void closeDialog() => Navigator.of(context, rootNavigator: true).pop();
 
   void goBack<T>({result}) => navigationKey.currentState?.pop(result);

example/model_generator/enums.yaml

@@ -75,3 +75,12 @@ DoubleStatus:
     status_3:
       properties:
         value: 3.3
+
+DuplicateEnum:
+  path: status
+  type: enum
+  values:
+    one: 1
+    two: 2
+    three: 3
+    #four: 3 # Duplicate value throws error

lib/config/yml_generator_config.dart

@@ -226,6 +226,11 @@ class YmlGeneratorConfig {
           );
         }
 
+        _checkDuplicateEnumValue(
+          fields: fields,
+          enumName: key,
+        );
+
         final enumModel = EnumModel(
           addJsonValueToProperties: value['use_default_json_value'] ?? true,
           generateExtension: value['generate_extension'] == true,
@@ -286,6 +291,24 @@ class YmlGeneratorConfig {
     });
   }
 
+  void _checkDuplicateEnumValue({
+    required List<EnumField> fields,
+    required String enumName,
+  }) {
+    final seenKeys = <String>{};
+
+    for (final field in fields) {
+      final jsonValue = field.values
+          .firstWhereOrNull((field) => field.propertyName == 'jsonValue');
+      if (jsonValue == null) continue;
+      final key = '${jsonValue.value}-${jsonValue.propertyName}';
+      if (!seenKeys.add(key)) {
+        throw Exception(
+            'Duplicate jsonValue ${jsonValue.value} found on field ${field.name} on enum $enumName');
+      }
+    }
+  }
+
   Field getField(String name, YamlMap property,
       {required bool disallowNullForDefaults}) {
     try {

test/writer/enum_model_reader_test.dart

@@ -388,6 +388,21 @@ Gender:
   values:
     MALE: 1
     FEMALE: female
+""",
+            ));
+
+    test(
+        'Enum cant have duplicate json value',
+        () => testEnumError(
+              expectedError:
+                  'Exception: Duplicate jsonValue 1 found on field female on enum Gender',
+              enumYml: """
+Gender:
+  path: user/person/
+  type: enum
+  values:
+    MALE: 1
+    FEMALE: 1
 """,
             ));
   });