-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Closed
flutter/packages
#4347Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: contributor-productivityTeam-specific productivity, code health, technical debt.Team-specific productivity, code health, technical debt.p: video_playerThe Video Player pluginThe Video Player pluginpackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.
Description
flutter/plugins has largely now standardized on the current test_driver/ + integration_test/ file structure for integration tests. video_player has that, but also has a second test that is a test_driver/video_player_test.dart+test_driver/video_player.dart pair. That test is currently skipped, so not currently running, but says it can be un-skipped when a bug is fixed that is in fact already fixed.
We should move that test to the new integration test file, migrate it from flutter_driver to integration_test, and remove the flutter_plugin_tools support for that test structure.
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: contributor-productivityTeam-specific productivity, code health, technical debt.Team-specific productivity, code health, technical debt.p: video_playerThe Video Player pluginThe Video Player pluginpackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.