Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[web] Run engine tests on Safari locally by launching safari installed on MacOS#14555

Merged
nturgut merged 4 commits into
flutter-team-archive:masterfrom
nturgut:safari_tests
Dec 19, 2019
Merged

[web] Run engine tests on Safari locally by launching safari installed on MacOS#14555
nturgut merged 4 commits into
flutter-team-archive:masterfrom
nturgut:safari_tests

Conversation

@nturgut

@nturgut nturgut commented Dec 18, 2019

Copy link
Copy Markdown
Contributor

Launching safari browser on the operating system (MacOS).

Current status:
+261 methods passing -46 methods failed.

how to run:

felt test --browser=safari

or

felt test --browser=safari --safari-version=system

soon I'll also add felt test --browser=safari --safari-version=latest for downloading and running tech preview version once we have LUCI setup.

fixes: flutter/flutter#47231

@nturgut nturgut added the Work in progress (WIP) Not ready (yet) for review! label Dec 18, 2019
@auto-assign auto-assign Bot requested a review from liyuqian December 18, 2019 18:24
@nturgut nturgut requested review from yjbanov and removed request for liyuqian December 18, 2019 18:24
@nturgut nturgut added affects: tests and removed Work in progress (WIP) Not ready (yet) for review! labels Dec 18, 2019
… a permision pop-up. Note: there is no headless mode in safari
@nturgut nturgut changed the title [web] Launch safari browser [web] Run engine tests on Safari locally by launching safari installed on MacOS Dec 18, 2019
Comment thread lib/web_ui/dev/safari_installation.dart Outdated
Comment thread lib/web_ui/dev/safari_installation.dart Outdated
Comment thread lib/web_ui/dev/safari_installation.dart Outdated
@nturgut nturgut merged commit f5b877a into flutter-team-archive:master Dec 19, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 20, 2019
flar pushed a commit to flutter/flutter that referenced this pull request Dec 20, 2019
* bb118c6 Roll src/third_party/dart 8d11c1dce64a..642f8d052fd7 (1 commits) (flutter-team-archive/engine#14574)

* 09c434d Use ELF for Dart AOT snapshots on Fuchsia. (flutter-team-archive/engine#13896)

* e1e7851 Roll fuchsia/sdk/core/linux-amd64 from VdBKA... to uFFWW... (flutter-team-archive/engine#14575)

* f317f8f Roll src/third_party/skia c76ac8e325c7..77742c350371 (1 commits) (flutter-team-archive/engine#14576)

* 2ba5633 instructions for running firefox/safari tests (flutter-team-archive/engine#14562)

* 9cf1e46 Roll src/third_party/dart 642f8d052fd7..7113fc79a83c (3 commits) (flutter-team-archive/engine#14578)

* f5b877a [web] Run engine tests on Safari locally by launching safari installed on MacOS (flutter-team-archive/engine#14555)

* 68d9196 Fix DOM-based ParagraphRuler.hitTest() (flutter-team-archive/engine#14504)

* bb65df8 Roll src/third_party/skia 77742c350371..a8352ccaae37 (8 commits) (flutter-team-archive/engine#14579)

* ad1ab56 Roll src/third_party/dart 7113fc79a83c..e50d98cd5651 (8 commits) (flutter-team-archive/engine#14580)

* 22413ef Update formatting in web_ui scene bulder to match flutter style and dartfmt. (flutter-team-archive/engine#14539)

* 6e825e7 Roll fuchsia/sdk/core/mac-amd64 from Ykb4b... to f51Q_... (flutter-team-archive/engine#14584)

* 1d3bb8c Fix message_loop_fuchsia and thus enable fml_tests and flow_tests for Fuchsia (flutter-team-archive/engine#14583)

* 1f7bb9d Wire up OpacityLayer to Scenic (flutter-team-archive/engine#14577)

* 11db035 Roll src/third_party/skia a8352ccaae37..87e9ddb675b6 (11 commits) (flutter-team-archive/engine#14585)

* bd58af7 Roll src/third_party/dart e50d98cd5651..141fcfa61092 (3 commits) (flutter-team-archive/engine#14586)

* 929b1ed Engine support for ImageFiltered widget (flutter-team-archive/engine#14491)

* 40b84fc Fix lint warnings across web_ui, add missing browserEngine case in text field. (flutter-team-archive/engine#14535)

* ea1d330 Roll fuchsia/sdk/core/linux-amd64 from uFFWW... to 25LzW... (flutter-team-archive/engine#14587)

* 854d5f8 Roll src/third_party/skia 87e9ddb675b6..7e2dea568299 (1 commits) (flutter-team-archive/engine#14589)
filmil pushed a commit to filmil/engine that referenced this pull request Mar 13, 2020
…d on MacOS (flutter-team-archive#14555)

* Launch safari browser

* Adding safari tests to supported browsers

* changing the directrory for redirect file. This works without opening a permision pop-up. Note: there is no headless mode in safari

* addressing reviewer comments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

[web] Support Safari on MacOS web engine unit tests

3 participants