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

[path_provider] Replace path_provider_linux widget tests with simple unit tests#3812

Merged
stuartmorgan-g merged 1 commit into
flutter-team-archive:masterfrom
stuartmorgan-g:path-provider-linux-widget-test-removal
Apr 19, 2021
Merged

[path_provider] Replace path_provider_linux widget tests with simple unit tests#3812
stuartmorgan-g merged 1 commit into
flutter-team-archive:masterfrom
stuartmorgan-g:path-provider-linux-widget-test-removal

Conversation

@stuartmorgan-g

Copy link
Copy Markdown
Contributor

These widget tests have been flaky recently, and there's no reason not to just use unit tests instead.

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 signed the CLA.
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
  • I updated CHANGELOG.md to add a description of the change.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
  • All existing and new tests are passing.

@cyanglaz cyanglaz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@blasten blasten left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks! I was looking into this issue as well

@stuartmorgan-g

Copy link
Copy Markdown
Contributor Author

Landing on red since the removed test is the reason submit-queue is red.

@stuartmorgan-g stuartmorgan-g merged commit e52725a into flutter-team-archive:master Apr 19, 2021
@stuartmorgan-g stuartmorgan-g deleted the path-provider-linux-widget-test-removal branch April 19, 2021 17:19
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 19, 2021
@ditman

ditman commented Apr 19, 2021

Copy link
Copy Markdown
Contributor

Thanks for deflaking these!!

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 19, 2021
bhaveshptl pushed a commit to bhaveshptl/plugins that referenced this pull request Apr 23, 2021
* master: (397 commits)
  [in_app_purchase] Implementation of platform interface (flutter-team-archive#3781)
  [google_sign_in] Add todo WRT correctly setting X-Goog-AuthUser header (flutter-team-archive#3819)
  [tools] fix version check command not working for new packages (flutter-team-archive#3818)
  [camera] android-rework part 1: Base classes to support Android Camera features (flutter-team-archive#3795)
  fix MD (flutter-team-archive#3815)
  Path provider windows crash fix (flutter-team-archive#3814)
  [local_auth] docs update (flutter-team-archive#3103)
  Update PULL_REQUEST_TEMPLATE.md (flutter-team-archive#3801)
  [quick_actions] handle cold start on iOS correctly (flutter-team-archive#3811)
  Replace path_provider_linux widget tests with simple unit tests (flutter-team-archive#3812)
  [sensors] format dart code based on the new dart formatter (flutter-team-archive#3809)
  [google_sign_in] Fix "pick account" on iOS (flutter-team-archive#3805)
  [image_picker_platform_interface] Added pickMultiImage (flutter-team-archive#3782)
  [in_app_purchase] Added currency code and numerical price to product detail model. (flutter-team-archive#3794)
  [local_auth] Fix iOS crash when no localizedReason (flutter-team-archive#3780)
  Fix and update version checks (flutter-team-archive#3792)
  [in_app_purchase] Configured example app to use StoreKit Testing on iOS 14 (flutter-team-archive#3772)
  [local_auth] Unnecessary reassignment in example removed (flutter-team-archive#2983)
  [flutter_webview] Fix `allowsInlineMediaPlayback` ignored on iOS (flutter-team-archive#3791)
  Switch script/tools over to the new analysis options (flutter-team-archive#3777)
  ...
yasargil added a commit to yasargil/plugins that referenced this pull request Apr 28, 2021
* master: (79 commits)
  Fix grammatical error in contributing guide (flutter-team-archive#3217)
  [google_sign_in_web] fix README typos.
  [tool] combine run and runAndExitOnError (flutter-team-archive#3827)
  [camera] android-rework part 2: Android auto focus feature (flutter-team-archive#3796)
  [in_app_purchase_platform_interface] Added additional fields to ProductDetails (flutter-team-archive#3826)
  Move all null safety packages' min dart sdk to 2.12.0 (flutter-team-archive#3822)
  [path_provider_*] code cleanup: sort directives (flutter-team-archive#3823)
  [in_app_purchase] Implementation of platform interface (flutter-team-archive#3781)
  [google_sign_in] Add todo WRT correctly setting X-Goog-AuthUser header (flutter-team-archive#3819)
  [tools] fix version check command not working for new packages (flutter-team-archive#3818)
  [camera] android-rework part 1: Base classes to support Android Camera features (flutter-team-archive#3795)
  fix MD (flutter-team-archive#3815)
  Path provider windows crash fix (flutter-team-archive#3814)
  [local_auth] docs update (flutter-team-archive#3103)
  Update PULL_REQUEST_TEMPLATE.md (flutter-team-archive#3801)
  [quick_actions] handle cold start on iOS correctly (flutter-team-archive#3811)
  Replace path_provider_linux widget tests with simple unit tests (flutter-team-archive#3812)
  [sensors] format dart code based on the new dart formatter (flutter-team-archive#3809)
  [google_sign_in] Fix "pick account" on iOS (flutter-team-archive#3805)
  [image_picker_platform_interface] Added pickMultiImage (flutter-team-archive#3782)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants