Skip to content

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Apr 13, 2016

Refactor the package:flutter_test API according to the new design based on finder objects.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might want to have a tester.renderObjectOf(...) function similar to the tester.stateOf function.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Also added tester.elementOf(...)

@abarth
Copy link
Contributor

abarth commented Apr 13, 2016

LGTM

I'd probably add a renderObjectOf before landing. I'm sure we'll iterate on this after landing, but it's such a big patch that you'll probably want to get it in before it gets out of date.

After landing, please email flutter-dev explaining how to use the new system and giving some pointers about converting old tests to the new system.

Thanks!

@yjbanov yjbanov force-pushed the finder-objects branch 4 times, most recently from c9dce82 to cd0d42d Compare April 14, 2016 06:06
@yjbanov yjbanov merged commit f3a4f72 into flutter:master Apr 14, 2016
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants