Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand deprecation policy to package:flutter_test #122323

Open
1 of 4 tasks
Piinks opened this issue Mar 9, 2023 · 5 comments
Open
1 of 4 tasks

Expand deprecation policy to package:flutter_test #122323

Piinks opened this issue Mar 9, 2023 · 5 comments
Assignees
Labels
a: tests "flutter test", flutter_test, or one of our tests. framework flutter/packages/flutter repository. See also f: labels. P4 Priority 4 issue (default for bugs, things we're likely to work on) team Infra upgrades, team productivity, code health, technical debt. See also team: labels. tech-debt Technical debt, code quality, testing, etc.

Comments

@Piinks
Copy link
Contributor

Piinks commented Mar 9, 2023

This is the tracking issue for the proposal to expand the deprecation policy to package:flutter_test

When first introduced, we decided to first focus the application of the deprecation policy on just the flutter package. During that time we worked on the process and gathered feedback from users. This process has become pretty stable, and so we are looking to expand it to other packages, with the first being flutter_test.

This will involve:

If we were to do this after the next release (not the current round that completed with #119580 but the next round), these would be up for removal:

class / method property deprecated in
AnimationSheetBuilder display 2.3
AnimationSheetBuilder sheetSize 2.3
TestWidgetsFlutterBinding addTime 2.6
TestWidgetsFlutterBinding runAsync 2.6
TestWidgetsFlutterBinding.runTest timeout 2.6
AutomatedTestWidgetsFlutterBinding.runTest timeout 2.6
LiveTestWidgetsFlutterBinding.runTest timeout 2.6
testWidgets initialTimeout 2.6

cc @goderbauer @pdblasi-google

@Piinks Piinks added a: tests "flutter test", flutter_test, or one of our tests. team Infra upgrades, team productivity, code health, technical debt. See also team: labels. framework flutter/packages/flutter repository. See also f: labels. P4 Priority 4 issue (default for bugs, things we're likely to work on) tech-debt Technical debt, code quality, testing, etc. labels Mar 9, 2023
@Piinks Piinks self-assigned this Mar 9, 2023
@gslender
Copy link
Contributor

gslender commented Mar 9, 2023

Is this appropriate place to raise issues with depreciation of methods and classes (in the flutter_test package) that are still needed because no equivalent method or class exists?

@Piinks
Copy link
Contributor Author

Piinks commented Mar 9, 2023

Is this appropriate place to raise issues with depreciation of methods and classes that are still needed because no equivalent method or class exists?

Definitely! I think all of the listed APIs have migration paths, but if we are missing one we certainly would love to know!

@workerbee22
Copy link

workerbee22 commented Mar 9, 2023

If we are extending to flutter_test would you consider also integration_test (now also part of the framework)? Then all core testing components covered 🤔 Because lets face it, Dart fixes for flutter_test would largely also appy to Dart code used within integration_test?

@Piinks
Copy link
Contributor Author

Piinks commented Mar 9, 2023

If we are extending to flutter_test would you consider also integration_test (now also part of the framework)?

Great question! The long term goal is to expand the policy to cover all packages in flutter/flutter, and maybe even dart:ui. 🙂

We're just doing this at a considerate pace since it means more potential breaking changes for developers.

@pdblasi-google
Copy link
Contributor

@Piinks I added a link to a ticket for adding dart fix support to the description. I plan on starting on it in the next couple weeks, hopefully early next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: tests "flutter test", flutter_test, or one of our tests. framework flutter/packages/flutter repository. See also f: labels. P4 Priority 4 issue (default for bugs, things we're likely to work on) team Infra upgrades, team productivity, code health, technical debt. See also team: labels. tech-debt Technical debt, code quality, testing, etc.
Projects
None yet
Development

No branches or pull requests

4 participants