Skip to content

Make it an error for native-test to run no unit tests for a package #85469

@stuartmorgan-g

Description

@stuartmorgan-g

Currently if an Android plugin has no native unit tests, we silently skip it. This means it's easy to accidentally add a plugin without a test harness, or to accidentally break the running of unit tests (e.g., if their location changes but the script isn't updated).

Once #83358 is fixed, or at least further along, we should make it an error for a plugin that has an Android implementation in the package to not run any unit tests, and explicitly --exclude any we still need to have skipped. This makes it much harder to silently do the wrong thing, and is consistent with changes that have been recently made to other test commands in the tooling.

Metadata

Metadata

Labels

P1High-priority issues at the top of the work listc: contributor-productivityTeam-specific productivity, code health, technical debt.p: toolingAffects the flutter_plugin_tools packagepackageflutter/packages repository. See also p: labels.platform-androidAndroid applications specifically

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions