added script to massage stable channel tests

gaaclarke · gaaclarke · commit 755e897bec83 · 2021-10-25T16:10:53.000-07:00
diff --git a/.cirrus.yml b/.cirrus.yml
@@ -197,6 +197,8 @@ task:
         - export CIRRUS_CHANGE_MESSAGE=""
         - export CIRRUS_COMMIT_MESSAGE=""
         - ./script/tool_runner.sh lint-android # must come after build-examples
+      stable_channel_conditional_script:
+        - dart ./ci/stable_conditional.dart $CHANNEL
       native_unit_test_script:
         # Unsetting CIRRUS_CHANGE_MESSAGE and CIRRUS_COMMIT_MESSAGE as they
         # might include non-ASCII characters which makes Gradle crash.
diff --git a/ci/stable_conditional.dart b/ci/stable_conditional.dart
@@ -0,0 +1,69 @@
+// Copyright 2013 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+//
+// stable_conditional.dart
+//
+// Performs simple find and replace operations for conditional compilation
+// before executing stable channel tests.
+//
+// Example input:
+//   int main() {
+//     // FLUTTER_STABLE_CHANNEL_BEGIN
+//     printf("hello world\n");
+//     // FLUTTER_STABLE_CHANNEL_REPLACE
+//     // printf("goodbye world\n");
+//     // FLUTTER_STABLE_CHANNEL_END
+//   }
+//
+// Example output:
+//   int main() {
+//     printf("goodbye world\n");
+//   }
+
+import 'dart:convert' show LineSplitter;
+import 'dart:io' show Directory, FileSystemEntity, File;
+
+final RegExp _isSourceRegex =
+    RegExp(r'\.cc$|\.java$|\.m$\.h$|\.c$|\.swift$|\.kt$');
+final RegExp _replacer = RegExp(
+    r'^\s*// FLUTTER_STABLE_CHANNEL_BEGIN(.*?)^\s*// FLUTTER_STABLE_CHANNEL_REPLACE(.*?)^\s*// FLUTTER_STABLE_CHANNEL_END',
+    multiLine: true,
+    dotAll: true);
+final RegExp _commentRemover = RegExp(r'^(\s*)\/+\s*(.*)');
+const String _newline = '\n';
+
+bool _isSourceFile(FileSystemEntity entity) =>
+    _isSourceRegex.hasMatch(entity.path);
+
+void _process(FileSystemEntity entity) {
+  const LineSplitter splitter = LineSplitter();
+  final String text = File(entity.path).readAsStringSync();
+  String replaced = '';
+  int index = 0;
+  for (final RegExpMatch match in _replacer.allMatches(text)) {
+    replaced += text.substring(index, match.start);
+    for (final String line in splitter.convert(match.group(2)!)) {
+      final RegExpMatch? commentRemoverMatch = _commentRemover.firstMatch(line);
+      if (commentRemoverMatch != null) {
+        replaced += commentRemoverMatch.group(1)! +
+            commentRemoverMatch.group(2)! +
+            _newline;
+      }
+    }
+    index = match.end;
+  }
+  if (replaced.isNotEmpty) {
+    replaced += text.substring(index, text.length);
+    File(entity.path).writeAsStringSync(replaced);
+    print('modified: ${entity.path}');
+  }
+}
+
+void main(List<String> args) {
+  final String channel = args[0];
+  if (channel == 'stable') {
+    final Directory dir = Directory('.');
+    dir.list(recursive: true).where(_isSourceFile).forEach(_process);
+  }
+}
diff --git a/packages/android_intent/android/src/test/java/io/flutter/plugins/androidintent/MethodCallHandlerImplTest.java b/packages/android_intent/android/src/test/java/io/flutter/plugins/androidintent/MethodCallHandlerImplTest.java
@@ -55,7 +55,11 @@ public void startListening_registersChannel() {
     methodCallHandler.startListening(messenger);
 
     verify(messenger, times(1))
+        // FLUTTER_STABLE_CHANNEL_BEGIN
         .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class), eq(null));
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   @Test
@@ -67,9 +71,15 @@ public void startListening_unregistersExistingChannel() {
     methodCallHandler.startListening(secondMessenger);
 
     // Unregisters the first and then registers the second.
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null, null);
     verify(secondMessenger, times(1))
         .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class), eq(null));
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
+    // verify(secondMessenger, times(1))
+    //     .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   @Test
@@ -79,7 +89,11 @@ public void stopListening_unregistersExistingChannel() {
 
     methodCallHandler.stopListening();
 
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null, null);
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   @Test
@@ -88,7 +102,11 @@ public void stopListening_doesNothingWhenUnset() {
 
     methodCallHandler.stopListening();
 
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null, null);
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null);
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   @Test
diff --git a/packages/camera/camera/android/src/test/java/io/flutter/plugins/camera/DartMessengerTest.java b/packages/camera/camera/android/src/test/java/io/flutter/plugins/camera/DartMessengerTest.java
@@ -32,10 +32,14 @@ public class DartMessengerTest {
   private static class FakeBinaryMessenger implements BinaryMessenger {
     private final List<ByteBuffer> sentMessages = new ArrayList<>();
 
+    // TODO(aaclarke): Remove when https://github.com/flutter/engine/pull/29147 is on master.
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     @Override
-    BinaryMessenger.TaskQueue makeBackgroundTaskQueue() {
+    public BinaryMessenger.TaskQueue makeBackgroundTaskQueue() {
       return null;
     }
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // FLUTTER_STABLE_CHANNEL_END
 
     @Override
     public void send(@NonNull String channel, ByteBuffer message) {
@@ -47,11 +51,17 @@ public void send(@NonNull String channel, ByteBuffer message, BinaryReply callba
       send(channel, message);
     }
 
+    // TODO(aaclarke): Remove when https://github.com/flutter/engine/pull/29147 is on master.
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     @Override
     public void setMessageHandler(
         @NonNull String channel,
         BinaryMessageHandler handler,
         @Nullable BinaryMessenger.TaskQueue taskQueue) {}
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // @Override
+    // public void setMessageHandler(@NonNull String channel, BinaryMessageHandler handler) {}
+    // FLUTTER_STABLE_CHANNEL_END
 
     List<ByteBuffer> getMessages() {
       return new ArrayList<>(sentMessages);
diff --git a/packages/quick_actions/quick_actions/android/src/test/java/io/flutter/plugins/quickactions/QuickActionsTest.java b/packages/quick_actions/quick_actions/android/src/test/java/io/flutter/plugins/quickactions/QuickActionsTest.java
@@ -33,10 +33,14 @@ public class QuickActionsTest {
   private static class TestBinaryMessenger implements BinaryMessenger {
     public MethodCall lastMethodCall;
 
+    // TODO(aaclarke): Remove when https://github.com/flutter/engine/pull/29147 is on master.
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     @Override
-    BinaryMessenger.TaskQueue makeBackgroundTaskQueue() {
+    public BinaryMessenger.TaskQueue makeBackgroundTaskQueue() {
       return null;
     }
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // FLUTTER_STABLE_CHANNEL_END
 
     @Override
     public void send(@NonNull String channel, @Nullable ByteBuffer message) {
@@ -54,13 +58,21 @@ public void send(
       }
     }
 
+    // TODO(aaclarke): Remove when https://github.com/flutter/engine/pull/29147 is on master.
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     @Override
     public void setMessageHandler(
         @NonNull String channel,
         @Nullable BinaryMessageHandler handler,
         @Nullable BinaryMessenger.TaskQueue taskQueue) {
       // Do nothing.
     }
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // @Override
+    // public void setMessageHandler(
+    //    @NonNull String channel,
+    //    @Nullable BinaryMessageHandler handler) {}
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   static final int SUPPORTED_BUILD = 25;
diff --git a/packages/url_launcher/url_launcher/android/src/test/java/io/flutter/plugins/urllauncher/MethodCallHandlerImplTest.java b/packages/url_launcher/url_launcher/android/src/test/java/io/flutter/plugins/urllauncher/MethodCallHandlerImplTest.java
@@ -44,7 +44,11 @@ public void startListening_registersChannel() {
     methodCallHandler.startListening(messenger);
 
     verify(messenger, times(1))
+        // FLUTTER_STABLE_CHANNEL_BEGIN
         .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class), eq(null));
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   @Test
@@ -56,9 +60,15 @@ public void startListening_unregistersExistingChannel() {
     methodCallHandler.startListening(secondMessenger);
 
     // Unregisters the first and then registers the second.
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null, null);
     verify(secondMessenger, times(1))
         .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class), eq(null));
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // verify(firstMessenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
+    // verify(secondMessenger, times(1))
+    //     .setMessageHandler(eq(CHANNEL_NAME), any(BinaryMessageHandler.class));
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   @Test
@@ -68,7 +78,11 @@ public void stopListening_unregistersExistingChannel() {
 
     methodCallHandler.stopListening();
 
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null, null);
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // verify(messenger, times(1)).setMessageHandler(CHANNEL_NAME, null);
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   @Test
@@ -77,7 +91,11 @@ public void stopListening_doesNothingWhenUnset() {
 
     methodCallHandler.stopListening();
 
+    // FLUTTER_STABLE_CHANNEL_BEGIN
     verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null, null);
+    // FLUTTER_STABLE_CHANNEL_REPLACE
+    // verify(messenger, never()).setMessageHandler(CHANNEL_NAME, null);
+    // FLUTTER_STABLE_CHANNEL_END
   }
 
   @Test
