Skip to content

[Testing] Refactoring Feature Matrix UITest Cases for SearchBar Control - #38191

Draft
NafeelaNazhir wants to merge 6 commits into
dotnet:mainfrom
NafeelaNazhir:searchbar-refactoring
Draft

[Testing] Refactoring Feature Matrix UITest Cases for SearchBar Control#38191
NafeelaNazhir wants to merge 6 commits into
dotnet:mainfrom
NafeelaNazhir:searchbar-refactoring

Conversation

@NafeelaNazhir

Copy link
Copy Markdown
Contributor

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

This pull request enhances the SearchBar feature matrix test pages by expanding the set of configurable properties and improving the test UI for better usability and coverage. The changes include adding new bindable properties, updating the options page to expose more features, adding Material3 test variants, and refining input handling for better reliability.

SearchBar feature expansion and UI improvements:

  • SearchBarControlPage.xaml.cs now resets the shared view model via SearchBarViewModel.Reset() instead of recreating and rebinding the SearchBar on every navigation, improving reliability of the options flow.
  • The options page (SearchBarOptionsPage.xaml/.xaml.cs) introduces new controls for configuring SearchIconColor, Background, and ReturnType, and fixes the TextTransform/FontAutoScalingEnabled radio button handling so selections are applied correctly.
  • SearchBarViewModel.cs adds backing properties for SearchIconColor, ReturnType, and Background, along with a Reset() method to restore default values between test runs.
  • Material3_SearchBarControlPage.xaml.cs binds the new Background, ReturnType, and SearchIconColor properties for Material3 coverage.

Test coverage:

  • SearchBarFeatureTests.cs is updated with new test cases covering FlowDirection, IsEnabled, SearchIconColor, BackgroundColor, Shadow with Background, text changed events, and TextTransform behavior.
  • SearchBarMaterial3FeatureTests.cs adds Material3-specific coverage for SearchIconColor and BackgroundColor.

Issues Identified

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 38191

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 38191"

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service dotnet-policy-service Bot added the community ✨ Community Contribution label Sep 4, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hey there @@NafeelaNazhir! 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 Sep 4, 2026
@Ahamed-Ali Ahamed-Ali added the area-controls-searchbar SearchBar control label Sep 4, 2026
@sheiksyedm

Copy link
Copy Markdown
Contributor

/azp run maui-pr-uitests

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@vishnumenon2684

Copy link
Copy Markdown
Contributor

/azp run maui-pr-uitests , maui-pr-devicetests

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 2 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-controls-searchbar SearchBar control community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration platform/android platform/ios

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants