-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Update SemanticsUpdateBuilderSpy for the conversion of SemanticsUpdateBuilder into a base class #126570
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This ends up causing some pain if we have to make changes upstream that add a method.
Can we override noSuchMethod or extend Fake on this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added extends Fake
2780079 to
a1d819c
Compare
|
auto label is removed for flutter/flutter, pr: 126570, due to - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label. |
…eBuilder into a base class See flutter#123756
a1d819c to
5a704a3
Compare
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…eBuilder into a base class (flutter#126570) See flutter#123756
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…icsUpdateBuilder into a base class (flutter/flutter#126570)
…icsUpdateBuilder into a base class (flutter/flutter#126570)
See #123756