commit 95214adca76275849cf28e428c41e7a83f814fce
author Konstantin Scheglov <scheglov@google.com> Mon Jul 06 16:11:58 2020 -0700
committer GitHub <noreply@github.com> Mon Jul 06 16:11:58 2020 -0700
tree 84e00ab7ac466adf4ea671bceb4a0e2876690cd4
parent cfc94922b19b672b599110dd34ea173251a3ac18

Update to analyzer: ^0.39.12, and use overrideKnownFeaturesAsync() instead of dart:cli. (#2250)

pubspec.yaml

diff --git a/pubspec.yaml b/pubspec.yaml
index 55e20fe..555a7b6 100644
--- a/pubspec.yaml
+++ b/pubspec.yaml
@@ -7,7 +7,7 @@
   sdk: '>=2.7.0 <3.0.0'
 
 dependencies:
-  analyzer: ^0.39.11
+  analyzer: ^0.39.12
   args: '>=1.5.0 <2.0.0'
   collection: ^1.2.0
   cli_util: ^0.1.3+2

test/experiment_options_test.dart

diff --git a/test/experiment_options_test.dart b/test/experiment_options_test.dart
index 10f1b03..b0421b8 100644
--- a/test/experiment_options_test.dart
+++ b/test/experiment_options_test.dart
@@ -5,7 +5,6 @@
 /// Unit tests for lib/src/experiment_options.dart.
 library dartdoc.experiment_options_test;
 
-import 'dart:cli';
 import 'dart:io';
 
 import 'package:analyzer/src/dart/analysis/experiments.dart';
@@ -19,9 +18,9 @@
   ExperimentalFeature defaultOnNotExpired, defaultOffNotExpired;
   ExperimentalFeature defaultOnExpired, defaultOffExpired;
 
-  void withSyntheticExperimentalFeatures(
-    void Function() operation,
-  ) {
+  Future<void> withSyntheticExperimentalFeatures(
+    Future<void> Function() operation,
+  ) async {
     defaultOnNotExpired = ExperimentalFeature(
       index: 0,
       enableString: 'a',
@@ -55,16 +54,14 @@
       firstSupportedVersion: null,
     );
 
-    overrideKnownFeatures(
+    await overrideKnownFeaturesAsync(
       {
         'a': defaultOnNotExpired,
         'b': defaultOffNotExpired,
         'c': defaultOnExpired,
         'd': defaultOffExpired,
       },
-      () {
-        operation();
-      },
+      operation,
     );
   }
 
@@ -78,25 +75,20 @@
   });
 
   group('Experimental options test', () {
-    void withExperimentOptions(
+    Future<void> withExperimentOptions(
       void Function(DartdocOptionSet) operation,
-    ) {
-      withSyntheticExperimentalFeatures(() {
-        // The enclosing function expects only synchronous function argument.
-        // But `fromOptionGenerators` is asynchronous.
-        // So, we have to use `waitFor` to adapt it.
-        var experimentOptions = waitFor(
-          DartdocOptionSet.fromOptionGenerators(
-            'dartdoc',
-            [createExperimentOptions],
-          ),
+    ) async {
+      await withSyntheticExperimentalFeatures(() async {
+        var experimentOptions = await DartdocOptionSet.fromOptionGenerators(
+          'dartdoc',
+          [createExperimentOptions],
         );
         operation(experimentOptions);
       });
     }
 
-    test('Defaults work for all options', () {
-      withExperimentOptions((experimentOptions) {
+    test('Defaults work for all options', () async {
+      await withExperimentOptions((experimentOptions) {
         experimentOptions.parseArguments([]);
         var tester = DartdocOptionContext(experimentOptions, emptyTempDir);
         expect(tester.experimentStatus.isEnabled(defaultOnNotExpired), isTrue);
@@ -107,8 +99,8 @@
       });
     });
 
-    test('Overriding defaults works via args', () {
-      withExperimentOptions((experimentOptions) {
+    test('Overriding defaults works via args', () async {
+      await withExperimentOptions((experimentOptions) {
         // Set all experiments to non-default values.
         experimentOptions.parseArguments([
           '--enable-experiment',