Skip to content

Compatibility pass on flutter/foundation tests for JavaScript compilation. (1) - #33349

Merged
jonahwilliams merged 6 commits into
flutter:masterfrom
jonahwilliams:web_testing_1
Jun 7, 2019
Merged

jonahwilliams merged 6 commits into
flutter:masterfrom
jonahwilliams:web_testing_1

Conversation

@jonahwilliams

Copy link
Copy Markdown
Contributor

Description

Flags certain test cases in foundation using package:test tags in order to facilitate landing support for running all framework tests in a browser.

Example: dart:isolate usage, 64 bit usage, stack trace introspection.

@Hixie

Hixie commented May 27, 2019

Copy link
Copy Markdown
Contributor

Can we use something less magical and hard to debug? Like skip or something?

Comment thread packages/flutter/test/foundation/diagnostics_test.dart Outdated
Comment thread packages/flutter/test/foundation/diagnostics_test.dart Outdated
@jonahwilliams

Copy link
Copy Markdown
Contributor Author

I've opened #33406 to add browser safe indirection to the Platform getters, as well as a new isBrowser getter to better utilize skip. I'll update these PRs to use skip instead of tags

@jonahwilliams

Copy link
Copy Markdown
Contributor Author

Ahh merge catastrophe, my git-foo fails

@jonahwilliams jonahwilliams reopened this May 29, 2019
Comment thread packages/flutter_test/lib/src/window.dart Outdated
@jonahwilliams
jonahwilliams requested a review from yjbanov June 3, 2019 22:20
@jonahwilliams
jonahwilliams merged commit 42a14b5 into flutter:master Jun 7, 2019
@jonahwilliams
jonahwilliams deleted the web_testing_1 branch June 7, 2019 04:22
tango5614 added a commit to tango5614/flutter that referenced this pull request Jun 7, 2019
* master: (25 commits)
  Increase daemon protocol version for getSupportedPlatforms (flutter#33980)
  skip web test on crazy import (flutter#34017)
  Compatibility pass on flutter/foundation tests for JavaScript compilation. (1) (flutter#33349)
  0602dbb Roll src/third_party/dart 9dcb026b26..6e0d978505 (72 commits) (flutter#34027)
  Add chrome stable to dockerfile and web shard (flutter#33787)
  Codegen an entrypoint for flutter web applications (flutter#33956)
  Revert "Reland "Text inline widgets, TextSpan rework" (flutter#33946)" (flutter#34002)
  Revert "Re-add deprecated method for plugin migration compatibility. (flutter#34006)" (flutter#34022)
  Remove print (flutter#34004)
  Manual roll the engine to land the timing API (flutter#33989)
  Make plugins Swift-first on macOS (flutter#33997)
  Re-add deprecated method for plugin migration compatibility. (flutter#34006)
  make sure version check includes hotfixes (flutter#33459)
  Respond to AndroidView focus events. (flutter#33901)
  Add 'doctor' support for Windows (flutter#33872)
  Add use_frameworks to macOS Podfile template (flutter#33987)
  [Material] Create a themable Range Slider (continuous and discrete) (flutter#31681)
  Updating names to correct versioning convention (flutter#33865)
  Whitelist adb.exe heap corruption exit code. (flutter#33951)
  [flutter_tool] Fix 'q' for Fuchsia profile/debug mode (flutter#33846)
  ...
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

a: tests "flutter test", flutter_test, or one of our tests platform-web Web applications specifically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants