KEMBAR78
[Testing] Disable the failed ValidateWebViewScreenshot test in net 10.0 by HarishKumarSF4517 · Pull Request #30013 · dotnet/maui · GitHub
Skip to content

Conversation

@HarishKumarSF4517
Copy link
Contributor

@HarishKumarSF4517 HarishKumarSF4517 commented Jun 16, 2025

This pull request introduces a conditional compilation directive to the Issue14825 test class to exclude the ValidateWebViewScreenshot test fails on Android due to a Image capturing in net10.0 release mode. The most important changes are as follows:

Test adjustments for platform-specific issues:

  • Added #if TEST_FAILS_ON_ANDROID and #endif directives around the ValidateWebViewScreenshot test method to conditionally exclude it on Android. This is related to issue #30010. (src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue14825.cs, [1] [2]

@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Jun 16, 2025
@dotnet-policy-service
Copy link
Contributor

Hey there @@HarishKumarSF4517! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@dotnet-policy-service dotnet-policy-service bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Jun 16, 2025
@ghost ghost added area-testing Unit tests, device tests testing-green-ci labels Jun 16, 2025
@HarishKumarSF4517 HarishKumarSF4517 marked this pull request as ready for review June 16, 2025 14:54
@Copilot Copilot AI review requested due to automatic review settings June 16, 2025 14:54
@HarishKumarSF4517 HarishKumarSF4517 requested a review from a team as a code owner June 16, 2025 14:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR conditionally disables the failing ValidateWebViewScreenshot test on Android to mitigate failures in net10.0 release mode.

  • Wrapped the test in #if TEST_FAILS_ON_ANDROID and #endif directives.
  • Provides a reference (#30010) for further context on the failure.
Comments suppressed due to low confidence (1)

src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue14825.cs:14

  • Disabling the test may reduce test coverage for the screenshot functionality on Android. Consider adding detailed documentation or a follow-up task to re-enable the test once the underlying Android issue (#30010) is resolved.
#if TEST_FAILS_ON_ANDROID //For more information see: https://github.com/dotnet/maui/issues/30010

@jsuarezruiz
Copy link
Contributor

/azp run MAUI-UITests-public

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ghost ghost requested a review from rmarinho June 16, 2025 15:40
@rmarinho rmarinho merged commit bc9aeb8 into dotnet:net10.0 Jun 16, 2025
78 of 79 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-testing Unit tests, device tests community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration testing-green-ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants