-
-
Notifications
You must be signed in to change notification settings - Fork 147
Add report_url output for test report URLs #1456
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
Conversation
Co-authored-by: mikepenz <1476232+mikepenz@users.noreply.github.com>
Co-authored-by: mikepenz <1476232+mikepenz@users.noreply.github.com>
|
||||||||||||||||||||||
|
|
||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mikepenz thanks for the quick turnaround on this as usual! It looks like this was a very simple update, but the tests that Copilot are a bit... lacking. 😁
| } | ||
| ] | ||
|
|
||
| const reportUrls = checkInfos.map(info => info.url).join('\n') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like this is just testing that Array.join works?
| const reportUrls = checkInfos.map(info => info.url).join('\n') | ||
| expect(reportUrls).toBe( | ||
| 'https://github.com/owner/repo/runs/123\nhttps://github.com/owner/repo/runs/456\nhttps://github.com/owner/repo/runs/789' | ||
| ) | ||
| }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again, this isn't testing the action; just the JavaScript stdlib.
| const reportUrls = checkInfos.map(info => info.url).join('\n') | ||
| expect(reportUrls).toBe('') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.
Enables workflows to programmatically access test report URLs for linking in PR comments or cross-workflow coordination.
Changes
report_url: Contains newline-separated URLs of all created test reportsCheckInfoobjects populated during check creationUsage
For multiple reports, URLs are newline-separated and can be processed with standard shell tools.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.