Skip to content

Commit 976f5c5

Browse files
committed
Fix VirtualFileResult and PhysicalFileResult messages.
1 parent afec7e7 commit 976f5c5

File tree

4 files changed

+10
-6
lines changed

4 files changed

+10
-6
lines changed

src/FluentAssertions.AspNetCore.Mvc/PhysicalFileResultAssertions.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,8 @@ public FileResultAssertions WithFileName(string expectedFileName, string reason
5656
Execute.Assertion
5757
.ForCondition(string.Equals(expectedFileName, actualFileName, StringComparison.OrdinalIgnoreCase))
5858
.BecauseOf(reason, reasonArgs)
59-
.FailWith(FailureMessages.CommonFailMessage, "PhysicalFileResult.FileName", expectedFileName, actualFileName);
59+
.WithDefaultIdentifier("PhysicalFileResult.FileName")
60+
.FailWith(FailureMessages.CommonFailMessage2, expectedFileName, actualFileName);
6061
return this;
6162
}
6263

src/FluentAssertions.AspNetCore.Mvc/VirtualFileResultAssertions.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,8 @@ public FileResultAssertions WithFileName(string expectedFileName, string reason
5656
Execute.Assertion
5757
.ForCondition(string.Equals(expectedFileName, actualFileName, StringComparison.OrdinalIgnoreCase))
5858
.BecauseOf(reason, reasonArgs)
59-
.FailWith(FailureMessages.CommonFailMessage, "VirtualFileResult.FileName", expectedFileName, actualFileName);
59+
.WithDefaultIdentifier("VirtualFileResult.FileName")
60+
.FailWith(FailureMessages.CommonFailMessage2, expectedFileName, actualFileName);
6061
return this;
6162
}
6263

tests/FluentAssertions.AspNetCore.Mvc.Tests/PhysicalFileResultAssertions_Tests.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
using FluentAssertions.AspNetCore.Mvc.Tests.Helpers;
2+
using FluentAssertions.Mvc.Tests.Helpers;
23
using Microsoft.AspNetCore.Mvc;
34
using System;
45
using Xunit;
@@ -26,11 +27,11 @@ public void WithFileName_GivenUnexpectedValue_ShouldFail()
2627
string actualFileName = "Test1.txt";
2728
string expectedFileName = "Test2.txt";
2829
ActionResult result = TestDataGenerator.CreatePhysicalFileResult(actualFileName);
29-
var failureMessage = "Expected \"PhysicalFileResult.FileName\" to be '\"Test2.txt\"' but found '\"Test1.txt\"'";
30+
var failureMessage = FailureMessageHelper.ExpectedContextToBeXButY("PhysicalFileResult.FileName", expectedFileName, actualFileName);
3031

3132
Action a = () => result.Should()
3233
.BePhysicalFileResult()
33-
.WithFileName(expectedFileName);
34+
.WithFileName(expectedFileName, "it is {0}", 10);
3435

3536
a.Should().Throw<Exception>()
3637
.WithMessage(failureMessage);

tests/FluentAssertions.AspNetCore.Mvc.Tests/VirtualFileResultAssertions_Tests.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
using FluentAssertions.AspNetCore.Mvc.Tests.Helpers;
2+
using FluentAssertions.Mvc.Tests.Helpers;
23
using Microsoft.AspNetCore.Mvc;
34
using System;
45
using Xunit;
@@ -26,11 +27,11 @@ public void WithFileName_GivenUnexpectedValue_ShouldFail()
2627
string actualFileName = "Test1.txt";
2728
string expectedFileName = "Test2.txt";
2829
ActionResult result = TestDataGenerator.CreateVirtualFileResult(actualFileName);
29-
var failureMessage = "Expected \"VirtualFileResult.FileName\" to be '\"Test2.txt\"' but found '\"Test1.txt\"'";
30+
var failureMessage = FailureMessageHelper.ExpectedContextToBeXButY("VirtualFileResult.FileName", expectedFileName, actualFileName);
3031

3132
Action a = () => result.Should()
3233
.BeVirtualFileResult()
33-
.WithFileName(expectedFileName);
34+
.WithFileName(expectedFileName, "it is {0}", 10);
3435

3536
a.Should().Throw<Exception>()
3637
.WithMessage(failureMessage);

0 commit comments

Comments
 (0)