Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Fix logging in end-to-end tests #8028

Merged
merged 2 commits into from
Mar 11, 2022
Merged

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Mar 11, 2022

Fixes element-hq/element-web#21386

There were two issues:

  • Lack of null checking on the object to be logged
  • Somewhere in a puppeteer update requestfinished became a bit of a non-fatal race

This change is marked as an internal change (Task), so will not be included in the changelog.

Preview: https://pr8028--matrix-react-sdk.netlify.app
⚠️ Do you trust the author of this PR? Maybe this build will steal your keys or give you malware. Exercise caution. Use test accounts.

@turt2live turt2live added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Mar 11, 2022
@codecov
Copy link

codecov bot commented Mar 11, 2022

Codecov Report

Merging #8028 (3fc7368) into develop (9082e07) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #8028   +/-   ##
========================================
  Coverage    26.66%   26.66%           
========================================
  Files          862      862           
  Lines        51954    51954           
  Branches     13159    13159           
========================================
  Hits         13854    13854           
  Misses       38100    38100           

Apparently puppeteer can race on this
@turt2live turt2live force-pushed the travis/fix-e2e-test-logging branch from 56bc819 to 3fc7368 Compare March 11, 2022 04:19
@turt2live turt2live marked this pull request as ready for review March 11, 2022 04:21
@turt2live turt2live requested a review from a team as a code owner March 11, 2022 04:21
@turt2live turt2live merged commit 4a36f9b into develop Mar 11, 2022
@turt2live turt2live deleted the travis/fix-e2e-test-logging branch March 11, 2022 17:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

end-to-end tests broken trying to read stack of undefined
2 participants