Skip to content

Unskip test case - #50031

Merged
nagilson merged 1 commit into
mainfrom
Simon/Unskip_testcase_RunPublishItemsOutputGroupOutputsTest
Aug 4, 2025
Merged

nagilson merged 1 commit into
mainfrom
Simon/Unskip_testcase_RunPublishItemsOutputGroupOutputsTest

Conversation

@SimonZhao888

@SimonZhao888 SimonZhao888 commented Aug 4, 2025 •

Copy link
Copy Markdown
Member

Re-enable the test case: RunPublishItemsOutputGroupOutputsTest(specifyRid: true, singleFile: true)

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 unskips a previously disabled test case by removing the Skip parameter from an InlineData attribute. The test was originally skipped due to a GitHub issue (#49926) but is now being re-enabled.

  • Removed the Skip parameter from one InlineData test case
  • Re-enabled testing of the scenario with specifyRid=true and singleFile=true

[Theory]
[InlineData(true, false)]
[InlineData(true, true, Skip = "https://github.com/dotnet/sdk/issues/49926")]
[InlineData(true, true)]

Copilot AI Aug 4, 2025

Copy link

Choose a reason for hiding this comment

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

When unskipping a test that was previously disabled due to an issue, it's important to verify that the underlying issue (#49926) has been resolved. Consider adding a comment or updating the commit message to indicate why this test is now safe to run.

Copilot generated this review using guidance from repository custom instructions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants