Skip to content

Commit 2ef1e3f

Browse files
ByteBlastByteBlast
ByteBlast
authored and
ByteBlast
committed
Merge pull request #51 from ByteBlast/master
Bumped version and documented breaking change
2 parents 18774c5 + c332c90 commit 2ef1e3f

File tree

2 files changed

+26
-1
lines changed

2 files changed

+26
-1
lines changed

BREAKING_CHANGES.md

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,28 @@
1+
# Version 3.0.0
2+
3+
## ShouldRedirectTo Method
4+
5+
It used to be the case that when you invoked `ShouldRedirectTo<TController>` where `TController` is the same type as the `Controller` under test like this:
6+
7+
var sut = new HomeController();
8+
sut.WithCallTo(c => c.Index())
9+
.ShouldRedirectTo<HomeController>(c => c.Index());
10+
11+
an `ActionResultAssertionException` would be thrown.
12+
13+
An exception is no longer thrown.
14+
15+
### Reason
16+
17+
There is no reason why a test like this one should fail.
18+
19+
You can read the original problem specification and discussion [here](https://github.com/TestStack/TestStack.FluentMVCTesting/issues/47).
20+
21+
22+
### Fix
23+
24+
If your project has been impacted by this particular breaking change, you might consider reevaluate the correctness of the affected tests.
25+
126
# Version 2.0.0
227

328
## ShouldRenderFileStream Method

NextVersion.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
2.3.0
1+
3.0.0

0 commit comments

Comments
 (0)