Skip to content

Conversation

@kzu
Copy link
Member

@kzu kzu commented Sep 27, 2025

We were not including the actual exception message as the first line in the GH comment report.

image

Fixes #85

@kzu kzu added the enhancement New feature or request label Sep 27, 2025
@kzu kzu force-pushed the dev/errormessage branch 3 times, most recently from 9b1aafb to 21eba4e Compare September 27, 2025 23:51
We were not including the actual exception message as the first line in the GH comment report.

Fixes #85
@kzu
Copy link
Member Author

kzu commented Sep 28, 2025

3 failed 5 passed 3 failed 3 skipped
3 failed 5 passed 3 failed 3 skipped
2 failed 6 passed 2 failed 3 skipped

🧪 Details on macOS Unix 15.6.1
❌ Demo.Tests.Fails_With_Complex_StackTrace
System.InvalidOperationException : This should not happen!
   at Demo.Tests.Unexpected() in /_/src/Demo/Tests.cs:line 52
   at Demo.Tests.Run() in /_/src/Demo/Tests.cs:line 50
   at Demo.Tests.<Fails_With_Complex_StackTrace>b__6_0() in /_/src/Demo/Tests.cs:line 36
   at Demo.Tests.Fails_With_Complex_StackTrace() in /_/src/Demo/Tests.cs:line 38
❌ Demo.Tests.Fails_With_Output
Assert.Equal() Failure: Values differ
Expected: 42
Actual:   22
   at Demo.Tests.Fails_With_Output() in /_/src/Demo/Tests.cs:line 30

> It was going so well...
> Yet you never know
> Which is why you sprinkle all these WriteLines 👀

❌ Demo.Tests.FailsOnlyOnUnix
Fails on Unix
   at Demo.Tests.FailsOnlyOnUnix() in /_/src/Demo/Tests.cs:line 48
🧪 Details on Ubuntu 24.04.3 LTS
❌ Demo.Tests.Fails_With_Complex_StackTrace
System.InvalidOperationException : This should not happen!
   at Demo.Tests.Unexpected() in /_/src/Demo/Tests.cs:line 52
   at Demo.Tests.Run() in /_/src/Demo/Tests.cs:line 50
   at Demo.Tests.<Fails_With_Complex_StackTrace>b__6_0() in /_/src/Demo/Tests.cs:line 36
   at Demo.Tests.Fails_With_Complex_StackTrace() in /_/src/Demo/Tests.cs:line 38
❌ Demo.Tests.Fails_With_Output
Assert.Equal() Failure: Values differ
Expected: 42
Actual:   22
   at Demo.Tests.Fails_With_Output() in /_/src/Demo/Tests.cs:line 30

> It was going so well...
> Yet you never know
> Which is why you sprinkle all these WriteLines 👀

❌ Demo.Tests.FailsOnlyOnUnix
Fails on Unix
   at Demo.Tests.FailsOnlyOnUnix() in /_/src/Demo/Tests.cs:line 48
🧪 Details on Microsoft Windows 10.0.26100
❌ Demo.Tests.Fails_With_Complex_StackTrace
System.InvalidOperationException : This should not happen!
   at Demo.Tests.Unexpected() in /_/src/Demo/Tests.cs:line 52
   at Demo.Tests.Run() in /_/src/Demo/Tests.cs:line 50
   at Demo.Tests.<Fails_With_Complex_StackTrace>b__6_0() in /_/src/Demo/Tests.cs:line 36
   at Demo.Tests.Fails_With_Complex_StackTrace() in /_/src/Demo/Tests.cs:line 38
❌ Demo.Tests.Fails_With_Output
Assert.Equal() Failure: Values differ
Expected: 42
Actual:   22
   at Demo.Tests.Fails_With_Output() in /_/src/Demo/Tests.cs:line 30

> It was going so well...
> Yet you never know
> Which is why you sprinkle all these WriteLines 👀

from dotnet-trx v42.42.380-pr90 on .NET 8.0.20 with 💜 by @devlooped

@kzu kzu merged commit 31c2d8a into main Sep 28, 2025
11 checks passed
@kzu kzu deleted the dev/errormessage branch September 28, 2025 22:02
@devlooped devlooped locked and limited conversation to collaborators Oct 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ErrorInfo.Message not displayed

2 participants