-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Closed
flutter/packages
#4440Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work lista: tests"flutter test", flutter_test, or one of our tests"flutter test", flutter_test, or one of our testsc: contributor-productivityTeam-specific productivity, code health, technical debt.Team-specific productivity, code health, technical debt.p: palette_generatorPalette generator packagePalette generator packagepackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.team-ecosystemOwned by Ecosystem teamOwned by Ecosystem teamtriaged-ecosystemTriaged by Ecosystem teamTriaged by Ecosystem team
Description
Currently the Dart unit tests all use dart:io to load test image, which means we need to opt this package out of the new browser mode tests that I'm bringing up in flutter/packages. Since the package itself supports web, the tests should be reworked to not load test files directly from disk.
Once that's done, we can remove the dart_tests.yaml from the package, so tests will run on CI.
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work lista: tests"flutter test", flutter_test, or one of our tests"flutter test", flutter_test, or one of our testsc: contributor-productivityTeam-specific productivity, code health, technical debt.Team-specific productivity, code health, technical debt.p: palette_generatorPalette generator packagePalette generator packagepackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.team-ecosystemOwned by Ecosystem teamOwned by Ecosystem teamtriaged-ecosystemTriaged by Ecosystem teamTriaged by Ecosystem team