[go: nahoru, domu]

Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

[instrumentation_adapter] Add stub iOS implementation and example app #2052

Merged
merged 6 commits into from
Sep 9, 2019

Conversation

collinjackson
Copy link
Contributor
@collinjackson collinjackson commented Sep 9, 2019

Added a simple example app.
Fixes iOS runtime failures when running instrumentation adapter tests using Flutter driver.

$ flutter drive test_driver/widget.dart 
Using device iPhone X.
Starting application: test_driver/widget.dart
Running Xcode build...                                                  
 ├─Assembling Flutter resources...                           7.5s
 └─Compiling, linking and signing...                         2.8s
Xcode build done.                                           11.8s
flutter: Observatory listening on http://127.0.0.1:49678/uxZDVLh_nLM=/
[info ] FlutterDriver: Connecting to Flutter application at http://127.0.0.1:49678/uxZDVLh_nLM=/
[trace] FlutterDriver: Isolate found with number: 3000794421129895
[trace] FlutterDriver: Isolate is paused at start.
[trace] FlutterDriver: Attempting to resume isolate
[trace] FlutterDriver: Waiting for service extension
flutter: 00:00 +0: verify text
[info ] FlutterDriver: Connected to Flutter application.
flutter: 00:00 +1: (tearDownAll)
flutter: 00:00 +2: All tests passed!
Stopping application instance.

Required for landing #2025

@collinjackson collinjackson changed the title Add stub iOS implementation and example app [instrumentation_adapter] Add stub iOS implementation and example app Sep 9, 2019
Copy link
Contributor
@digiter digiter left a comment

Choose a reason for hiding this comment

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

LGTM

super.onCreate(savedInstanceState)
GeneratedPluginRegistrant.registerWith(this)
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you intent to mix the use of java and kotlin? I'm fine with either way, but usually keeping it consistent is better to reduce confusion or dependency issues.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hmm, I'll guess I'll switch it to the Java version. Flutter create is defaulting to Kotlin but most of the plugins repo is still Java.

@collinjackson collinjackson merged commit cd38eb3 into flutter:master Sep 9, 2019
noelmansour pushed a commit to snowble/plugins that referenced this pull request Nov 13, 2019
…flutter#2052)

* Add stub iOS implementation and example app
* Gracefully handle running tests on the host
mormih pushed a commit to mormih/plugins that referenced this pull request Nov 17, 2019
…flutter#2052)

* Add stub iOS implementation and example app
* Gracefully handle running tests on the host
sungmin-park pushed a commit to sungmin-park/flutter-plugins that referenced this pull request Dec 17, 2019
…flutter#2052)

* Add stub iOS implementation and example app
* Gracefully handle running tests on the host
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
3 participants