Skip to content

Escape markup in CLI output from errors too#33

Merged
kzu merged 1 commit into
mainfrom
dev/outputescape
Jul 29, 2024
Merged

Escape markup in CLI output from errors too#33
kzu merged 1 commit into
mainfrom
dev/outputescape

Conversation

@kzu

@kzu kzu commented Jul 29, 2024

Copy link
Copy Markdown
Member

No description provided.

@kzu kzu added the bug Something isn't working label Jul 29, 2024
@kzu kzu enabled auto-merge (rebase) July 29, 2024 14:59
@devlooped-bot

devlooped-bot commented Jul 29, 2024 •
edited
Loading

Copy link
Copy Markdown
Member

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

🧪 Details on Ubuntu 22.04.4 LTS
❌ Demo.Tests.Fails_With_Complex_StackTrace 
   at Demo.Tests.Unexpected() in /_/src/Demo/Tests.cs:line 49
   at Demo.Tests.Run() in /_/src/Demo/Tests.cs:line 47
   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 
   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 
   at Demo.Tests.FailsOnlyOnUnix() in /_/src/Demo/Tests.cs:line 45

✅ Demo.Tests.Parameterized(culture: "de")
✅ Demo.Tests.Parameterized(culture: "en")
✅ Demo.Tests.Parameterized(culture: "es")
✅ Demo.Tests.Parameterized(culture: "fr")
❔ Demo.Tests.Skipped_Test_Does_Not_Run => Shouldn't run for now :)

✅ Demo.Tests.Test_With_Output

> Hello, world from xunit ITestOutputHelper!

❔ Demo.Tests.WindowsOnlyTest => Test only runs on Win32NT
🧪 Details on macOS Unix 14.5.0
❌ Demo.Tests.Fails_With_Complex_StackTrace 
   at Demo.Tests.Unexpected() in /_/src/Demo/Tests.cs:line 49
   at Demo.Tests.Run() in /_/src/Demo/Tests.cs:line 47
   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 
   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 
   at Demo.Tests.FailsOnlyOnUnix() in /_/src/Demo/Tests.cs:line 45

✅ Demo.Tests.Parameterized(culture: "de")
✅ Demo.Tests.Parameterized(culture: "en")
✅ Demo.Tests.Parameterized(culture: "es")
✅ Demo.Tests.Parameterized(culture: "fr")
❔ Demo.Tests.Skipped_Test_Does_Not_Run => Shouldn't run for now :)

✅ Demo.Tests.Test_With_Output

> Hello, world from xunit ITestOutputHelper!

❔ Demo.Tests.WindowsOnlyTest => Test only runs on Win32NT
🧪 Details on Microsoft Windows 10.0.20348
❌ Demo.Tests.Fails_With_Complex_StackTrace 
   at Demo.Tests.Unexpected() in /_/src/Demo/Tests.cs:line 49
   at Demo.Tests.Run() in /_/src/Demo/Tests.cs:line 47
   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 
   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 => Test only runs on Unix
✅ Demo.Tests.Parameterized(culture: "de")
✅ Demo.Tests.Parameterized(culture: "en")
✅ Demo.Tests.Parameterized(culture: "es")
✅ Demo.Tests.Parameterized(culture: "fr")
❔ Demo.Tests.Skipped_Test_Does_Not_Run => Shouldn't run for now :)

✅ Demo.Tests.Test_With_Output

> Hello, world from xunit ITestOutputHelper!

✅ Demo.Tests.WindowsOnlyTest

> This test runs only on Windows

from dotnet-trx on .NET 8.0.7 with 💜

@kzu kzu merged commit ea8010f into main Jul 29, 2024
@kzu kzu deleted the dev/outputescape branch July 29, 2024 15:02
@devlooped devlooped locked and limited conversation to collaborators Feb 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kzu @devlooped-bot