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

[share] Fix bug on iPad where origin is null and enable XCUITests in the repo #3210

Merged
merged 2 commits into from
Oct 27, 2020

Conversation

cyanglaz
Copy link
Contributor

@cyanglaz cyanglaz commented Oct 27, 2020

xctests enabled xctests in repo, also added a sample test in the share plugin.

This pr includes: #3189 and #3188

Copy link
Member

@jmagman jmagman left a comment

Choose a reason for hiding this comment

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

LGTM based on #3188

@cyanglaz cyanglaz changed the title Merge xctests branch to master [share] Fix bug on iPad where origin is null and enable XCUITests in the repo Oct 27, 2020
@cyanglaz cyanglaz merged commit 642290b into master Oct 27, 2020
@cyanglaz cyanglaz deleted the xctest branch October 27, 2020 23:48
yasargil added a commit to yasargil/plugins that referenced this pull request Oct 30, 2020
* master: (48 commits)
  [video_player] Add toString() to Caption (flutter#3233)
  [google_maps_flutter_web] Show one InfoWindow at a time. (flutter#3224)
  [in_app_purchase] Bump version (flutter#3227)
  [google_maps_flutter] Overhaul lifecycle management in GoogleMapsPlugin (flutter#3213)
  [in_app_purchase] Remove the custom analysis options, fix failing lints. (flutter#3220)
  [video_player]Fixes Playing video from asset on Android (flutter#3123)
  [camera] Added documentation about video not working correctly on Android emulators (flutter#3180)
  Revert "update api"
  update api
  [wifi_info_flutter] Method channel name fixed for android (flutter#3207)
  [share] Fix bug on iPad where `origin` is null and enable XCUITests in the repo (flutter#3210)
  [google_maps_flutter] Clean up google_maps_flutter plugin (flutter#3206)
  Exclude generated_plugin_registrant.cc (flutter#3198)
  broaden the constraint on package:vm_service (flutter#3208)
  Remove unnecessary work around from test in prep for vm_service migration (flutter#3209)
  Add windows directory to examples (flutter#3149)
  [video_player] Upgrade ExoPlayer (flutter#3204)
  [android_alarm_manager] Removed deprecated display1 (flutter#3200)
  [Connectivity] wifi removal (flutter#3173)
  [wifi_info_flutter] make it ready for initial 1.0.0 release  (flutter#3191)
  ...
FlutterSu pushed a commit to FlutterSu/flutter-plugins that referenced this pull request Nov 20, 2020
…n the repo (flutter#3210)

* enable xctest in .cirrus.yml (flutter#3189)

* [share] Fix bug on iPad where `origin` is null and enable XCUITests (flutter#3188)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants