Skip to content

Add macOS Tests (Part 3: Widgets)#48997

Merged
gspencergoog merged 1 commit intoflutter:masterfrom
gspencergoog:macos_tests_widgets
Jan 17, 2020
Merged

Add macOS Tests (Part 3: Widgets)#48997
gspencergoog merged 1 commit intoflutter:masterfrom
gspencergoog:macos_tests_widgets

Conversation

@gspencergoog
Copy link
Copy Markdown
Contributor

Description

This contains the test changes for adding TargetPlatform.macOS, which was done in #43457.

The main goal of this PR is to enable tests in the widgets library that are currently running only on iOS to also run on macOS, but only for the tests where that makes sense. For instance, we don't run any of the haptic feedback tests on macOS.

Related Issues

Tests

  • Converts most iOS-only tests to be variant tests that test both iOS and macOS.

Breaking Change

  • No, this is not a breaking change.

@fluttergithubbot fluttergithubbot added the framework flutter/packages/flutter repository. See also f: labels. label Jan 16, 2020
@gspencergoog gspencergoog mentioned this pull request Jan 16, 2020
1 task
Copy link
Copy Markdown
Contributor

@justinmc justinmc left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks for doing all the work to do this the "good" way!

@gspencergoog gspencergoog merged commit f013b25 into flutter:master Jan 17, 2020
@gspencergoog gspencergoog deleted the macos_tests_widgets branch March 13, 2020 16:09
@lock
Copy link
Copy Markdown

lock bot commented Apr 2, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@lock lock bot locked and limited conversation to collaborators Apr 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants