Skip to content

[HouseKeeping] Fix inconsistant namespace in HostApp - #35210

Merged
kubaflo merged 1 commit into
dotnet:inflight/currentfrom
NirmalKumarYuvaraj:test-refactor
Apr 29, 2026
Merged

[HouseKeeping] Fix inconsistant namespace in HostApp#35210
kubaflo merged 1 commit into
dotnet:inflight/currentfrom
NirmalKumarYuvaraj:test-refactor

Conversation

@NirmalKumarYuvaraj

@NirmalKumarYuvaraj NirmalKumarYuvaraj commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

Description of Change

This pull request standardizes the namespace declarations for several test case files related to issues in the test suite. The changes ensure that all files use the correct Microsoft.Maui.TestCases.Tests.Issues namespace, improving consistency and maintainability.

Namespace corrections:

  • Changed the namespace in Issue22075.cs from Microsoft.Maui.TestCases.Tests.Issue to Microsoft.Maui.TestCases.Tests.Issues to match the naming convention.
  • Updated the namespace in Issue28968.cs from Microsoft.Maui.TestCases.Tests.Tests.Issues to Microsoft.Maui.TestCases.Tests.Issues.
  • Updated the namespace in Issue29588.cs from Microsoft.Maui.TestCases.Tests.Tests.Issues to Microsoft.Maui.TestCases.Tests.Issues.
  • Changed the namespace in Issue33227.cs from Maui.Controls.TestCases.Tests.Issues to Microsoft.Maui.TestCases.Tests.Issues.
@github-actions

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 -- 35210

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 35210"
@dotnet-policy-service dotnet-policy-service Bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Apr 29, 2026
@NirmalKumarYuvaraj
NirmalKumarYuvaraj marked this pull request as ready for review April 29, 2026 10:40
Copilot AI review requested due to automatic review settings April 29, 2026 10:40

Copilot AI 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.

Pull request overview

This PR standardizes the namespace declarations for a handful of Appium issue tests in TestCases.Shared.Tests so they align with the rest of the issue-test suite’s namespace layout.

Changes:

  • Update Issue33227 to use Microsoft.Maui.TestCases.Tests.Issues instead of Maui.Controls.TestCases.Tests.Issues.
  • Fix two issue tests that were under the duplicated namespace segment ...Tests.Tests.Issues.
  • Fix a singular/plural mismatch (Issue -> Issues) for an issue test namespace.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue33227.cs Renames namespace to the standard Microsoft.Maui.TestCases.Tests.Issues.
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue29588.cs Removes duplicated Tests.Tests segment from namespace.
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue28968.cs Removes duplicated Tests.Tests segment from namespace.
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue22075.cs Fixes namespace pluralization (Issue -> Issues).
using UITest.Core;

namespace Maui.Controls.TestCases.Tests.Issues;
namespace Microsoft.Maui.TestCases.Tests.Issues;

Copilot AI Apr 29, 2026

Copy link

Choose a reason for hiding this comment

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

PR title/description mention fixing namespaces in the HostApp, but this change is in TestCases.Shared.Tests (Appium test project). Please update the PR title/description to reflect the actual scope (tests project), and consider fixing the spelling of “inconsistant” -> “inconsistent”.

Copilot uses AI. Check for mistakes.
@kubaflo
kubaflo changed the base branch from main to inflight/current April 29, 2026 21:11
@kubaflo
kubaflo merged commit 70cf74a into dotnet:inflight/current Apr 29, 2026
33 of 36 checks passed
@github-actions github-actions Bot added this to the .NET 10 SR7 milestone Apr 29, 2026
github-actions Bot pushed a commit that referenced this pull request May 6, 2026
<!--
!!!!!!! MAIN IS THE ONLY ACTIVE BRANCH. MAKE SURE THIS PR IS TARGETING
MAIN. !!!!!!!
-->

### Description of Change

<!-- Enter description of the fix in this section -->
This pull request standardizes the namespace declarations for several
test case files related to issues in the test suite. The changes ensure
that all files use the correct `Microsoft.Maui.TestCases.Tests.Issues`
namespace, improving consistency and maintainability.

**Namespace corrections:**

* Changed the namespace in `Issue22075.cs` from
`Microsoft.Maui.TestCases.Tests.Issue` to
`Microsoft.Maui.TestCases.Tests.Issues` to match the naming convention.
* Updated the namespace in `Issue28968.cs` from
`Microsoft.Maui.TestCases.Tests.Tests.Issues` to
`Microsoft.Maui.TestCases.Tests.Issues`.
* Updated the namespace in `Issue29588.cs` from
`Microsoft.Maui.TestCases.Tests.Tests.Issues` to
`Microsoft.Maui.TestCases.Tests.Issues`.
* Changed the namespace in `Issue33227.cs` from
`Maui.Controls.TestCases.Tests.Issues` to
`Microsoft.Maui.TestCases.Tests.Issues`.
@github-actions github-actions Bot locked and limited conversation to collaborators May 30, 2026
@github-actions github-actions Bot modified the milestones: .NET 10 SR7, .NET 10 SR9 Jul 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration t/housekeeping ♻︎

4 participants