这是indexloc提供的服务,不要输入任何密码
Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Set automatic simulator rotation in scenario test #24985

Merged
merged 1 commit into from
Mar 15, 2021

Conversation

jmagman
Copy link
Member

@jmagman jmagman commented Mar 13, 2021

The PlatformViewRotation iOS scenario test only passes if the Simulator app has Rotate Device Automatically selected.
Screen Shot 2021-03-12 at 4 18 42 PM

Currently, the infrastructure team is manually selecting this menu item when installing Xcode. Instead, set the default that controls this setting on the host before running the test.

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides].
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See [testing the engine] for instructions on
    writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the [CLA].
  • All existing and new tests are passing.
  • The reviewer has submitted any presubmit flakes in this PR using the [engine presubmit flakes form] before re-triggering the failure.

@jmagman jmagman self-assigned this Mar 13, 2021
@jmagman jmagman requested a review from gaaclarke March 13, 2021 00:25
@jmagman
Copy link
Member Author

jmagman commented Mar 13, 2021

@godofredoc if this works the infra team should hopefully not need to toggle this manually.

Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

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

lgtm. I'm assuming there's no option to xcodebuild for this?

@jmagman
Copy link
Member Author

jmagman commented Mar 13, 2021

lgtm. I'm assuming there's no option to xcodebuild for this?

No, it's a setting in the Simulator app for all simulators, not per-simulator and not per-build. Also, no guarantee this doesn't break in a future version of Xcode, I poked until I found this myself, it's not documented anywhere.

@jmagman jmagman merged commit 1ea7dc6 into flutter:master Mar 15, 2021
@jmagman jmagman deleted the rotate-sim branch March 15, 2021 22:50
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 16, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 16, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 16, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 16, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 16, 2021
zanderso pushed a commit to flutter/flutter that referenced this pull request Mar 16, 2021
* b2d02f7 Roll Fuchsia Linux SDK from GsRYVri0-... to aRqEDMjwC... (flutter/engine#24990)

* 3a1a3e6 Roll Skia from 7854da39b3c1 to 4fb13e026b6b (20 revisions) (flutter/engine#24993)

* e3a84f9 Fixed issue where the gpu disable syncswitch was being overridden after init. (flutter/engine#24958)

* aa83691 fix selectable text selections are not announced in voice over (flutter/engine#24933)

* 04b0451 fixes reference retaining issue in flutter text input plugin (flutter/engine#24768)

* 1ea7dc6 Set automatic simulator rotation in scenario test (flutter/engine#24985)

* 38977a5 Implement AXPlatformNodeBase::GetInstanceCountForTesting (flutter/engine#24999)

* a25b0de Roll Fuchsia Mac SDK from xOxFrRRO6... to pmsuWkRQA... (flutter/engine#24997)

* 3270c87 Roll Skia from 4fb13e026b6b to 1aa25c3217b6 (25 revisions) (flutter/engine#25001)

* 96d5104 [deferred components] Handle base module loading units (flutter/engine#24983)

* 6a3d8c7 Roll Skia from 1aa25c3217b6 to ead52dc068d5 (1 revision) (flutter/engine#25005)

* 0f52360 Hardware Keyboard: macOS (flutter/engine#23469)
hjfreyer pushed a commit to hjfreyer/engine that referenced this pull request Mar 22, 2021
chriscraws pushed a commit to chriscraws/engine that referenced this pull request Mar 23, 2021
duanqz pushed a commit to duanqz/engine that referenced this pull request Apr 16, 2021
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.

4 participants