v2.7.3 - Fix/enum const filter

CHANGELOG.md

@@ -1,3 +1,18 @@
+## 2.7.3
+
+- `lib/src/reflection_factory_builder.dart`:
+  - `_EnumTree<T>`: Filtered enum entries to include only those marked as `const` when generating `_valuesByName` map.
+
+- `pubspec.yaml`:
+  - Updated dependencies:
+    - `build`: ^4.0.3
+    - `analyzer`: ^8.4.1
+    - `dart_style`: ^3.1.3
+  - Updated dev_dependencies:
+    - `build_runner`: ^2.10.4
+    - `build_test`: ^3.5.4
+    - `test`: ^1.29.0
+
 ## 2.7.2
 
 - `BasicDartType`: added `function`.

bump.sh

@@ -0,0 +1,11 @@
+#!/bin/bash
+
+APIKEY=$1
+shift  # remove the first argument (API key) from "$@"
+
+## dart pub global activate dart_bump
+
+dart_bump . \
+  --extra-file "lib/src/reflection_factory_base.dart=String\\s+VERSION\\s+=\\s+['\"](.*)['\"]" \
+  --api-key $APIKEY \
+  "$@"

lib/src/reflection_factory_base.dart

@@ -20,7 +20,7 @@ import 'reflection_factory_utils.dart';
 /// Class with all registered reflections ([ClassReflection]).
 class ReflectionFactory {
   // ignore: constant_identifier_names
-  static const String VERSION = '2.7.2';
+  static const String VERSION = '2.7.3';
 
   static final ReflectionFactory _instance = ReflectionFactory._();
 

lib/src/reflection_factory_builder.dart

@@ -1405,6 +1405,7 @@ class _EnumTree<T> extends RecursiveElementVisitor2<T> {
 
     var enumsEntries = entries.entries
         .where((e) => e.value.thisType == enumType)
+        .where((e) => e.value.isConst)
         .sortedBy((e) => e.key)
         .toList();
 

pubspec.yaml

@@ -1,15 +1,15 @@
 name: reflection_factory
 description: Allows Dart reflection with an easy approach, even for third-party classes, using code generation portable for all Dart platforms.
-version: 2.7.2
+version: 2.7.3
 homepage: https://github.com/gmpassos/reflection_factory
 
 environment:
   sdk: '>=3.9.0 <4.0.0'
 
 dependencies:
-  build: ^4.0.2
-  analyzer: ^8.4.0
-  dart_style: ^3.1.2
+  build: ^4.0.3
+  analyzer: ^8.4.1
+  dart_style: ^3.1.3
   meta: ^1.17.0
   mime: ^2.0.0
   base_codecs: ^1.0.1
@@ -21,12 +21,12 @@ dependencies:
   logging: ^1.3.0
 
 dev_dependencies:
-  build_runner: ^2.10.1
-  build_test: ^3.5.1
+  build_runner: ^2.10.4
+  build_test: ^3.5.4
   lints: ^6.0.0
   pubspec_parse: ^1.5.0
   data_serializer: ^1.2.1
   #dependency_validator: ^5.0.2
-  test: ^1.26.3
+  test: ^1.29.0
   coverage: ^1.15.0
   benchmark: ^0.3.0

test/reflection_factory_build_test.dart

@@ -1490,6 +1490,81 @@ void main() {
       },
     );
 
+    test('EnableReflection + advanced enum', () async {
+      var builder = ReflectionBuilder(verbose: true);
+
+      var sourceAssets = {
+        '$_pkgName|lib/status.dart': '''
+
+          import 'package:reflection_factory/reflection_factory.dart';
+
+          part 'status.reflection.g.dart';
+
+          @EnableReflection()
+          enum Status {
+            a(10),
+            b(20);
+
+            static final defaultStatus = Status.a;
+
+            final int id;
+
+            const Status(this.id);
+
+            bool get isFoo => true;
+
+            int calc(int m) => 10 * m;
+          }
+
+        ''',
+      };
+
+      final readerWriter = TestReaderWriter(rootPackage: _pkgName);
+      await readerWriter.testing.loadIsolateSources();
+
+      await testBuilder(
+        builder,
+        sourceAssets,
+        readerWriter: readerWriter,
+        generateFor: {'$_pkgName|lib/status.dart'},
+        outputs: {
+          '$_pkgName|lib/status.reflection.g.dart': decodedMatches(
+            allOf(
+              allOf(
+                contains('GENERATED CODE - DO NOT MODIFY BY HAND'),
+                contains(
+                  'BUILDER: reflection_factory/${ReflectionFactory.VERSION}',
+                ),
+                contains("part of 'status.dart'"),
+              ),
+              allOf([
+                contains('Status\$reflection extends EnumReflection<Status>'),
+                contains('Status\$reflectionExtension'),
+                isNot(contains('Foo\$reflection extends')),
+                isNot(contains('Foo\$reflectionExtension')),
+              ]),
+              allOf([
+                contains(
+                  'final Expando<Status\$reflection> _objectReflections',
+                ),
+                contains('factory Status\$reflection([Status? object]) {'),
+                contains(
+                  ' _valuesByName = const <String, Status>{\n'
+                  '    \'a\': Status.a,\n'
+                  '    \'b\': Status.b,\n'
+                  '  };\n',
+                ),
+                contains('_fieldsNames = const <String>[\'id\', \'isFoo\'];'),
+              ]),
+            ),
+          ),
+        },
+        onLog: (msg) {
+          _printToConsole(msg);
+        },
+      );
+    });
+
     test('ReflectionBridge', () async {
       var builder = ReflectionBuilder(verbose: true);