Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Testing] Remove IgnoreIfPlatforms usage on UITests #22525

Closed
jsuarezruiz opened this issue May 20, 2024 · 0 comments · Fixed by #22526
Closed

[Testing] Remove IgnoreIfPlatforms usage on UITests #22525

jsuarezruiz opened this issue May 20, 2024 · 0 comments · Fixed by #22526
Assignees
Labels
area-testing Unit tests, device tests platform/android 🤖 platform/iOS 🍎 platform/macOS 🍏 macOS / Mac Catalyst platform/windows 🪟 s/triaged Issue has been reviewed t/bug Something isn't working
Milestone

Comments

@jsuarezruiz
Copy link
Contributor

jsuarezruiz commented May 20, 2024

Description

Remove IgnoreIfPlatforms usage on UITests.

IgnoreIfPlatforms is a extension method used until now in some UITests to ignore the test on some platforms. However, the sample app was launched, navigated to the issue page and then ignored. This takes some valious time. In this PR, we are removing the extension method and including some Ignore attributes to ignore the test directly.

Steps to Reproduce

No response

Link to public reproduction project repository

No response

Version with bug

8.0.40 SR5

Is this a regression from previous behavior?

No, this is something new

Last version that worked well

Unknown/Other

Affected platforms

iOS, Android, Windows, macOS

Affected platform versions

No response

Did you find any workaround?

No response

Relevant log output

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-testing Unit tests, device tests platform/android 🤖 platform/iOS 🍎 platform/macOS 🍏 macOS / Mac Catalyst platform/windows 🪟 s/triaged Issue has been reviewed t/bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants