search

matkoch / resharper-fluentassertions

MIT License
13 stars 2 forks source link

Broken Quick-fixes : NUnit assertion can be replaced with FluentAssertions equivalents #23

Open Purgator opened 2 years ago

Purgator commented 2 years ago

Hello there,

If I write an assertion with nUnit I get a gentle message that tells me a migration to fluentAssertions is possible.

image

However, I don't get any quick fix. I tried several cases and I cannot get any quick fix. I'm pretty sure I used it in the past and was working very well.

Plugin version : 2022.1.0-eap03 Rider version : 2022.1.2

DmitriyMaksimov commented 2 years ago

I have the same issue - installed the plugin to convert tests, but can't get the quick fix.

matkoch commented 2 years ago

See #22. The quick-fix is not broken, it just does not exist for this particular assertion type.

Purgator commented 2 years ago

See #22. The quick-fix is not broken, it just does not exist for this particular assertion type.

Can you send me a quick fix that works then ? So I can try. Assert.That() being the most common NUnit assertion, I am really surprised that is not supported. I tried different kinds of assertions and none works. I cannot prove it but I'm pretty sure I used this plugin not long ago to convert some assertions.

inyutin-maxim commented 2 years ago

Hi guys. I can offer you as a temporary solution the Meziantou.FluentAssertions Analyzers package, I think later I will return to the project and continue develop migrations