fluentassertions.analyzers

fluentassertions.analyzers copied to clipboard

Consider adding support for debugging through Visual Studio

[{"_id":"63626d998041c95dfb1c71f6","body":"It's possible to debug a test and step into the analyzer's code. I'm not very familiar with the Visual Studio debug instance.\r\n\r\nI'll be happy to add it (pull request :wink:)","issue_id":1662062721744,"origin_id":398295633,"user_origin_id":9786571,"create_time":1529391692,"update_time":1529391704,"id":1667394969696,"updated_at":"2022-11-02T13:16:09.696000Z","created_at":"2022-11-02T13:16:09.696000Z"}]

The default Roslyn analyzer template allows one to run the analyzer in a Visual Studio Debug instance, which makes debugging your analyzers incredibly easy. This project does not have Visual...

ErikSchierboom

support for nullables assertions

[]

https://github.com/fluentassertions/fluentassertions/blob/5.2.0/docs/_data/tips/nullables.yml

Meir017

Missing negation when transforming OnlyContain

[{"_id":"6362661fcecf4e081a26f7da","body":"Here's how others are negating expressions:\r\nhttps:\/\/github.com\/JosefPihrt\/Roslynator\/blob\/9098bd81a98e5b57ce65546628cb8d8349ca888f\/source\/Core\/CSharp\/Helpers\/LogicalNegationHelper.cs","issue_id":1662062721750,"origin_id":336620032,"user_origin_id":919634,"create_time":1507969850,"update_time":1507969850,"id":1667393055055,"updated_at":"2022-11-02T12:44:15.055000Z","created_at":"2022-11-02T12:44:15.055000Z"}]

When applying the codefix for `CollectionShouldNotContainProperty`, the comparison inside the lambda should also be negated. ```c# actual.Should().OnlyContain(x => x.OtherProperty == expectedValue); // before actual.Should().NotContain(x => x.OtherProperty == expectedValue); // after...

jnyrup

Test cases

[]

Nice project! I've attached a project with failing tests for all the before and after examples I listed in the Fluent Assertions docs. [UnitTests2.zip](https://github.com/Meir017/FluentAssertions.BestPractices/files/1367205/UnitTests2.zip)

jnyrup

Common DiagnosticId prefix for all analyzer rules

[]

### Description I added fluentassertions.analyzers to a solution (with >100 projects) that I can I fix/improve my assertions. But based on the missing common prefix in the DiagnosticId/Code it is...

kirchsth

Rewrite order when handling `Assert.AreEqual`

[{"_id":"6640af144ecfda50de0c726b","body":"@cervengoc great suggestion!","issue_id":1712636690164,"origin_id":1930755822,"user_origin_id":9786571,"create_time":1707253812,"update_time":1707253812,"id":1715515156060,"updated_at":"2024-05-12T11:59:16.059000Z","created_at":"2024-05-12T11:59:16.059000Z"}]

First or all, thank you for this project, looks very useful. Second, sorry if this is a duplicate, I've tried to look around in the open or closed issues. ###...

cervengoc

Request: Give diagnosis when ThrowAsync etc. is not awaited

[]

### Description I find that the analyzers should warn people if they say `.Should().ThrowAsync()` and similar, without actually awaiting the task produced by Fluent Assertions. ### Complete minimal example reproducing...

JeppeSN

tests: use dotnet 8 for testing

[]

Meir017

Add analyzers and code fixers for Shouldly

[]

for this https://github.com/shouldly/shouldly

Meir017

Add DateTime tips analyzers

[]

see https://github.com/fluentassertions/fluentassertions/pull/2422/files

Meir017