Skip to content

Commit 27b5209

Browse files
committed
Documented breaking change
1 parent 06c6fc0 commit 27b5209

File tree

1 file changed

+22
-2
lines changed

1 file changed

+22
-2
lines changed

BREAKING_CHANGES.md

Lines changed: 22 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ You can read the original problem specification and discussion [here](https://g
2323

2424
If your project has been impacted by this particular breaking change, you might consider reevaluate the correctness of the affected tests.
2525

26-
## Error Messages
26+
## Error Message Quotes
2727

2828
Some error messages surrounded actual values in double quotes. Others surrounded the values in single quotes. In version 3.0.0 *all* values are surrounded in single quotes.
2929

@@ -33,7 +33,27 @@ Consistency.
3333

3434
### Fix
3535

36-
Amend all effected tests to expect single quotes instead of double quotes.
36+
Amend any affected tests to expect single quotes instead of double quotes.
37+
38+
## Error Message Lambda Expression
39+
40+
In error messages, lambda expressions arguments are now surrounded in a pair of parentheses. For example:
41+
42+
... to pass the given condition (model => (model.Property1 != null))
43+
44+
will now look like this:
45+
46+
... to pass the given condition ((model) => (model.Property1 != null))
47+
48+
As you can see, the argument called `model` is now surrounded in parentheses.
49+
50+
###Reason
51+
52+
FluentMVCTesting now uses [ExpressionToString](https://github.com/JakeGinnivan/ExpressionToString) to humanize expression trees. ExpressionToString surrounds arguments in parentheses.
53+
54+
###Fix
55+
56+
Amend any affected tests to expect lambda expression arguments to be surrounded in parentheses.
3757

3858
# Version 2.0.0
3959

0 commit comments

Comments
 (0)