-
Notifications
You must be signed in to change notification settings - Fork 329
Add support for running tests from "integration_test" with "flutter test -d deviceId" #3133
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
24927a5
to
c49315d
Compare
Thanks! Just tested this out manually (running, debugging) on macOS with a few devices connected and it seems to work great (after patching the version in |
Great! I'm not sure if there may be anything else, but perhaps I think I have integration tests here that do a bunch of things for flutter test (inc. expression eval etc.), so maybe I can get them to run over an integration test too to give some more coverage. I'll fix this branch up and work on the tests and let you know if I find any issues. |
c49315d
to
1db97ea
Compare
…ted via "flutter test"
This appears to not be supported anymore (at least when there is a plugin, like integration_test in dev_dependencies).
1db97ea
to
675b6ca
Compare
I've re-tested this and added a few tests, and all seems good to me. Based on flutter/flutter@9e3eadb, I'm setting the version number to enable this as |
Awesome! |
Fixes #3009.