Version 2.11.0-166.0.dev

Merge commit '280f639fdaa87575d5e13d3f020c3460da01c414' into 'dev'

Author: Dart CI

CHANGELOG.md

@@ -6,6 +6,8 @@
 
 ### Dart2JS
 
+* Removed `--no-defer-class-types` and `--no-new-deferred-split'.
+
 ### Tools
 
 #### Dartanalyzer

DEPS

@@ -144,7 +144,7 @@ vars = {
   "source_maps_rev": "53eb92ccfe6e64924054f83038a534b959b12b3e",
   "source_span_rev": "cc7c4288a83f71ecef3414199947b52a8c112c65",
   "sse_tag": "e5cf68975e8e87171a3dc297577aa073454a91dc",
-  "stack_trace_tag": "d3813ca0a77348e0faf0d6af0cc17913e36afa39",
+  "stack_trace_tag": "a958966148516dfa64e2b54c14492175da5cc8e1",
   "stagehand_tag": "v3.3.9",
   "stream_channel_tag": "c446774fd077c9bdbd6235a7aadc661ef60a9727",
   "string_scanner_rev": "1b63e6e5db5933d7be0a45da6e1129fe00262734",

pkg/_fe_analyzer_shared/pubspec.yaml

@@ -1,5 +1,5 @@
 name: _fe_analyzer_shared
-version: 9.0.0
+version: 10.0.0
 description: Logic that is shared between the front_end and analyzer packages.
 homepage: https://github.com/dart-lang/sdk/tree/master/pkg/_fe_analyzer_shared
 

pkg/analysis_server/lib/src/lsp/server_capabilities_computer.dart

@@ -224,6 +224,8 @@ class ServerCapabilitiesComputer {
         language: 'yaml', scheme: 'file', pattern: '**/pubspec.yaml');
     final analysisOptionsFile = DocumentFilter(
         language: 'yaml', scheme: 'file', pattern: '**/analysis_options.yaml');
+    final fixDataFile = DocumentFilter(
+        language: 'yaml', scheme: 'file', pattern: '**/lib/fix_data.yaml');
 
     final pluginTypes = _server.pluginManager.plugins
         .expand((plugin) => plugin.currentSession?.interestingFiles ?? const [])
@@ -243,6 +245,7 @@ class ServerCapabilitiesComputer {
       ...allTypes,
       pubspecFile,
       analysisOptionsFile,
+      fixDataFile,
     }.toList();
 
     final registrations = <Registration>[];

pkg/analysis_server/lib/src/services/correction/bulk_fix_processor.dart

@@ -140,6 +140,11 @@ class BulkFixProcessor {
     LintNames.prefer_conditional_assignment: [
       ReplaceWithConditionalAssignment.newInstance,
     ],
+    // TODO (pq): can produce results incompatible w/ `unnecessary_const`
+    // LintNames.prefer_const_constructors: [
+    //   AddConst.newInstance,
+    //   ReplaceNewWithConst.newInstance,
+    // ],
     LintNames.prefer_const_constructors_in_immutables: [
       AddConst.newInstance,
     ],

pkg/analysis_server/lib/src/services/correction/fix/data_driven/transform_set_error_code.dart

@@ -15,42 +15,42 @@ class TransformSetErrorCode extends ErrorCode {
    * 0: the actual type of the value
    */
   static const TransformSetErrorCode invalidValue = TransformSetErrorCode(
-      'invalidValue',
+      'invalid_value',
       "The value of '{0}' should be of type '{1}' but is of type '{2}'.");
 
   /**
    * Parameters:
    * 0: the missing key
    */
   static const TransformSetErrorCode missingKey =
-      TransformSetErrorCode('missingKey', "Missing the required key '{0}'.");
+      TransformSetErrorCode('missing_key', "Missing the required key '{0}'.");
 
   /**
    * No parameters.
    */
   static const TransformSetErrorCode missingTemplateEnd = TransformSetErrorCode(
-      'missingTemplateEnd', "Missing the end brace for the template.");
+      'missing_template_end', "Missing the end brace for the template.");
 
   /**
    * Parameters:
    * 0: the missing key
    */
   static const TransformSetErrorCode undefinedVariable = TransformSetErrorCode(
-      'undefinedVariable', "The variable '{0}' is not defined.");
+      'undefined_variable', "The variable '{0}' is not defined.");
 
   /**
    * Parameters:
    * 0: the unsupported key
    */
-  static const TransformSetErrorCode unsupportedKey =
-      TransformSetErrorCode('unsupportedKey', "The key '{0}' isn't supported.");
+  static const TransformSetErrorCode unsupportedKey = TransformSetErrorCode(
+      'unsupported_key', "The key '{0}' isn't supported.");
 
   /**
    * Parameters:
    * 0: the message produced by the YAML parser
    */
   static const TransformSetErrorCode yamlSyntaxError =
-      TransformSetErrorCode('yamlSyntaxError', "Parse error: {0}");
+      TransformSetErrorCode('yaml_syntax_error', "Parse error: {0}");
 
   /// Initialize a newly created error code.
   const TransformSetErrorCode(String name, String message,

pkg/analysis_server/test/lsp/diagnostic_test.dart

@@ -250,6 +250,21 @@ void f() {
     expect(diagnostics, isNull);
   }
 
+  Future<void> test_fixDataFile() async {
+    var fixDataPath = join(projectFolderPath, 'lib', 'fix_data.yaml');
+    var fixDataUri = Uri.file(fixDataPath);
+    newFile(fixDataPath, content: '''
+version: latest
+''').path;
+
+    final firstDiagnosticsUpdate = waitForDiagnostics(fixDataUri);
+    await initialize();
+    final initialDiagnostics = await firstDiagnosticsUpdate;
+    expect(initialDiagnostics, hasLength(1));
+    expect(initialDiagnostics.first.severity, DiagnosticSeverity.Error);
+    expect(initialDiagnostics.first.code, 'invalid_value');
+  }
+
   Future<void> test_fromPlugins_dartFile() async {
     await checkPluginErrorsForFile(mainFilePath);
   }

pkg/analysis_server/test/src/services/correction/fix/bulk/add_const_test.dart

@@ -9,10 +9,36 @@ import 'bulk_fix_processor.dart';
 
 void main() {
   defineReflectiveSuite(() {
+    defineReflectiveTests(AddConstToConstructorTest);
     defineReflectiveTests(AddConstToImmutableConstructorTest);
   });
 }
 
+@reflectiveTest
+class AddConstToConstructorTest extends BulkFixProcessorTest {
+  @override
+  String get lintCode => LintNames.prefer_const_constructors;
+
+  /// Disabled in BulkFixProcessor.
+  @failingTest
+  Future<void> test_singleFile() async {
+    addMetaPackage();
+    await resolveTestUnit(r'''
+class C {
+  const C([C c]);
+}
+var c = C(C());
+''');
+    // TODO (pq): results are incompatible w/ `unnecessary_const`
+    await assertHasFix(r'''
+class C {
+  const C([C c]);
+}
+var c = const C(const C());
+''');
+  }
+}
+
 @reflectiveTest
 class AddConstToImmutableConstructorTest extends BulkFixProcessorTest {
   @override

pkg/analysis_server/test/src/services/correction/fix/bulk/replace_new_with_const_test.dart

@@ -0,0 +1,41 @@
+// Copyright (c) 2020, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+import 'package:analysis_server/src/services/linter/lint_names.dart';
+import 'package:test_reflective_loader/test_reflective_loader.dart';
+
+import 'bulk_fix_processor.dart';
+
+void main() {
+  defineReflectiveSuite(() {
+    defineReflectiveTests(ReplaceNewWithConstTest);
+  });
+}
+
+@reflectiveTest
+class ReplaceNewWithConstTest extends BulkFixProcessorTest {
+  @override
+  String get lintCode => LintNames.prefer_const_constructors;
+
+  /// Disabled in BulkFixProcessor.
+  @failingTest
+  Future<void> test_singleFile() async {
+    await resolveTestUnit(r'''
+class C {
+  const C();
+}
+main() {
+  print('${new C()} ${new C()}');
+}
+''');
+    await assertHasFix(r'''
+class C {
+  const C();
+}
+main() {
+  print('${const C()} ${const C()}');
+}
+''');
+  }
+}

pkg/analysis_server/test/src/services/correction/fix/bulk/test_all.dart

@@ -49,6 +49,7 @@ import 'remove_unnecessary_new_test.dart' as remove_unnecessary_new;
 import 'rename_to_camel_case_test.dart' as rename_to_camel_case;
 import 'replace_colon_with_equals_test.dart' as replace_colon_with_equals;
 import 'replace_final_with_const_test.dart' as replace_final_with_const;
+import 'replace_new_with_const_test.dart' as replace_new_with_const;
 import 'replace_null_with_closure_test.dart' as replace_null_with_closure;
 import 'replace_with_conditional_assignment_test.dart'
     as replace_with_conditional_assignment;
@@ -102,6 +103,7 @@ void main() {
     replace_with_conditional_assignment.main();
     replace_colon_with_equals.main();
     replace_final_with_const.main();
+    replace_new_with_const.main();
     replace_null_with_closure.main();
     replace_with_is_empty.main();
     replace_with_tear_off.main();