Skip to content

Commit 4d67a36

Browse files
committed
histogram: refactor to use min/max of validateNumber
Fix NumberIsNaN is called two times and change error code to ERR_OUT_OF_RANGE when RangeError is occurred in test.
1 parent 3bef549 commit 4d67a36

File tree

2 files changed

+3
-9
lines changed

2 files changed

+3
-9
lines changed

lib/internal/histogram.js

+2-8
Original file line numberDiff line numberDiff line change
@@ -186,10 +186,7 @@ class Histogram {
186186
percentile(percentile) {
187187
if (!isHistogram(this))
188188
throw new ERR_INVALID_THIS('Histogram');
189-
validateNumber(percentile, 'percentile');
190-
191-
if (NumberIsNaN(percentile) || percentile <= 0 || percentile > 100)
192-
throw new ERR_INVALID_ARG_VALUE.RangeError('percentile', percentile);
189+
validateNumber(percentile, 'percentile', 1, 100);
193190

194191
return this[kHandle]?.percentile(percentile);
195192
}
@@ -201,10 +198,7 @@ class Histogram {
201198
percentileBigInt(percentile) {
202199
if (!isHistogram(this))
203200
throw new ERR_INVALID_THIS('Histogram');
204-
validateNumber(percentile, 'percentile');
205-
206-
if (NumberIsNaN(percentile) || percentile <= 0 || percentile > 100)
207-
throw new ERR_INVALID_ARG_VALUE.RangeError('percentile', percentile);
201+
validateNumber(percentile, 'percentile', 1, 100);
208202

209203
return this[kHandle]?.percentileBigInt(percentile);
210204
}

test/sequential/test-performance-eventloopdelay.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ const { sleep } = require('internal/util');
9191
() => histogram.percentile(i),
9292
{
9393
name: 'RangeError',
94-
code: 'ERR_INVALID_ARG_VALUE'
94+
code: 'ERR_OUT_OF_RANGE'
9595
}
9696
);
9797
});

0 commit comments

Comments
 (0)