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

TASK: Show testcafe output in circle ci overview #3911

Open
wants to merge 2 commits into
base: 8.3
Choose a base branch
from

Conversation

mhsdesign
Copy link
Member

@mhsdesign mhsdesign commented Jan 25, 2025

Apparently none of the options for saucectl run

--show-console-log
--verbose
--live-logs

do output the actual live console logs from testcafe to the circle ci shell.

--show-console-log seems to output them at the end but only for happy cases.
This pr introduces a separate window where just the output is cleanly shown - hopefully also on error.

image

Further the artifact upload was fixed as the path was wrong.

What I did

How I did it

How to verify it

@mhsdesign mhsdesign force-pushed the task/improve-ci-debug-information branch from 0e6e94f to 5a4a019 Compare January 25, 2025 11:25
@mhsdesign mhsdesign force-pushed the task/improve-ci-debug-information branch from 5a4a019 to 20ac678 Compare January 25, 2025 11:34
@mhsdesign mhsdesign changed the title TASK: Improve CI debug information TASK: Show testcafe output in circle ci overview Jan 25, 2025
@markusguenther
Copy link
Member

So, should we also apply this to older branches? Thanks for digging deeper into this!

Apparently neither of the options for `saucectl run`

--show-console-log
--verbose
--live-logs

change anything regarding streaming the testcafe log output directly to the circle ci shell.

To make debugging easier we read the console.logs
@mhsdesign mhsdesign force-pushed the task/improve-ci-debug-information branch from 689a581 to 38b4708 Compare February 5, 2025 09:17
@mhsdesign mhsdesign changed the base branch from 9.0 to 8.3 February 5, 2025 09:18
@github-actions github-actions bot added 8.3 and removed 9.0 labels Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants