Skip to content

Conversation

@Tenno641
Copy link

Fixes xunit/xunit#3446

Code fixer was incorrectly removing second argument (userMessage) when converting Assert.True(!false, userMessage: "") to Assert.False(false)

  • Modified BooleanAssertsShouldNotBeNegatedFixer to preserve all arguments
  • Added tests to ensure userMessage (both positional and named) is preserved by the code fix.

Note: This is a clean version of my earlier PR, which I closed due to some git issues. Sorry about the noise.

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.

xUnit2022 code fixer removes the second argument

1 participant