Skip to content

fix(e2e): show command output on error#84

Merged
davidfestal merged 1 commit intoredhat-developer:mainfrom
gashcrumb:show-output-on-error
Mar 17, 2026
Merged

fix(e2e): show command output on error#84
davidfestal merged 1 commit intoredhat-developer:mainfrom
gashcrumb:show-output-on-error

Conversation

@gashcrumb
Copy link
Member

@gashcrumb gashcrumb commented Mar 16, 2026

This change updates the CLI e2e test spec so that it prints the full
output of whatever command fails during execution. This change also
includes some tidying of the e2e test output in general so it's
hopefully easier to follow, for example avoiding jest's console.log
wrapper.

Assisted-by: Cursor Desktop

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

@gashcrumb gashcrumb requested a review from kadel March 16, 2026 17:47
@gashcrumb gashcrumb force-pushed the show-output-on-error branch from cba9c49 to b620490 Compare March 16, 2026 18:00
This change updates the CLI e2e test spec so that it prints the full
output of whatever command fails during execution.  This change also
includes some tidying of the e2e test output in general so it's
hopefully easier to follow, for example avoiding jest's console.log
wrapper.

Assisted-by: Cursor Desktop

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@gashcrumb gashcrumb force-pushed the show-output-on-error branch from b620490 to d972eb1 Compare March 16, 2026 18:03
@sonarqubecloud
Copy link

@gashcrumb
Copy link
Member Author

we'll need to bring this over to the release-1.8 and release-1.9 branch to help debug what's going on with this and this

@gashcrumb gashcrumb requested a review from davidfestal March 17, 2026 11:22
@davidfestal davidfestal merged commit 3c4f9eb into redhat-developer:main Mar 17, 2026
6 checks passed
gashcrumb added a commit to gashcrumb/rhdh-cli that referenced this pull request Mar 18, 2026
This change updates the CLI e2e test spec so that it prints the full
output of whatever command fails during execution.  This change also
includes some tidying of the e2e test output in general so it's
hopefully easier to follow, for example avoiding jest's console.log
wrapper.

Assisted-by: Cursor Desktop

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants