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