Move AndroidX error handler to the end (#83135)

8a7f2dee · Emmanuel Garcia · GitHub · f7bcfa8e · 8a7f2dee · 8a7f2dee
Unverified Commit 8a7f2dee authored May 22, 2021 by Emmanuel Garcia Committed by GitHub May 22, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

gradle_errors.dart packages/flutter_tools/lib/src/android/gradle_errors.dart +1 -1

gradle_errors_test.dart ..._tools/test/general.shard/android/gradle_errors_test.dart +2 -1

No files found.
--- a/packages/flutter_tools/lib/src/android/gradle_errors.dart
+++ b/packages/flutter_tools/lib/src/android/gradle_errors.dart
@@ -72,10 +72,10 @@ final List<GradleHandledError> gradleErrors = <GradleHandledError>[
  permissionDeniedErrorHandler,
  flavorUndefinedHandler,
  r8FailureHandler,
-  androidXFailureHandler,
  minSdkVersion,
  transformInputIssue,
  lockFileDepMissing,
+  androidXFailureHandler, // Keep last since the pattern is broader.
 ];

 // Permission defined error message.

--- a/packages/flutter_tools/test/general.shard/android/gradle_errors_test.dart
+++ b/packages/flutter_tools/test/general.shard/android/gradle_errors_test.dart
@@ -28,12 +28,13 @@ void main() {
          permissionDeniedErrorHandler,
          flavorUndefinedHandler,
          r8FailureHandler,
-          androidXFailureHandler,
          minSdkVersion,
          transformInputIssue,
          lockFileDepMissing,
+          androidXFailureHandler,
        ])
      );
+      expect(gradleErrors.last, equals(androidXFailureHandler));
    });
  });