Skip to content

Commit 4e18b58

Browse files
[pigeon] Consolidate main Dart unit tests (flutter#4641)
Eliminates `flutter_null_safe_unit_tests`, moving the tests into the same consolidated test plugin that we are using for integration tests. This removes duplicate generation of `pigeons/` Dart output, and makes the overall test harness more consistent (e.g., we can now remove some bespoke generation code in the test scripts). (`mock_handler_tests` are still separate; that will be left for another PR.) This also removes analysis and Dart unit testing from the set of tests run by `run_tests.dart` (the backing for custom-tests in CI) since, as checked-in packages, they are already analyzed and tested by the normal CI steps that do Dart analysis and testing, so that was duplicating CI work. They are still part of `test.dart` for convenience of local testing during Pigeon development.
1 parent d00c1f9 commit 4e18b58

30 files changed

+703
-2973
lines changed

packages/pigeon/platform_tests/flutter_null_safe_unit_tests/.gitignore

Lines changed: 0 additions & 46 deletions
This file was deleted.

packages/pigeon/platform_tests/flutter_null_safe_unit_tests/.metadata

Lines changed: 0 additions & 10 deletions
This file was deleted.

packages/pigeon/platform_tests/flutter_null_safe_unit_tests/README.md

Lines changed: 0 additions & 3 deletions
This file was deleted.

packages/pigeon/platform_tests/flutter_null_safe_unit_tests/lib/.gitignore

Lines changed: 0 additions & 9 deletions
This file was deleted.

0 commit comments

Comments
 (0)