Skip to content

Test: re-enable test disabled by .NET SDK bug#5005

Merged
dtivel merged 1 commit intodevfrom
dev-dtivel-28131
Jan 12, 2023
Merged

Test: re-enable test disabled by .NET SDK bug#5005
dtivel merged 1 commit intodevfrom
dev-dtivel-28131

Conversation

@dtivel
Copy link
Copy Markdown
Contributor

@dtivel dtivel commented Jan 12, 2023

Bug

Fixes: NuGet/Home#12151

Regression? Last working version: .NET 6.x SDK

Description

.NET 7 SDK introduced a regression in parsing multiple key-value pairs as a property during pack. See dotnet/sdk#28131. I disabled a test impacted by this regression.

This PR re-enables that test.

Separately, I'll cherry-pick this into the https://github.com/NuGet/NuGet.Client/tree/release-6.4.x branch when I can.

PR Checklist

  • PR has a meaningful title

  • PR has a linked issue.

  • Described changes

  • Tests

    • Automated tests added
    • OR
    • Test exception
    • OR
    • N/A
  • Documentation

    • Documentation PR or issue filled
    • OR
    • N/A

@dtivel dtivel requested a review from a team as a code owner January 12, 2023 17:27
@dtivel dtivel merged commit b5a4466 into dev Jan 12, 2023
@dtivel dtivel deleted the dev-dtivel-28131 branch January 12, 2023 19:06
dtivel added a commit that referenced this pull request Jan 12, 2023
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.

Test: re-enable test after .NET 7 SDK has fix

2 participants