Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for reporting to github #11

Merged
merged 3 commits into from
Jul 4, 2024
Merged

Add support for reporting to github #11

merged 3 commits into from
Jul 4, 2024

Conversation

kzu
Copy link
Member

@kzu kzu commented Jul 4, 2024

Two features here:

  • Report summary and details as a comment on the associated PR (if any)
  • Issue :error: notices with the paths of the failed paths and the error message.

@kzu kzu added the enhancement New feature or request label Jul 4, 2024
@kzu kzu force-pushed the dev/github branch 5 times, most recently from fb24d00 to 4f6ec21 Compare July 4, 2024 04:54
Two features here:
- Report summary and details as a comment on the associated PR (if any)
- Issue :error: notices with the paths of the failed paths and the error message.
We need to normalize to the fully qualified directory in order for relative paths to work properly.
That would be to run on a step always even if tests fail:

```
      - name: 🧪 show
        if: always()
        run:
          dotnet run --no-build --project ./src/dotnet-trx/ --output
```
@devlooped-bot
Copy link
Member

👉 Run 8 tests in ~ 3 seconds
     ✅ 5 passed
     ❌ 2 failed
     ❔ 1 skipped

🧪 Details
❌ Demo.Tests.Fails_With_Complex_StackTrace
   at Demo.Tests.Unexpected() in /_/src/Demo/Tests.cs:line 43
   at Demo.Tests.Run() in /_/src/Demo/Tests.cs:line 41
   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.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!

from dotnet-trx with 💜

@devlooped devlooped deleted a comment from devlooped-bot Jul 4, 2024
@kzu kzu merged commit c166255 into main Jul 4, 2024
7 checks passed
@kzu kzu deleted the dev/github branch July 4, 2024 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants