Skip to content

Delete long-disabled Bundle_can_be_renamed test - #131621

Merged
elinor-fung merged 2 commits into
mainfrom
copilot/delete-bundle-can-be-renamed-test
Jul 31, 2026
Merged

Delete long-disabled Bundle_can_be_renamed test#131621
elinor-fung merged 2 commits into
mainfrom
copilot/delete-bundle-can-be-renamed-test

Conversation

Copilot AI commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Bundle_can_be_renamed_while_running has been marked [ActiveIssue] since it was first committed due to non-deterministic failures (shared TestFixture across [Theory] runs). There have been no reported issues in this area in six years, so the (disabled) test wouldn't provide much value.

Fixes #38013

cc @dotnet/appmodel @AaronRobinsonMSFT

Copilot AI review requested due to automatic review settings July 30, 2026 22:50

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.

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
16 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI linked an issue Jul 30, 2026 that may be closed by this pull request
The Bundle_can_be_renamed test (in BundleRename.cs) has been disabled
via [ActiveIssue] since it was initially added due to non-deterministic
failures. Since there have been no issues around this functionality in
six years, delete the test and its associated AppWithWait test asset.

Closes #38013

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 30, 2026 22:55

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.

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

@github-actions github-actions Bot added the area-Single-File only for closed issues label Jul 30, 2026
Copilot AI changed the title [WIP] Remove disabled Bundle_can_be_renamed test Delete long-disabled Bundle_can_be_renamed test Jul 30, 2026
Copilot AI requested a review from elinor-fung July 30, 2026 23:00
@elinor-fung
elinor-fung marked this pull request as ready for review July 30, 2026 23:05
@elinor-fung elinor-fung added area-Host and removed area-Single-File only for closed issues labels Jul 30, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@elinor-fung
elinor-fung merged commit 34a3c5d into main Jul 31, 2026
88 checks passed
@elinor-fung
elinor-fung deleted the copilot/delete-bundle-can-be-renamed-test branch July 31, 2026 21:28
@github-project-automation github-project-automation Bot moved this to Done in AppModel Jul 31, 2026
@dotnet-milestone-bot dotnet-milestone-bot Bot added this to the 11.0-rc1 milestone Aug 2, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Enable Apphost.Bundle.Test Bundle_can_be_renamed()

4 participants