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.

Sign up for GitHub

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

Jump to bottom

[Bug]: test reports are missing on the CI when using github actions/upload-artifact@4 #1198

Closed
fpaul-1A opened this issue Jan 9, 2024 · 3 comments · Fixed by #2161
Closed

[Bug]: test reports are missing on the CI when using github actions/upload-artifact@4 #1198

fpaul-1A opened this issue Jan 9, 2024 · 3 comments · Fixed by #2161
Labels
bug Something isn't working triage

Comments

@fpaul-1A
Copy link
Contributor

fpaul-1A commented Jan 9, 2024 •
edited
Loading

Package name

CI

Package version

10

Reproduction steps

Run the workflow for any PR on release/10.0.0-next

Current result

The tests are executed and the xml reports are uploaded, but the html report are not added to the workflow run

Expected result

As currently working on main, the html reports should be added to the workflow run

Additional comments

No response
@fpaul-1A fpaul-1A added the bug Something isn't working label Jan 9, 2024
@fpaul-1A
Copy link
Contributor Author

fpaul-1A commented Jan 9, 2024

A similar issue has been raised on the test-reporter github dorny/test-reporter#343

@fpaul-1A fpaul-1A mentioned this issue Jan 17, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Jan 17, 2024
@fpaul-1A
ci: downgrade upload-artifact to v3 (#1239)
153dd10 
## Proposed change

dorny/test-reporter doesn't support artifact v4
#1198

## Related issues

- 🐛 Fixes #(issue)
- 🚀 Feature #(issue)

<!-- Please make sure to follow the contributing guidelines on
https://github.com/amadeus-digital/Otter/blob/main/CONTRIBUTING.md -->
@pginoux-1A
Copy link
Contributor

Still waiting for a fix.

@vscaiceanu-1a
Copy link
Member

dorny/test-reporter#363 we can keep an eye on this issue too.

@fpaul-1A fpaul-1A linked a pull request Sep 12, 2024 that will close this issue
chore: update upload-artifact #2161
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: update upload-artifact AmadeusITGroup/otter
4 participants
@matthieu-crouzet @vscaiceanu-1a @fpaul-1A @pginoux-1A