Skip to content

Commit

Permalink
Update the conclusion
Browse files Browse the repository at this point in the history
  • Loading branch information
stephenfuqua committed Dec 3, 2018
1 parent 4bb2dda commit 0cb0d96
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -208,13 +208,15 @@ small to read too much into it, but this is an important finding and something
to watch out for. Based on documentation and richness of syntax I would want to
use Fluent Assertions, but if the project has a large number of tests, the small
performance difference could add up to a meaningful increase in total execution
time. If time to fully evaluate is lacking, then I feel that my best choice is
to upgrade Should to Shoudly and perhaps make time to pitch in on improving the
documentation.
time. If time to fully evaluate is lacking, then I feel that my best choices are
to either (a) focus on Assert.That syntax or (b) upgrade Should to Shoudly and
perhaps make time to pitch in on improving the documentation. Leaving existing
Should tests in place would seem to be harmless since the performance is good.

| Framework | Documentation | Richness | Performance |
|----------|-------------|---------|--------------------|
| NUnit3 | ++ | + | / |
| NUnit3 (Classic) | ++ | + | / |
| NUnit3 (Constraint) | ++ | ++ | ++ |
| FluentAssertions | +++ | +++ | - |
| Should | + | + | ++ |
| Shouldly | + | ++ | ++ |

0 comments on commit 0cb0d96

Please sign in to comment.