-
Notifications
You must be signed in to change notification settings - Fork 30.1k
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.
Reactions are currently unavailable
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