-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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 teamtriaged-toolTriaged by Flutter Tool teamTriaged by Flutter Tool team
Description
Discovered while working on #150575
Example:
flutter/packages/flutter_tools/test/commands.shard/permeable/build_apk_test.dart
Lines 112 to 125 in 2b218fd
| testUsingContext('logs success', () async { | |
| final String projectPath = await createProject(tempDir, | |
| arguments: <String>['--no-pub', '--template=app']); | |
| await runBuildApkCommand(projectPath); | |
| expect(testUsage.events, contains( | |
| const TestUsageEvent( | |
| 'tool-command-result', | |
| 'apk', | |
| label: 'success', | |
| ), | |
| )); | |
| }, |
I'm assuming these tests should be making sure we are sending GA4 events as well (using package:unified_analytics).
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 teamtriaged-toolTriaged by Flutter Tool teamTriaged by Flutter Tool team