Skip to content
Permalink
Browse files

test: make flaky pummel test more reliable

Increase the number of iterations from 1e4 to 1e5.  Makes the test pass
for me locally when previously it would fail 9 out of 10 times because
the running time was not enough to smooth away the outliers.

Fixes: #8744
PR-URL: #9241
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: not-an-aardvark <not-an-aardvark@users.noreply.github.com>
  • Loading branch information...
bnoordhuis authored and evanlucas committed Oct 22, 2016
1 parent 4be1ba5 commit 204461925bda6466e79ac25f4bb0f9b76d06fa56
Showing with 1 addition and 1 deletion.
  1. +1 −1 test/pummel/test-crypto-timing-safe-equal-benchmarks.js
@@ -28,7 +28,7 @@ function runOneBenchmark(...args) {
}

function getTValue(compareFunc) {
const numTrials = 10000;
const numTrials = 1e5;
const bufSize = 10000;
// Perform benchmarks to verify that timingSafeEqual is actually timing-safe.

0 comments on commit 2044619

Please sign in to comment.
You can’t perform that action at this time.