Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions TUnit.Assertions.Tests/AssertConditions/BecauseTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -144,4 +144,26 @@ await Assert.That(variable).IsFalse().Because(because1)
var exception = await Assert.ThrowsAsync<AssertionException>(action);
await Assert.That(exception.Message).Contains(because1).And.Contains(because2);
}

[Test]
public async Task Because_Message_Appears_Inline_With_Expectation()
{
var expectedMessage = """
Expected to be false, because this is the reason
but found True

at Assert.That(variable).IsFalse().Because("this is the reason")
""";

var variable = true;

var action = async () =>
{
await Assert.That(variable).IsFalse().Because("this is the reason");
};

var exception = await Assert.ThrowsAsync<AssertionException>(action);
await Assert.That(exception.Message.NormalizeLineEndings())
.IsEqualTo(expectedMessage.NormalizeLineEndings());
}
}
18 changes: 10 additions & 8 deletions TUnit.Assertions/Core/Assertion.cs
Original file line number Diff line number Diff line change
Expand Up @@ -212,22 +212,24 @@ public OrContinuation<TValue> Or
/// </summary>
protected Exception CreateException(AssertionResult result)
{
var message = $"""
Expected {GetExpectation()}
but {result.Message}

at {Context.ExpressionBuilder}
""";
var expectation = GetExpectation();

if (_becauseMessage != null)
{
// Check if message already starts with "because" to avoid duplication
// Append because message inline with the expectation
var becausePrefix = _becauseMessage.StartsWith("because ", StringComparison.OrdinalIgnoreCase)
? _becauseMessage
: $"because {_becauseMessage}";
message += $"\n\n{becausePrefix}";
expectation = $"{expectation}, {becausePrefix}";
}

var message = $"""
Expected {expectation}
but {result.Message}

at {Context.ExpressionBuilder}
""";

return new AssertionException(message);
}

Expand Down
Loading