Repository navigation
CI: run the url-to-license-mapping tests only once - #641
Merged
Merged
Conversation
test_windows and the release job both discovered test executables with a NuGet*.Test glob. NuGetUtility.UrlToLicenseMapping.Test matches it, so the Selenium suite ran again there on top of its own test_url_to_license_mapping job. Name the two projects each job is meant to cover instead, and drop the now-unused Chrome setup from the release job. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Contributor
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This was referenced Sep 11, 2026
This was referenced Sep 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
test_windowsand the release job both discovered test executables with aNuGet*.Testglob.NuGetUtility.UrlToLicenseMapping.Testmatches it, so the Selenium suite ran a second time on top of its owntest_url_to_license_mappingjob (visible in #636).Both jobs now name the two projects they are meant to cover. The release job's
browser-actions/setup-chromestep is removed with it, since Chrome was only needed by the url-mapping tests.Two consequences:
NuGet*.Testproject no longer joins those jobs on its own. It has to be added to the list.test_windowsinstead of being silently skipped, which the glob hid.🤖 Generated with Claude Code