test: fix flaky test-zlib-unused-weak.js #38149
Closed
Conversation
|
Fast-track? |
|
+1 to fast track |
|
lgtm |
oyyd
added a commit
that referenced
this pull request
Apr 9, 2021
PR-URL: #38149 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
|
Landed in f96dffb, thanks! |
targos
added a commit
that referenced
this pull request
May 1, 2021
PR-URL: #38149 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Merged
danielleadams
added a commit
that referenced
this pull request
May 8, 2021
PR-URL: #38149 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Failed test: https://ci.nodejs.org/job/node-test-commit-linux/nodes=alpine-last-latest-x64/40825/testReport/junit/(root)/test/parallel_test_zlib_unused_weak/
I think the delta should be positive. Calling
gc()beforezlib.createGzip()should make it more stable.