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

[ci] Report when dotnet-test-slicer fails #8259

Merged
merged 1 commit into from
Aug 10, 2023

Conversation

pjcollins
Copy link
Member

Changes to the dotnet-test-slicer task from commit a427688 may have
caused it to fail silently if any issues occur. Updating the task to
fail if anything is written to stderr should fix this.

Changes to the `dotnet-test-slicer` task from commit a427688 may have
caused it to fail silently if any issues occur.  Updating the task to
fail if anything is written to stderr should fix this.
@pjcollins pjcollins marked this pull request as ready for review August 9, 2023 23:30
@pjcollins pjcollins requested a review from jpobst August 9, 2023 23:31
Copy link
Contributor

@jpobst jpobst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need separate logic for if it has a filter or not. If the filter is omitted or empty, it should be ignored:

https://github.com/jpobst/dotnet-test-slicer/blob/main/src/DotNet.Test.Slicer/NUnitTestDiscoverer.cs#L20-L21

@pjcollins
Copy link
Member Author

Unfortunately if the param is passed but empty the the argument parser library reports an error and exits before it can get there:

Required argument missing for option: '--test-filter'.

@pjcollins pjcollins merged commit 4249da3 into dotnet:main Aug 10, 2023
44 of 49 checks passed
@pjcollins pjcollins deleted the error-on-slice branch August 10, 2023 15:01
@jpobst
Copy link
Contributor

jpobst commented Aug 10, 2023

Huh, guess that's something System.CommandLine is doing. Thanks!

grendello added a commit to grendello/xamarin-android that referenced this pull request Aug 10, 2023
* main:
  [Mono.Android.dll] Fix [SupportedOSPlatform] warnings. (dotnet#8247)
  Bump to xamarin/monodroid@a5248591 (dotnet#8258)
  [Mono.Android] Bind `java.time` and `java.time.chrono` packages. (dotnet#8088)
  Bump to xamarin/Java.Interop/main@5adb4d4 (dotnet#8251)
  [ci] Use fewer agents for macOS MSBuild tests (dotnet#8257)
  [ci] Report when dotnet-test-slicer fails (dotnet#8259)
  [monodroid] Do not build host or classic runtimes (dotnet#8256)
@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants