Exempt leak test config files (#4145)

test/flutter_test_config.dart is a test configuration file, so should not trigger the testing requirement.
flutter · Jan 2, 2025 · 7c7e1d9 · 7c7e1d9
1 parent 5b582dc
commit 7c7e1d9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/app_dart/lib/src/request_handlers/github/webhook_subscription.dart b/app_dart/lib/src/request_handlers/github/webhook_subscription.dart
@@ -539,6 +539,7 @@ class GithubWebhookSubscription extends SubscriptionHandler {
         filename.startsWith('dev/integration_tests/') ||
         filename.startsWith('docs/') ||
         filename.startsWith('${engineBasePath}docs/') ||
+        filename.endsWith('test/flutter_test_config.dart') ||
         // ↓↓↓ Begin engine specific paths ↓↓↓
         filename == 'DEPS' || // note: in monorepo; DEPS is still at the root.
         isBuildPythonScript ||

diff --git a/app_dart/test/request_handlers/github/webhook_subscription_test.dart b/app_dart/test/request_handlers/github/webhook_subscription_test.dart
@@ -791,13 +791,15 @@ void main() {
           PullRequestFile()..filename = 'dev/devicelab/lib/versions/gallery.dart',
           PullRequestFile()..filename = 'dev/integration_tests/some_package/android/build.gradle',
           PullRequestFile()..filename = 'docs/app_anatomy.svg',
+          PullRequestFile()..filename = 'example/test/flutter_test_config.dart',
           PullRequestFile()..filename = 'impeller/fixtures/dart_tests.dart',
           PullRequestFile()..filename = 'impeller/golden_tests/golden_tests.cc',
           PullRequestFile()..filename = 'impeller/playground/playground.cc',
           PullRequestFile()..filename = 'shell/platform/embedder/tests/embedder_test_context.cc',
           PullRequestFile()..filename = 'shell/platform/embedder/fixtures/main.dart',
           PullRequestFile()..filename = 'testing/test_gl_surface.h',
           PullRequestFile()..filename = 'tools/clangd_check/bin/main.dart',
+          PullRequestFile()..filename = 'test/flutter_test_config.dart',
         ]),
       );