-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Closed
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.team-toolOwned by Flutter Tool teamOwned by Flutter Tool teamtoolAffects the "flutter" command-line tool. See also t: labels.Affects the "flutter" command-line tool. See also t: labels.triaged-toolTriaged by Flutter Tool teamTriaged by Flutter Tool team
Description
See
flutter/dev/devicelab/bin/tasks/flavors_test.dart
Lines 61 to 64 in d136976
| if (assetManifest.containsKey('assets/free/free.txt')) { | |
| return TaskResult.failure('Assets declared with a flavor not equal to the ' | |
| 'argued --flavor value should not be bundled.'); | |
| } |
and
flutter/dev/devicelab/bin/tasks/flavors_test_ios.dart
Lines 69 to 72 in d136976
| if (assetManifest.containsKey('assets/free/free.txt')) { | |
| return TaskResult.failure('Assets declared with a flavor not equal to the ' | |
| 'argued --flavor value should not be bundled.'); | |
| } |
In these tests, we are building the app with a --flavor of paid. The test code I just linked ensures that we don't bundle assets declared with a flavor of free. However, I think the tests should also test that an asset with a flavor of paid is bundled.
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.team-toolOwned by Flutter Tool teamOwned by Flutter Tool teamtoolAffects the "flutter" command-line tool. See also t: labels.Affects the "flutter" command-line tool. See also t: labels.triaged-toolTriaged by Flutter Tool teamTriaged by Flutter Tool team