Migration: fix DecoratedType.toString to support named function parameters.

stereotype441 · commit-bot@chromium.org · commit a436c0621fdf · 2019-06-30T16:39:17.000Z
Change-Id: Ia49e4b381de5928d26be16fe4a9ab45fe8a64678 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/107515 Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
diff --git a/pkg/nnbd_migration/lib/src/decorated_type.dart b/pkg/nnbd_migration/lib/src/decorated_type.dart
@@ -237,9 +237,12 @@ class DecoratedType {
       if (type.typeFormals.isNotEmpty) {
         formals = '<${type.typeFormals.join(', ')}>';
       }
-      assert(type.namedParameterTypes.isEmpty &&
-          namedParameters.isEmpty); // TODO(paulberry)
-      var args = positionalParameters.map((p) => p.toString()).join(', ');
+      List<Object> argStrings =
+          positionalParameters.map((p) => p.toString()).toList();
+      for (var entry in namedParameters.entries) {
+        argStrings.add('${entry.key}: ${entry.value}');
+      }
+      var args = argStrings.join(', ');
       return '$returnType Function$formals($args)$trailing';
     } else if (type is DynamicTypeImpl) {
       return 'dynamic';
