Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[UI Test] Revert 56771 test #2861

Merged
merged 2 commits into from
Jun 4, 2018
Merged

[UI Test] Revert 56771 test #2861

merged 2 commits into from
Jun 4, 2018

Conversation

samhouts
Copy link
Member

@samhouts samhouts commented May 29, 2018 •
edited
Loading

Description of Change

New test added in #2512 caused 56771 to start failing.

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of the target branch at time of PR
  • Changes adhere to coding standard
  • Consolidate commits as makes sense

@samhouts
Copy link
Member Author

build --uitests

@rmarinho
Copy link
Member

nop :(

https://devdiv.visualstudio.com/DevDiv/Xamarin%20Forms/_releaseProgress?releaseId=108003&environmentId=345944&extensionId=ms.vss-test-web.test-result-in-release-environment-editor-tab&_a=release-environment-extension

@samhouts samhouts changed the title [UI Test] Remove conflicting automationId from 56771 test [UI Test] Revert 56771 test May 30, 2018
@rmarinho
Copy link
Member

build --uitests

1 similar comment
@samhouts
Copy link
Member Author

build --uitests

@samhouts samhouts added the t/housekeeping ♻︎ Internal only changes, won't be included in release notes label May 31, 2018
@samhouts samhouts requested review from rmarinho and kingces95 May 31, 2018 16:54
@rmarinho rmarinho merged commit 4185d8d into 3.0.0 Jun 4, 2018
@samhouts samhouts added this to the 3.0.0 milestone Jun 6, 2018
@samhouts samhouts deleted the fix_56771_test branch July 17, 2018 16:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rmarinho rmarinho Awaiting requested review from rmarinho

@kingces95 kingces95 Awaiting requested review from kingces95

Assignees
No one assigned
Labels
t/housekeeping ♻︎ Internal only changes, won't be included in release notes
Projects
None yet
Milestone
3.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@samhouts @rmarinho