It's too late to complain. But there is one thing that annoys me every time. I use ReSharper, and every time i type
with open bracket, ReSharper puts closed bracket and puts my cursor inside. So i get
Should(|) with a cursor inside. And I need to press one more key (right) to move cursor out. As I understand that is because the "Should" method has optional "message" parameter, which however is very rarely used.
I wish that "Should" method wouldn't have this parameter. Instead it might use some explicit way to provide failure message. For example:
something.Should().Not.Be.Null().OnFailure("something should not be null");