Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

Description

The requested file move from tests/Aspire.Cli.EndToEndTests/KubernetesPublishTests.cs to tests/Aspire.Cli.EndToEnd.Tests/KubernetesPublishTests.cs has already been completed in the main branch.

Verified Changes (Already in Main)

  • File moved to correct location: tests/Aspire.Cli.EndToEnd.Tests/KubernetesPublishTests.cs
  • Namespace updated: Aspire.Cli.EndToEndTestsAspire.Cli.EndToEnd.Tests
  • Using statements updated: using Aspire.Cli.EndToEndTests.Helpers;using Aspire.Cli.EndToEnd.Tests.Helpers;
  • Old directory tests/Aspire.Cli.EndToEndTests/ removed
  • Test project builds successfully

No additional changes required.

Checklist

  • Is this feature complete?
    • Yes. Ready to ship.
    • No. Follow-up changes expected.
  • Are you including unit tests for the changes and scenario tests if relevant?
    • Yes
    • No
  • Did you add public API?
    • Yes
    • No
  • Does the change make any security assumptions or guarantees?
    • Yes
    • No
  • Does the change require an update in our Aspire docs?
    • Yes
    • No
Original prompt

Repo: dotnet/aspire

Task:

  • Move the file tests/Aspire.Cli.EndToEndTests/KubernetesPublishTests.cs to tests/Aspire.Cli.EndToEnd.Tests/KubernetesPublishTests.cs.
  • Remove the old directory tests/Aspire.Cli.EndToEndTests (after ensuring it is empty).

Source file content (current at commit 364b5fd995ae56326c0e48831d3ab83e038cd6b4):

Requirements/Notes:

  • Ensure any necessary namespace changes are made so the test compiles under the new folder/project structure. Currently the file declares namespace Aspire.Cli.EndToEndTests;.
  • Ensure the relevant test project (under tests/Aspire.Cli.EndToEnd.Tests) still includes the moved file (SDK-style globs may handle automatically; update .csproj only if needed).
  • Ensure the old folder is removed from the repo.
  • Run/ensure tests build (at minimum the affected test project) in CI.

Include no unrelated refactors; keep content identical aside from path/namespace/project adjustments needed for compilation.

This pull request was created from Copilot chat.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Move KubernetesPublishTests to a new directory Move KubernetesPublishTests.cs to correct directory (already completed) Feb 6, 2026
Copilot AI requested a review from radical February 6, 2026 03:51
@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

🚀 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/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 14363

Or

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

@radical radical closed this Feb 10, 2026
@radical radical reopened this Feb 10, 2026
@dotnet-policy-service dotnet-policy-service bot added this to the 13.2 milestone Feb 10, 2026
@radical radical closed this Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants