From 8ecdb5fd5ac669c2c313269274189ab99bf2d2cf Mon Sep 17 00:00:00 2001 From: Robinson Christmann Date: Wed, 22 Nov 2023 14:52:27 +0100 Subject: [PATCH 1/4] Init fixing generator to allow for 'value' identifier in method params WIP --- generator/lib/src/generator.dart | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/generator/lib/src/generator.dart b/generator/lib/src/generator.dart index 2f1183e47..eb95902d5 100644 --- a/generator/lib/src/generator.dart +++ b/generator/lib/src/generator.dart @@ -59,7 +59,7 @@ class RetrofitGenerator extends GeneratorForAnnotation { static const _onSendProgress = 'onSendProgress'; static const _onReceiveProgress = 'onReceiveProgress'; static const _path = 'path'; - static const _valueVar = 'value'; + static const _valueVar = '_value'; bool hasCustomOptions = false; /// Global options specified in the `build.yaml` @@ -828,7 +828,7 @@ You should create a new class to encapsulate the response. ); } } else { - blocks.add(const Code('final value = $_resultVar.data!;')); + blocks.add(const Code('final $_valueVar = $_resultVar.data!;')); } } else { if (_isBasicType(returnType)) { @@ -856,7 +856,7 @@ You should create a new class to encapsulate the response. .assign(refer('await $_dioVar.fetch').call([options])) .statement, ) - ..add(const Code('final value = $_resultVar.data;')); + ..add(const Code('final $_valueVar = $_resultVar.data;')); } else if (_typeChecker(GeneratedMessage).isSuperTypeOf(returnType)) { blocks.add( declareFinal(_resultVar) @@ -865,7 +865,7 @@ You should create a new class to encapsulate the response. .statement, ); blocks.add(Code( - "final value = await compute(${_displayString(returnType)}.fromBuffer, $_resultVar.data!);")); + "final $_valueVar = await compute(${_displayString(returnType)}.fromBuffer, $_resultVar.data!);")); } else { final fetchType = returnType.isNullable ? 'Map?' @@ -954,7 +954,7 @@ You should create a new class to encapsulate the response. '''), ); } else { - blocks.add(Code('$returnAsyncWrapper value;')); + blocks.add(Code('$returnAsyncWrapper $_valueVar;')); } } From 493b3eb741be792f4f1e0c121667f53767688a77 Mon Sep 17 00:00:00 2001 From: Robinson Christmann Date: Fri, 1 Dec 2023 15:17:15 +0100 Subject: [PATCH 2/4] Add missing replacement of `'value'` to `_valueVar` in generator.dart --- generator/lib/src/generator.dart | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/generator/lib/src/generator.dart b/generator/lib/src/generator.dart index eb95902d5..e6481ed07 100644 --- a/generator/lib/src/generator.dart +++ b/generator/lib/src/generator.dart @@ -949,7 +949,7 @@ You should create a new class to encapsulate the response. if (isWrapped) { blocks.add( Code(''' - final httpResponse = HttpResponse(value, $_resultVar); + final httpResponse = HttpResponse($_valueVar, $_resultVar); $returnAsyncWrapper httpResponse; '''), ); From 4c175607af661004437e7b61e48ba84ab78a6fd3 Mon Sep 17 00:00:00 2001 From: Robinson Christmann Date: Mon, 1 Jul 2024 11:30:07 +0200 Subject: [PATCH 3/4] Fix tests and remove potentially wrong edit not covered by tests --- generator/lib/src/generator.dart | 12 +- generator/test/src/generator_test_src.dart | 212 ++++++++++----------- 2 files changed, 112 insertions(+), 112 deletions(-) diff --git a/generator/lib/src/generator.dart b/generator/lib/src/generator.dart index 131c468ad..e3b3a060e 100644 --- a/generator/lib/src/generator.dart +++ b/generator/lib/src/generator.dart @@ -403,7 +403,7 @@ class RetrofitGenerator extends GeneratorForAnnotation { final paths = _getAnnotations(m, retrofit.Path); var definePath = method.peek('path')?.stringValue; paths.forEach((k, v) { - final value = v.peek(_valueVar)?.stringValue ?? k.displayName; + final value = v.peek('value')?.stringValue ?? k.displayName; definePath = definePath?.replaceFirst( '{$value}', "\${${k.displayName}${k.type.element?.kind == ElementKind.ENUM ? _hasToJson(k.type) ? '.toJson()' : '.name' : ''}}", @@ -1377,7 +1377,7 @@ if (T != dynamic && ) { final queries = _getAnnotations(m, retrofit.Query); final queryParameters = queries.map((p, r) { - final key = r.peek(_valueVar)?.stringValue ?? p.displayName; + final key = r.peek('value')?.stringValue ?? p.displayName; final Expression value; if (_isBasicType(p.type) || p.type.isDartCoreList || @@ -1726,7 +1726,7 @@ ${bodyName.displayName} == null var anyNullable = false; final fields = _getAnnotations(m, retrofit.Field).map((p, r) { anyNullable |= p.type.nullabilitySuffix == NullabilitySuffix.question; - final fieldName = r.peek(_valueVar)?.stringValue ?? p.displayName; + final fieldName = r.peek('value')?.stringValue ?? p.displayName; final isFileField = _typeChecker(File).isAssignableFromType(p.type); if (isFileField) { log.severe( @@ -1768,7 +1768,7 @@ ${bodyName.displayName} == null parts.forEach((p, r) { final fieldName = r.peek('name')?.stringValue ?? - r.peek(_valueVar)?.stringValue ?? + r.peek('value')?.stringValue ?? p.displayName; final isFileField = _typeChecker(File).isAssignableFromType(p.type); final contentType = r.peek('contentType')?.stringValue; @@ -2018,7 +2018,7 @@ ${bodyName.displayName} == null Map _generateHeaders(MethodElement m) { final headers = _getMethodAnnotations(m, retrofit.Headers) - .map((e) => e.peek(_valueVar)) + .map((e) => e.peek('value')) .map( (value) => value?.mapValue.map( (k, v) { @@ -2047,7 +2047,7 @@ ${bodyName.displayName} == null final annotationsInParam = _getAnnotations(m, retrofit.Header); final headersInParams = annotationsInParam.map((k, v) { - final value = v.peek(_valueVar)?.stringValue ?? k.displayName; + final value = v.peek('value')?.stringValue ?? k.displayName; return MapEntry(value, refer(k.displayName)); }); headers.addAll(headersInParams); diff --git a/generator/test/src/generator_test_src.dart b/generator/test/src/generator_test_src.dart index 242ed27bf..75096f54e 100644 --- a/generator/test/src/generator_test_src.dart +++ b/generator/test/src/generator_test_src.dart @@ -346,8 +346,8 @@ abstract class UploadFileInfoPartTest { @ShouldGenerate( ''' - final value = User.fromJson(_result.data!); - return value; + final _value = User.fromJson(_result.data!); + return _value; ''', contains: true, ) @@ -359,8 +359,8 @@ abstract class GenericCast { @ShouldGenerate( ''' - final value = _result.data == null ? null : User.fromJson(_result.data!); - return value; + final _value = _result.data == null ? null : User.fromJson(_result.data!); + return _value; ''', contains: true, ) @@ -372,7 +372,7 @@ abstract class NullableGenericCast { @ShouldGenerate( ''' - yield value; + yield _value; ''', contains: true, ) @@ -386,13 +386,13 @@ enum TestEnum { A, B } @ShouldGenerate( ''' - final value = TestEnum.values.firstWhere( + final _value = TestEnum.values.firstWhere( (e) => e.name == _result.data, orElse: () => throw ArgumentError( 'TestEnum does not contain value \${_result.data}', ), ); - return value; + return _value; ''', contains: true, ) @@ -429,8 +429,8 @@ enum FromJsonEnum { @ShouldGenerate( ''' - final value = FromJsonEnum.fromJson(_result.data!); - return value; + final _value = FromJsonEnum.fromJson(_result.data!); + return _value; ''', contains: true, ) @@ -527,8 +527,8 @@ Map serializeUser(User object) => object.toJson(); @ShouldGenerate( ''' - final value = _result.data!; - return value; + final _value = _result.data!; + return _value; ''', contains: true, ) @@ -540,8 +540,8 @@ abstract class GenericCastBasicType { @ShouldGenerate( ''' - final value = _result.data; - return value; + final _value = _result.data; + return _value; ''', contains: true, ) @@ -671,12 +671,12 @@ abstract class TestCustomObjectBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => MapEntry( + var _value = _result.data!.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => User.fromJson(i as Map)) .toList())); - return value; + return _value; ''', contains: true, ) @@ -688,12 +688,12 @@ abstract class TestMapBody { @ShouldGenerate( ''' - var value = _result.data?.map((k, dynamic v) => MapEntry( + var _value = _result.data?.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => User.fromJson(i as Map)) .toList())); - return value; + return _value; ''', contains: true, ) @@ -705,9 +705,9 @@ abstract class NullableTestMapBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => + var _value = _result.data!.map((k, dynamic v) => MapEntry(k, User.fromJson(v as Map))); - return value; + return _value; ''', contains: true, ) @@ -719,9 +719,9 @@ abstract class TestMapBody2 { @ShouldGenerate( ''' - var value = _result.data?.map((k, dynamic v) => + var _value = _result.data?.map((k, dynamic v) => MapEntry(k, User.fromJson(v as Map))); - return value; + return _value; ''', contains: true, ) @@ -733,8 +733,8 @@ abstract class NullableTestMapBody2 { @ShouldGenerate( ''' - final value = _result.data!.cast(); - return value; + final _value = _result.data!.cast(); + return _value; ''', contains: true, ) @@ -746,8 +746,8 @@ abstract class TestBasicListString { @ShouldGenerate( ''' - final value = _result.data?.cast(); - return value; + final _value = _result.data?.cast(); + return _value; ''', contains: true, ) @@ -759,8 +759,8 @@ abstract class NullableTestBasicListString { @ShouldGenerate( ''' - final value = _result.data!.cast(); - return value; + final _value = _result.data!.cast(); + return _value; ''', contains: true, ) @@ -772,8 +772,8 @@ abstract class TestBasicListBool { @ShouldGenerate( ''' - final value = _result.data?.cast(); - return value; + final _value = _result.data?.cast(); + return _value; ''', contains: true, ) @@ -785,8 +785,8 @@ abstract class NullableTestBasicListBool { @ShouldGenerate( ''' - final value = _result.data!.cast(); - return value; + final _value = _result.data!.cast(); + return _value; ''', contains: true, ) @@ -798,8 +798,8 @@ abstract class TestBasicListInt { @ShouldGenerate( ''' - final value = _result.data?.cast(); - return value; + final _value = _result.data?.cast(); + return _value; ''', contains: true, ) @@ -811,8 +811,8 @@ abstract class NullableTestBasicListInt { @ShouldGenerate( ''' - final value = _result.data!.cast(); - return value; + final _value = _result.data!.cast(); + return _value; ''', contains: true, ) @@ -824,8 +824,8 @@ abstract class TestBasicListDouble { @ShouldGenerate( ''' - final value = _result.data?.cast(); - return value; + final _value = _result.data?.cast(); + return _value; ''', contains: true, ) @@ -912,7 +912,7 @@ abstract class TestHttpResponseVoid { @ShouldGenerate( ''' - final httpResponse = HttpResponse(value, _result); + final httpResponse = HttpResponse(_value, _result); ''', contains: true, ) @@ -924,7 +924,7 @@ abstract class TestHttpResponseObject { @ShouldGenerate( ''' - final httpResponse = HttpResponse(value, _result); + final httpResponse = HttpResponse(_value, _result); ''', contains: true, ) @@ -1118,8 +1118,8 @@ abstract class CustomOptions { @ShouldGenerate( ''' - final value = JsonMapper.fromMap(_result.data!)!; - return value; + final _value = JsonMapper.fromMap(_result.data!)!; + return _value; ''', contains: true, ) @@ -1134,9 +1134,9 @@ abstract class JsonMapperGenericCast { @ShouldGenerate( ''' - final value = + final _value = _result.data == null ? null : JsonMapper.fromMap(_result.data!)!; - return value; + return _value; ''', contains: true, ) @@ -1151,11 +1151,11 @@ abstract class NullableJsonMapperGenericCast { @ShouldGenerate( ''' - var value = _result.data! + var _value = _result.data! .map( (dynamic i) => JsonMapper.fromMap(i as Map)!) .toList(); - return value; + return _value; ''', contains: true, ) @@ -1170,12 +1170,12 @@ abstract class JsonMapperTestListBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => MapEntry( + var _value = _result.data!.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => JsonMapper.fromMap(i as Map)!) .toList())); - return value; + return _value; ''', contains: true, ) @@ -1190,9 +1190,9 @@ abstract class JsonMapperTestMapBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => + var _value = _result.data!.map((k, dynamic v) => MapEntry(k, JsonMapper.fromMap(v as Map)!)); - return value; + return _value; ''', contains: true, ) @@ -1207,8 +1207,8 @@ abstract class JsonMapperTestMapBody2 { @ShouldGenerate( ''' - final value = User.fromMap(_result.data!); - return value; + final _value = User.fromMap(_result.data!); + return _value; ''', contains: true, ) @@ -1223,8 +1223,8 @@ abstract class MapSerializableGenericCast { @ShouldGenerate( ''' - final value = _result.data == null ? null : User.fromMap(_result.data!); - return value; + final _value = _result.data == null ? null : User.fromMap(_result.data!); + return _value; ''', contains: true, ) @@ -1239,10 +1239,10 @@ abstract class NullableMapSerializableGenericCast { @ShouldGenerate( ''' - var value = _result.data! + var _value = _result.data! .map((dynamic i) => User.fromMap(i as Map)) .toList(); - return value; + return _value; ''', contains: true, ) @@ -1257,10 +1257,10 @@ abstract class MapSerializableTestListBody { @ShouldGenerate( ''' - var value = _result.data + var _value = _result.data ?.map((dynamic i) => User.fromMap(i as Map)) .toList(); - return value; + return _value; ''', contains: true, ) @@ -1275,12 +1275,12 @@ abstract class NullableMapSerializableTestListBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => MapEntry( + var _value = _result.data!.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => User.fromMap(i as Map)) .toList())); - return value; + return _value; ''', contains: true, ) @@ -1295,12 +1295,12 @@ abstract class MapSerializableTestMapBody { @ShouldGenerate( ''' - var value = _result.data?.map((k, dynamic v) => MapEntry( + var _value = _result.data?.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => User.fromMap(i as Map)) .toList())); - return value; + return _value; ''', contains: true, ) @@ -1315,9 +1315,9 @@ abstract class NullableMapSerializableTestMapBody { @ShouldGenerate( ''' - var value = _result.data!.map( + var _value = _result.data!.map( (k, dynamic v) => MapEntry(k, User.fromMap(v as Map))); - return value; + return _value; ''', contains: true, ) @@ -1332,9 +1332,9 @@ abstract class MapSerializableTestMapBody2 { @ShouldGenerate( ''' - var value = _result.data?.map( + var _value = _result.data?.map( (k, dynamic v) => MapEntry(k, User.fromMap(v as Map))); - return value; + return _value; ''', contains: true, ) @@ -1349,8 +1349,8 @@ abstract class NullableMapSerializableTestMapBody2 { @ShouldGenerate( ''' - final value = await compute(deserializeUser, _result.data!); - return value; + final _value = await compute(deserializeUser, _result.data!); + return _value; ''', contains: true, ) @@ -1365,10 +1365,10 @@ abstract class ComputeGenericCast { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : await compute(deserializeUser, _result.data!); - return value; + return _value; ''', contains: true, ) @@ -1383,11 +1383,11 @@ abstract class NullableComputeGenericCast { @ShouldGenerate( ''' - var value = await compute( + var _value = await compute( deserializeUserList, _result.data!.cast>(), ); - return value; + return _value; ''', contains: true, ) @@ -1402,13 +1402,13 @@ abstract class ComputeTestListBody { @ShouldGenerate( ''' - var value = _result.data == null + var _value = _result.data == null ? null : await compute( deserializeUserList, _result.data!.cast>(), ); - return value; + return _value; ''', contains: true, ) @@ -1423,12 +1423,12 @@ abstract class NullableComputeTestListBody { @ShouldGenerate( ''' - var value = Map.fromEntries(await Future.wait(_result.data!.entries.map( + var _value = Map.fromEntries(await Future.wait(_result.data!.entries.map( (e) async => MapEntry( e.key, await compute(deserializeUserList, (e.value as List).cast>()))))); - return value; + return _value; ''', contains: true, expectedLogItems: [ @@ -1449,12 +1449,12 @@ abstract class ComputeTestMapBody { @ShouldGenerate( ''' - var value = Map.fromEntries(await Future.wait(_result.data!.entries.map( + var _value = Map.fromEntries(await Future.wait(_result.data!.entries.map( (e) async => MapEntry( e.key, await compute(deserializeUserList, (e.value as List).cast>()))))); - return value; + return _value; ''', contains: true, expectedLogItems: [ @@ -1475,10 +1475,10 @@ abstract class NullableComputeTestMapBody { @ShouldGenerate( ''' - var value = Map.fromEntries(await Future.wait(_result.data!.entries.map( + var _value = Map.fromEntries(await Future.wait(_result.data!.entries.map( (e) async => MapEntry(e.key, await compute(deserializeUser, e.value as Map))))); - return value; + return _value; ''', contains: true, expectedLogItems: [ @@ -1499,14 +1499,14 @@ abstract class ComputeTestMapBody2 { @ShouldGenerate( ''' - var value = _result.data == null + var _value = _result.data == null ? null : Map.fromEntries(await Future.wait(_result.data!.entries.map( (e) async => MapEntry( e.key, await compute( deserializeUser, e.value as Map))))); - return value; + return _value; ''', contains: true, expectedLogItems: [ @@ -1657,11 +1657,11 @@ abstract class ListBodyShouldNotBeCleanTest { @ShouldGenerate( ''' - final value = GenericUser.fromJson( + final _value = GenericUser.fromJson( _result.data!, (json) => json as dynamic, ); - return value; + return _value; ''', contains: true, ) @@ -1673,7 +1673,7 @@ abstract class DynamicInnerGenericTypeShouldBeCastedAsDynamic { @ShouldGenerate( ''' - final value = GenericUser>.fromJson( + final _value = GenericUser>.fromJson( _result.data!, (json) => json is List ? json @@ -1681,7 +1681,7 @@ abstract class DynamicInnerGenericTypeShouldBeCastedAsDynamic { .toList() : List.empty(), ); - return value; + return _value; ''', contains: true, ) @@ -1693,7 +1693,7 @@ abstract class DynamicInnerListGenericTypeShouldBeCastedRecursively { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUser>.fromJson( _result.data!, @@ -1703,7 +1703,7 @@ abstract class DynamicInnerListGenericTypeShouldBeCastedRecursively { .toList() : List.empty(), ); - return value; + return _value; ''', contains: true, ) @@ -1715,11 +1715,11 @@ abstract class NullableDynamicInnerListGenericTypeShouldBeCastedRecursively { @ShouldGenerate( ''' - final value = GenericUser.fromJson( + final _value = GenericUser.fromJson( _result.data!, (json) => User.fromJson(json as Map), ); - return value; + return _value; ''', contains: true, ) @@ -1731,14 +1731,14 @@ abstract class DynamicInnerGenericTypeShouldBeCastedAsMap { @ShouldGenerate( ''' - final value = GenericUser>.fromJson( + final _value = GenericUser>.fromJson( _result.data!, (json) => GenericUser.fromJson( json as Map, (json) => User.fromJson(json as Map), ), ); - return value; + return _value; ''', contains: true, ) @@ -1750,13 +1750,13 @@ abstract class NestGenericTypeShouldBeCastedRecursively { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUser.fromJson( _result.data!, (json) => User.fromJson(json as Map), ); - return value; + return _value; ''', contains: true, ) @@ -1768,12 +1768,12 @@ abstract class NullableDynamicInnerGenericTypeShouldBeCastedAsMap { @ShouldGenerate( ''' - final value = GenericUser.fromJson( + final _value = GenericUser.fromJson( _result.data!, (json) => json == null ? null : User.fromJson(json as Map), ); - return value; + return _value; ''', contains: true, ) @@ -1785,7 +1785,7 @@ abstract class DynamicNullableInnerGenericTypeShouldBeCastedAsMap { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUser.fromJson( _result.data!, @@ -1793,7 +1793,7 @@ abstract class DynamicNullableInnerGenericTypeShouldBeCastedAsMap { ? null : User.fromJson(json as Map), ); - return value; + return _value; ''', contains: true, ) @@ -1805,13 +1805,13 @@ abstract class NullableDynamicNullableInnerGenericTypeShouldBeCastedAsMap { @ShouldGenerate( ''' - final value = GenericUser>.fromJson( + final _value = GenericUser>.fromJson( _result.data!, (json) => json is List ? json.map((i) => i as double).toList() : List.empty(), ); - return value; + return _value; ''', contains: true, ) @@ -1823,7 +1823,7 @@ abstract class DynamicInnerListGenericPrimitiveTypeShouldBeCastedRecursively { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUser>.fromJson( _result.data!, @@ -1831,7 +1831,7 @@ abstract class DynamicInnerListGenericPrimitiveTypeShouldBeCastedRecursively { ? json.map((i) => i as double).toList() : List.empty(), ); - return value; + return _value; ''', contains: true, ) @@ -1843,9 +1843,9 @@ abstract class NullableDynamicInnerListGenericPrimitiveTypeShouldBeCastedRecursi @ShouldGenerate( ''' - final value = GenericUserWithoutGenericArgumentFactories.fromJson( + final _value = GenericUserWithoutGenericArgumentFactories.fromJson( _result.data!); - return value; + return _value; ''', contains: true, ) @@ -1857,11 +1857,11 @@ abstract class DynamicInnerGenericTypeShouldBeWithoutGenericArgumentType { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUserWithoutGenericArgumentFactories.fromJson( _result.data!); - return value; + return _value; ''', contains: true, ) @@ -2091,4 +2091,4 @@ abstract class MultipartWithMultplePathParamsPUT { @Path("commentId") String commentId, @Part() Map body, ); -} \ No newline at end of file +} From 38b3f6b0b212160eaa43635f4e4d1a5af088302d Mon Sep 17 00:00:00 2001 From: Robinson Christmann Date: Mon, 1 Jul 2024 11:44:12 +0200 Subject: [PATCH 4/4] some fixes got overriden during merge of master --- generator/test/src/generator_test_src.dart | 216 ++++++++++----------- 1 file changed, 108 insertions(+), 108 deletions(-) diff --git a/generator/test/src/generator_test_src.dart b/generator/test/src/generator_test_src.dart index 5c81e47ab..dc444b54a 100644 --- a/generator/test/src/generator_test_src.dart +++ b/generator/test/src/generator_test_src.dart @@ -125,8 +125,8 @@ abstract class TestExtrasWithMap { @GET('/list/') @Extra({'key': 'value'}) Future list( - @Extras() Map extras, - ); + @Extras() Map extras, + ); } @ShouldGenerate( @@ -140,7 +140,7 @@ abstract class TestExtrasWithMap { abstract class TestExtrasWithObject { @GET('/list/') Future list( - @Extras() User u, + @Extras() User u, ); } @@ -390,8 +390,8 @@ abstract class UploadFileInfoPartTest { @ShouldGenerate( ''' - final value = User.fromJson(_result.data!); - return value; + final _value = User.fromJson(_result.data!); + return _value; ''', contains: true, ) @@ -403,8 +403,8 @@ abstract class GenericCast { @ShouldGenerate( ''' - final value = _result.data == null ? null : User.fromJson(_result.data!); - return value; + final _value = _result.data == null ? null : User.fromJson(_result.data!); + return _value; ''', contains: true, ) @@ -416,7 +416,7 @@ abstract class NullableGenericCast { @ShouldGenerate( ''' - yield value; + yield _value; ''', contains: true, ) @@ -430,13 +430,13 @@ enum TestEnum { A, B } @ShouldGenerate( ''' - final value = TestEnum.values.firstWhere( + final _value = TestEnum.values.firstWhere( (e) => e.name == _result.data, orElse: () => throw ArgumentError( 'TestEnum does not contain value \${_result.data}', ), ); - return value; + return _value; ''', contains: true, ) @@ -473,8 +473,8 @@ enum FromJsonEnum { @ShouldGenerate( ''' - final value = FromJsonEnum.fromJson(_result.data!); - return value; + final _value = FromJsonEnum.fromJson(_result.data!); + return _value; ''', contains: true, ) @@ -571,8 +571,8 @@ Map serializeUser(User object) => object.toJson(); @ShouldGenerate( ''' - final value = _result.data!; - return value; + final _value = _result.data!; + return _value; ''', contains: true, ) @@ -584,8 +584,8 @@ abstract class GenericCastBasicType { @ShouldGenerate( ''' - final value = _result.data; - return value; + final _value = _result.data; + return _value; ''', contains: true, ) @@ -715,12 +715,12 @@ abstract class TestCustomObjectBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => MapEntry( + var _value = _result.data!.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => User.fromJson(i as Map)) .toList())); - return value; + return _value; ''', contains: true, ) @@ -732,12 +732,12 @@ abstract class TestMapBody { @ShouldGenerate( ''' - var value = _result.data?.map((k, dynamic v) => MapEntry( + var _value = _result.data?.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => User.fromJson(i as Map)) .toList())); - return value; + return _value; ''', contains: true, ) @@ -749,9 +749,9 @@ abstract class NullableTestMapBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => + var _value = _result.data!.map((k, dynamic v) => MapEntry(k, User.fromJson(v as Map))); - return value; + return _value; ''', contains: true, ) @@ -763,9 +763,9 @@ abstract class TestMapBody2 { @ShouldGenerate( ''' - var value = _result.data?.map((k, dynamic v) => + var _value = _result.data?.map((k, dynamic v) => MapEntry(k, User.fromJson(v as Map))); - return value; + return _value; ''', contains: true, ) @@ -777,8 +777,8 @@ abstract class NullableTestMapBody2 { @ShouldGenerate( ''' - final value = _result.data!.cast(); - return value; + final _value = _result.data!.cast(); + return _value; ''', contains: true, ) @@ -790,8 +790,8 @@ abstract class TestBasicListString { @ShouldGenerate( ''' - final value = _result.data?.cast(); - return value; + final _value = _result.data?.cast(); + return _value; ''', contains: true, ) @@ -803,8 +803,8 @@ abstract class NullableTestBasicListString { @ShouldGenerate( ''' - final value = _result.data!.cast(); - return value; + final _value = _result.data!.cast(); + return _value; ''', contains: true, ) @@ -816,8 +816,8 @@ abstract class TestBasicListBool { @ShouldGenerate( ''' - final value = _result.data?.cast(); - return value; + final _value = _result.data?.cast(); + return _value; ''', contains: true, ) @@ -829,8 +829,8 @@ abstract class NullableTestBasicListBool { @ShouldGenerate( ''' - final value = _result.data!.cast(); - return value; + final _value = _result.data!.cast(); + return _value; ''', contains: true, ) @@ -842,8 +842,8 @@ abstract class TestBasicListInt { @ShouldGenerate( ''' - final value = _result.data?.cast(); - return value; + final _value = _result.data?.cast(); + return _value; ''', contains: true, ) @@ -855,8 +855,8 @@ abstract class NullableTestBasicListInt { @ShouldGenerate( ''' - final value = _result.data!.cast(); - return value; + final _value = _result.data!.cast(); + return _value; ''', contains: true, ) @@ -868,8 +868,8 @@ abstract class TestBasicListDouble { @ShouldGenerate( ''' - final value = _result.data?.cast(); - return value; + final _value = _result.data?.cast(); + return _value; ''', contains: true, ) @@ -956,7 +956,7 @@ abstract class TestHttpResponseVoid { @ShouldGenerate( ''' - final httpResponse = HttpResponse(value, _result); + final httpResponse = HttpResponse(_value, _result); ''', contains: true, ) @@ -968,7 +968,7 @@ abstract class TestHttpResponseObject { @ShouldGenerate( ''' - final httpResponse = HttpResponse(value, _result); + final httpResponse = HttpResponse(_value, _result); ''', contains: true, ) @@ -1162,8 +1162,8 @@ abstract class CustomOptions { @ShouldGenerate( ''' - final value = JsonMapper.fromMap(_result.data!)!; - return value; + final _value = JsonMapper.fromMap(_result.data!)!; + return _value; ''', contains: true, ) @@ -1178,9 +1178,9 @@ abstract class JsonMapperGenericCast { @ShouldGenerate( ''' - final value = + final _value = _result.data == null ? null : JsonMapper.fromMap(_result.data!)!; - return value; + return _value; ''', contains: true, ) @@ -1195,11 +1195,11 @@ abstract class NullableJsonMapperGenericCast { @ShouldGenerate( ''' - var value = _result.data! + var _value = _result.data! .map( (dynamic i) => JsonMapper.fromMap(i as Map)!) .toList(); - return value; + return _value; ''', contains: true, ) @@ -1214,12 +1214,12 @@ abstract class JsonMapperTestListBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => MapEntry( + var _value = _result.data!.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => JsonMapper.fromMap(i as Map)!) .toList())); - return value; + return _value; ''', contains: true, ) @@ -1234,9 +1234,9 @@ abstract class JsonMapperTestMapBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => + var _value = _result.data!.map((k, dynamic v) => MapEntry(k, JsonMapper.fromMap(v as Map)!)); - return value; + return _value; ''', contains: true, ) @@ -1251,8 +1251,8 @@ abstract class JsonMapperTestMapBody2 { @ShouldGenerate( ''' - final value = User.fromMap(_result.data!); - return value; + final _value = User.fromMap(_result.data!); + return _value; ''', contains: true, ) @@ -1267,8 +1267,8 @@ abstract class MapSerializableGenericCast { @ShouldGenerate( ''' - final value = _result.data == null ? null : User.fromMap(_result.data!); - return value; + final _value = _result.data == null ? null : User.fromMap(_result.data!); + return _value; ''', contains: true, ) @@ -1283,10 +1283,10 @@ abstract class NullableMapSerializableGenericCast { @ShouldGenerate( ''' - var value = _result.data! + var _value = _result.data! .map((dynamic i) => User.fromMap(i as Map)) .toList(); - return value; + return _value; ''', contains: true, ) @@ -1301,10 +1301,10 @@ abstract class MapSerializableTestListBody { @ShouldGenerate( ''' - var value = _result.data + var _value = _result.data ?.map((dynamic i) => User.fromMap(i as Map)) .toList(); - return value; + return _value; ''', contains: true, ) @@ -1319,12 +1319,12 @@ abstract class NullableMapSerializableTestListBody { @ShouldGenerate( ''' - var value = _result.data!.map((k, dynamic v) => MapEntry( + var _value = _result.data!.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => User.fromMap(i as Map)) .toList())); - return value; + return _value; ''', contains: true, ) @@ -1339,12 +1339,12 @@ abstract class MapSerializableTestMapBody { @ShouldGenerate( ''' - var value = _result.data?.map((k, dynamic v) => MapEntry( + var _value = _result.data?.map((k, dynamic v) => MapEntry( k, (v as List) .map((i) => User.fromMap(i as Map)) .toList())); - return value; + return _value; ''', contains: true, ) @@ -1359,9 +1359,9 @@ abstract class NullableMapSerializableTestMapBody { @ShouldGenerate( ''' - var value = _result.data!.map( + var _value = _result.data!.map( (k, dynamic v) => MapEntry(k, User.fromMap(v as Map))); - return value; + return _value; ''', contains: true, ) @@ -1376,9 +1376,9 @@ abstract class MapSerializableTestMapBody2 { @ShouldGenerate( ''' - var value = _result.data?.map( + var _value = _result.data?.map( (k, dynamic v) => MapEntry(k, User.fromMap(v as Map))); - return value; + return _value; ''', contains: true, ) @@ -1393,8 +1393,8 @@ abstract class NullableMapSerializableTestMapBody2 { @ShouldGenerate( ''' - final value = await compute(deserializeUser, _result.data!); - return value; + final _value = await compute(deserializeUser, _result.data!); + return _value; ''', contains: true, ) @@ -1409,10 +1409,10 @@ abstract class ComputeGenericCast { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : await compute(deserializeUser, _result.data!); - return value; + return _value; ''', contains: true, ) @@ -1427,11 +1427,11 @@ abstract class NullableComputeGenericCast { @ShouldGenerate( ''' - var value = await compute( + var _value = await compute( deserializeUserList, _result.data!.cast>(), ); - return value; + return _value; ''', contains: true, ) @@ -1446,13 +1446,13 @@ abstract class ComputeTestListBody { @ShouldGenerate( ''' - var value = _result.data == null + var _value = _result.data == null ? null : await compute( deserializeUserList, _result.data!.cast>(), ); - return value; + return _value; ''', contains: true, ) @@ -1467,12 +1467,12 @@ abstract class NullableComputeTestListBody { @ShouldGenerate( ''' - var value = Map.fromEntries(await Future.wait(_result.data!.entries.map( + var _value = Map.fromEntries(await Future.wait(_result.data!.entries.map( (e) async => MapEntry( e.key, await compute(deserializeUserList, (e.value as List).cast>()))))); - return value; + return _value; ''', contains: true, expectedLogItems: [ @@ -1493,12 +1493,12 @@ abstract class ComputeTestMapBody { @ShouldGenerate( ''' - var value = Map.fromEntries(await Future.wait(_result.data!.entries.map( + var _value = Map.fromEntries(await Future.wait(_result.data!.entries.map( (e) async => MapEntry( e.key, await compute(deserializeUserList, (e.value as List).cast>()))))); - return value; + return _value; ''', contains: true, expectedLogItems: [ @@ -1519,10 +1519,10 @@ abstract class NullableComputeTestMapBody { @ShouldGenerate( ''' - var value = Map.fromEntries(await Future.wait(_result.data!.entries.map( + var _value = Map.fromEntries(await Future.wait(_result.data!.entries.map( (e) async => MapEntry(e.key, await compute(deserializeUser, e.value as Map))))); - return value; + return _value; ''', contains: true, expectedLogItems: [ @@ -1543,14 +1543,14 @@ abstract class ComputeTestMapBody2 { @ShouldGenerate( ''' - var value = _result.data == null + var _value = _result.data == null ? null : Map.fromEntries(await Future.wait(_result.data!.entries.map( (e) async => MapEntry( e.key, await compute( deserializeUser, e.value as Map))))); - return value; + return _value; ''', contains: true, expectedLogItems: [ @@ -1701,11 +1701,11 @@ abstract class ListBodyShouldNotBeCleanTest { @ShouldGenerate( ''' - final value = GenericUser.fromJson( + final _value = GenericUser.fromJson( _result.data!, (json) => json as dynamic, ); - return value; + return _value; ''', contains: true, ) @@ -1717,7 +1717,7 @@ abstract class DynamicInnerGenericTypeShouldBeCastedAsDynamic { @ShouldGenerate( ''' - final value = GenericUser>.fromJson( + final _value = GenericUser>.fromJson( _result.data!, (json) => json is List ? json @@ -1725,7 +1725,7 @@ abstract class DynamicInnerGenericTypeShouldBeCastedAsDynamic { .toList() : List.empty(), ); - return value; + return _value; ''', contains: true, ) @@ -1737,7 +1737,7 @@ abstract class DynamicInnerListGenericTypeShouldBeCastedRecursively { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUser>.fromJson( _result.data!, @@ -1747,7 +1747,7 @@ abstract class DynamicInnerListGenericTypeShouldBeCastedRecursively { .toList() : List.empty(), ); - return value; + return _value; ''', contains: true, ) @@ -1759,11 +1759,11 @@ abstract class NullableDynamicInnerListGenericTypeShouldBeCastedRecursively { @ShouldGenerate( ''' - final value = GenericUser.fromJson( + final _value = GenericUser.fromJson( _result.data!, (json) => User.fromJson(json as Map), ); - return value; + return _value; ''', contains: true, ) @@ -1775,14 +1775,14 @@ abstract class DynamicInnerGenericTypeShouldBeCastedAsMap { @ShouldGenerate( ''' - final value = GenericUser>.fromJson( + final _value = GenericUser>.fromJson( _result.data!, (json) => GenericUser.fromJson( json as Map, (json) => User.fromJson(json as Map), ), ); - return value; + return _value; ''', contains: true, ) @@ -1794,13 +1794,13 @@ abstract class NestGenericTypeShouldBeCastedRecursively { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUser.fromJson( _result.data!, (json) => User.fromJson(json as Map), ); - return value; + return _value; ''', contains: true, ) @@ -1812,12 +1812,12 @@ abstract class NullableDynamicInnerGenericTypeShouldBeCastedAsMap { @ShouldGenerate( ''' - final value = GenericUser.fromJson( + final _value = GenericUser.fromJson( _result.data!, (json) => json == null ? null : User.fromJson(json as Map), ); - return value; + return _value; ''', contains: true, ) @@ -1829,7 +1829,7 @@ abstract class DynamicNullableInnerGenericTypeShouldBeCastedAsMap { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUser.fromJson( _result.data!, @@ -1837,7 +1837,7 @@ abstract class DynamicNullableInnerGenericTypeShouldBeCastedAsMap { ? null : User.fromJson(json as Map), ); - return value; + return _value; ''', contains: true, ) @@ -1849,13 +1849,13 @@ abstract class NullableDynamicNullableInnerGenericTypeShouldBeCastedAsMap { @ShouldGenerate( ''' - final value = GenericUser>.fromJson( + final _value = GenericUser>.fromJson( _result.data!, (json) => json is List ? json.map((i) => i as double).toList() : List.empty(), ); - return value; + return _value; ''', contains: true, ) @@ -1867,7 +1867,7 @@ abstract class DynamicInnerListGenericPrimitiveTypeShouldBeCastedRecursively { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUser>.fromJson( _result.data!, @@ -1875,7 +1875,7 @@ abstract class DynamicInnerListGenericPrimitiveTypeShouldBeCastedRecursively { ? json.map((i) => i as double).toList() : List.empty(), ); - return value; + return _value; ''', contains: true, ) @@ -1887,9 +1887,9 @@ abstract class NullableDynamicInnerListGenericPrimitiveTypeShouldBeCastedRecursi @ShouldGenerate( ''' - final value = GenericUserWithoutGenericArgumentFactories.fromJson( + final _value = GenericUserWithoutGenericArgumentFactories.fromJson( _result.data!); - return value; + return _value; ''', contains: true, ) @@ -1901,11 +1901,11 @@ abstract class DynamicInnerGenericTypeShouldBeWithoutGenericArgumentType { @ShouldGenerate( ''' - final value = _result.data == null + final _value = _result.data == null ? null : GenericUserWithoutGenericArgumentFactories.fromJson( _result.data!); - return value; + return _value; ''', contains: true, )