Skip to content

chore: re-enable worker spec failures#37015

Merged
codebytere merged 1 commit intomainfrom
enable-test-worker-init-failure
Feb 9, 2023
Merged

chore: re-enable worker spec failures#37015
codebytere merged 1 commit intomainfrom
enable-test-worker-init-failure

Conversation

@codebytere
Copy link
Member

@codebytere codebytere commented Jan 25, 2023

Description of Change

Refs nodejs/node-v8#246.
Refs V8BUG:13493.

Upstream WASM stack switching issues appears to have been fixed, and no longer crashes locally. Re-enable tests accordingly.

Checklist

Release Notes

Notes: none

@codebytere codebytere added semver/patch backwards-compatible bug fixes no-backport labels Jan 25, 2023
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jan 25, 2023
@codebytere codebytere marked this pull request as draft January 25, 2023 10:43
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jan 26, 2023
@codebytere codebytere force-pushed the enable-test-worker-init-failure branch 3 times, most recently from e4eb113 to e98c95e Compare February 8, 2023 11:17
@codebytere codebytere marked this pull request as ready for review February 8, 2023 11:48
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Feb 8, 2023
@codebytere codebytere force-pushed the enable-test-worker-init-failure branch from e98c95e to 43c1718 Compare February 9, 2023 09:04
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Feb 9, 2023
@codebytere codebytere merged commit 12ed401 into main Feb 9, 2023
@codebytere codebytere deleted the enable-test-worker-init-failure branch February 9, 2023 17:18
@release-clerk
Copy link

release-clerk bot commented Feb 9, 2023

No Release Notes

trop bot added a commit that referenced this pull request Feb 13, 2023
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
trop bot added a commit that referenced this pull request Feb 13, 2023
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
codebytere added a commit that referenced this pull request Feb 15, 2023
* fix: cookies filter secure and session invalid

Co-authored-by: Black-Hole1 <158blackhole@gmail.com>

* test: add test

Co-authored-by: Black-Hole1 <158blackhole@gmail.com>

* test: fix test failed

Co-authored-by: Black-Hole1 <158blackhole@gmail.com>

* test: fix test failed again

Co-authored-by: Black-Hole1 <158blackhole@gmail.com>

* fix: session check logic incorrect

Co-authored-by: Black-Hole1 <158blackhole@gmail.com>

* refactor: reset cookies filter session logic

Co-authored-by: Black-Hole1 <158blackhole@gmail.com>

* Update shell/browser/api/electron_api_cookies.cc

Co-authored-by: Robo <hop2deep@gmail.com>

Co-authored-by: Black-Hole <158blackhole@gmail.com>

* chore: re-enable worker spec failures (#37015)

Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

---------

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: Black-Hole1 <158blackhole@gmail.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
gecko19 pushed a commit to brightsign/electron that referenced this pull request Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-backport semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants