Skip to content

Improve path validation and reporting markup escaping#66

Merged
kzu merged 2 commits into
mainfrom
dev/validate
Feb 18, 2025
Merged

Improve path validation and reporting markup escaping#66
kzu merged 2 commits into
mainfrom
dev/validate

Conversation

@kzu

@kzu kzu commented Feb 18, 2025

Copy link
Copy Markdown
Member

No description provided.

@kzu kzu added the enhancement New feature or request label Feb 18, 2025
@kzu kzu enabled auto-merge (rebase) February 18, 2025 10:32
@kzu

kzu commented Feb 18, 2025 •
edited
Loading

Copy link
Copy Markdown
Member Author

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 macOS Unix 14.7.2
❌ 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 Ubuntu 24.04.1 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 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 v42.42.275-pr66 on .NET 8.0.12 with 💜

@kzu kzu merged commit fed330a into main Feb 18, 2025
@kzu kzu deleted the dev/validate branch February 18, 2025 10:35
@kzu kzu mentioned this pull request Feb 18, 2025
Closed
@devlooped devlooped locked and limited conversation to collaborators Mar 21, 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

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kzu