Skip to content

[cypress] Trim long cypress configs#167391

Merged
delanni merged 1 commit intoelastic:mainfrom
delanni:trim-long-cypress-logs
Sep 27, 2023
Merged

[cypress] Trim long cypress configs#167391
delanni merged 1 commit intoelastic:mainfrom
delanni:trim-long-cypress-logs

Conversation

@delanni
Copy link
Copy Markdown
Member

@delanni delanni commented Sep 27, 2023

Summary

In cypress tests, when the logs are output in the beginning, some binary buffers are also stringified, so when looking at logs, we have to scroll through thousands of lines of logs, that's annoying and doesn't help debugging.

I'd like that to be somewhat more readable, there are a few options:

  • the replacer should interpret buffers as strings, try to output it like that
    • not every binary buffer is printable
  • the replacer should redact the exact fields (certificateAuthority?)
    • too specific, might need fixing later
  • trim long arrays after 32 items
    • I chose this as a solution

Other suggestions are welcome.

Screenshot 2023-09-27 at 12 49 54

@delanni delanni added chore release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Sep 27, 2023
@delanni delanni changed the title [Ops] Trim long cypress configs [cypress] Trim long cypress configs Sep 27, 2023
@kibana-ci
Copy link
Copy Markdown

kibana-ci commented Sep 27, 2023

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #35 / serverless search UI landing page has serverless side nav
  • [job] [logs] FTR Configs #35 / serverless security UI Cases List "before all" hook in "Cases List"

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@delanni delanni merged commit 28fd3ff into elastic:main Sep 27, 2023
@delanni delanni deleted the trim-long-cypress-logs branch September 27, 2023 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting chore release_note:skip Skip the PR/issue when compiling release notes v8.11.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants