fix(sqlite): enumAsString generates varchar column instead of integer #597 (#604)

Author: tshedor

packages/brick_build/pubspec.yaml

@@ -12,7 +12,7 @@ environment:
 dependencies:
   analyzer: ">=6.11.0 <8.0.0"
   brick_core: ">=2.0.0 <3.0.0"
-  build: ">=2.4.2 <3.0.0"
+  build: ">=2.4.2 <2.5.0"
   dart_style: ">=3.0.0 <4.0.0"
   glob: ">=2.1.0 <3.0.0"
   logging: ">=1.0.0 <2.0.0"

packages/brick_build_test/pubspec.yaml

@@ -12,7 +12,7 @@ environment:
   sdk: ">=3.4.0 <4.0.0"
 
 dependencies:
-  build: ">=2.4.2 <3.0.0"
+  build: ">=2.4.2 <2.5.0"
   path: ">=1.9.0 <2.0.0"
   source_gen: ">=1.5.0 <3.0.0"
   source_gen_test: ">=1.0.0 <2.0.0"

packages/brick_graphql_generators/pubspec.yaml

@@ -15,7 +15,7 @@ dependencies:
   brick_core: ">=2.0.0 <3.0.0"
   brick_graphql: ">=4.0.0 <5.0.0"
   brick_json_generators: ">=4.0.0 <5.0.0"
-  build: ">=2.4.2 <3.0.0"
+  build: ">=2.4.2 <2.5.0"
   dart_style: ">=3.0.0 <4.0.0"
   source_gen: ">=1.5.0 <3.0.0"
 

packages/brick_graphql_generators/test/graphql_model_serdes_generator/test_enum_as_string.dart

@@ -8,22 +8,19 @@ Future<EnumAsString> _$EnumAsStringFromGraphql(
 }) async {
   return EnumAsString(
     hat: Hat.values.byName(data['hat']),
-    nullableHat:
-        data['nullableHat'] == null
-            ? null
-            : Hat.values.byName(data['nullableHat']),
-    hats:
-        data['hats']
-            .whereType<String>()
-            .map(Hat.values.byName)
-            .toList()
-            .cast<Hat>(),
-    nullableHats:
-        data['nullableHats']
-            .whereType<String>()
-            .map(Hat.values.byName)
-            ?.toList()
-            .cast<Hat?>(),
+    nullableHat: data['nullableHat'] == null
+        ? null
+        : Hat.values.byName(data['nullableHat']),
+    hats: data['hats']
+        .whereType<String>()
+        .map(Hat.values.byName)
+        .toList()
+        .cast<Hat>(),
+    nullableHats: data['nullableHats']
+        .whereType<String>()
+        .map(Hat.values.byName)
+        ?.toList()
+        .cast<Hat?>(),
   );
 }
 

packages/brick_graphql_generators/test/graphql_model_serdes_generator/test_from_json_to_json.dart

@@ -11,22 +11,19 @@ Future<ToFromJson> _$ToFromJsonFromGraphql(
 }) async {
   return ToFromJson(
     assoc: ToFromJsonAssoc.fromJson(data['assoc'] as String),
-    assocNullable:
-        data['assocNullable'] == null
-            ? null
-            : ToFromJsonAssoc.fromJson(data['assocNullable'] as String),
-    assocIterable:
-        data['assocIterable']
-            .map((d) => ToFromJsonAssoc.fromJson(d as String))
-            .toList()
-            .cast<ToFromJsonAssoc>(),
-    assocIterableNullable:
-        data['assocIterableNullable'] == null
-            ? null
-            : data['assocIterableNullable']
-                ?.map((d) => ToFromJsonAssoc.fromJson(d as String))
-                .toList()
-                .cast<ToFromJsonAssoc>(),
+    assocNullable: data['assocNullable'] == null
+        ? null
+        : ToFromJsonAssoc.fromJson(data['assocNullable'] as String),
+    assocIterable: data['assocIterable']
+        .map((d) => ToFromJsonAssoc.fromJson(d as String))
+        .toList()
+        .cast<ToFromJsonAssoc>(),
+    assocIterableNullable: data['assocIterableNullable'] == null
+        ? null
+        : data['assocIterableNullable']
+              ?.map((d) => ToFromJsonAssoc.fromJson(d as String))
+              .toList()
+              .cast<ToFromJsonAssoc>(),
   );
 }
 
@@ -39,8 +36,9 @@ Future<Map<String, dynamic>> _$ToFromJsonToGraphql(
     'assoc': instance.assoc.toJson(),
     'assocNullable': instance.assocNullable?.toJson(),
     'assocIterable': instance.assocIterable.map((s) => s.toJson()).toList(),
-    'assocIterableNullable':
-        instance.assocIterableNullable?.map((s) => s.toJson()).toList(),
+    'assocIterableNullable': instance.assocIterableNullable
+        ?.map((s) => s.toJson())
+        .toList(),
   };
 }
 

packages/brick_graphql_generators/test/graphql_model_serdes_generator_test.dart

@@ -88,6 +88,12 @@ class TestGenerator extends AnnotationSuperGenerator<GraphqlSerializable> {
 Future<void> generateExpectation(String filename, String output, {TestGenerator? generator}) async {
   final reader = await generateReader(filename);
   final generated = await (generator ?? _generator).generate(reader, MockBuildStep());
+
+  if (generated.trim() != output.trim()) {
+    // ignore: avoid_print
+    print(generated);
+  }
+
   expect(generated.trim(), output.trim());
 }
 
@@ -98,5 +104,11 @@ Future<void> generateAdapterExpectation(String filename, String output) async {
     annotation.annotation,
     null,
   );
+
+  if (generated.trim() != output.trim()) {
+    // ignore: avoid_print
+    print(generated);
+  }
+
   expect(generated.trim(), output.trim());
 }

packages/brick_json_generators/pubspec.yaml

@@ -13,7 +13,7 @@ dependencies:
   analyzer: ">=6.11.0 <8.0.0"
   brick_build: ">=4.0.0 <5.0.0"
   brick_core: ">=2.0.0 <3.0.0"
-  build: ">=2.4.2 <3.0.0"
+  build: ">=2.4.2 <2.5.0"
   dart_style: ">=3.0.0 <4.0.0"
   source_gen: ">=1.5.0 <3.0.0"
 

packages/brick_offline_first_build/pubspec.yaml

@@ -17,7 +17,7 @@ dependencies:
   brick_offline_first: ">=4.0.0 <5.0.0"
   brick_sqlite: ">=4.0.0 <5.0.0"
   brick_sqlite_generators: ">=4.0.0 <5.0.0"
-  build: ">=2.4.2 <3.0.0"
+  build: ">=2.4.2 <2.5.0"
   dart_style: ">=3.0.0 <4.0.0"
   logging: ">=1.0.0 <2.0.0"
   meta: ">=1.3.0 <2.0.0"

packages/brick_offline_first_build/test/__helpers__.dart

@@ -20,6 +20,12 @@ Future<void> generateExpectation(
 }) async {
   final reader = await generateReader(filename);
   final generated = await (generator ?? _generator).generate(reader, MockBuildStep());
+
+  if (generated.trim() != output.trim()) {
+    // ignore: avoid_print
+    print(generated);
+  }
+
   expect(generated.trim(), output.trim());
 }
 
@@ -34,6 +40,12 @@ Future<void> generateAdapterExpectation(
     annotation.annotation,
     MockBuildStep(),
   );
+
+  if (generated.trim() != output.trim()) {
+    // ignore: avoid_print
+    print(generated);
+  }
+
   expect(generated.trim(), output.trim());
 }
 

packages/brick_offline_first_build/test/offline_first_generator/test_custom_offline_first_serdes.dart

@@ -8,17 +8,17 @@ Future<CustomOfflineFirstSerdes> _$CustomOfflineFirstSerdesFromTest(
   OfflineFirstRepository? repository,
 }) async {
   return CustomOfflineFirstSerdes(
-    string:
-        data['string'] == null ? null : Serializable.fromTest(data['string']),
+    string: data['string'] == null
+        ? null
+        : Serializable.fromTest(data['string']),
     constructorFieldNullabilityMismatch:
         data['constructor_field_nullability_mismatch'] as bool?,
-    strings:
-        data['strings'] == null
-            ? null
-            : data['strings']
-                ?.map((c) => Serializable.fromTest(c as Map<String, dynamic>))
-                .toList()
-                .cast<Serializable>(),
+    strings: data['strings'] == null
+        ? null
+        : data['strings']
+              ?.map((c) => Serializable.fromTest(c as Map<String, dynamic>))
+              .toList()
+              .cast<Serializable>(),
   );
 }
 
@@ -41,19 +41,17 @@ Future<CustomOfflineFirstSerdes> _$CustomOfflineFirstSerdesFromSqlite(
   OfflineFirstRepository? repository,
 }) async {
   return CustomOfflineFirstSerdes(
-    string:
-        data['string'] == null
-            ? null
-            : Serializable.fromSqlite(data['string'] as int),
+    string: data['string'] == null
+        ? null
+        : Serializable.fromSqlite(data['string'] as int),
     constructorFieldNullabilityMismatch:
         data['constructor_field_nullability_mismatch'] == 1,
-    strings:
-        data['strings'] == null
-            ? null
-            : jsonDecode(data['strings'])
-                .map((c) => Serializable.fromSqlite(c as int))
-                .toList()
-                .cast<Serializable>(),
+    strings: data['strings'] == null
+        ? null
+        : jsonDecode(data['strings'])
+              .map((c) => Serializable.fromSqlite(c as int))
+              .toList()
+              .cast<Serializable>(),
   )..primaryKey = data['_brick_id'] as int;
 }