Skip to content

Commit 5889aba

Browse files
authored
test: use more lenient time window for runtime metric time window (#7467)
Not only rounding issues can happen, the overall measured time window might sometimes slightly deviate. The more lenient range should account for that.
1 parent 8e7b6ac commit 5889aba

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

packages/dd-trace/test/runtime_metrics.spec.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -527,8 +527,8 @@ function createGarbage (count = 50) {
527527
}
528528
if (metric === 'runtime.node.cpu.total') {
529529
assert(
530-
// Subtracting 0.02 since lower numbers can happen due to rounding issues.
531-
number >= expected - 0.02 && number <= expected + 1,
530+
// Subtracting 0.1 for time-window/baseline alignment numbers and due to rounding issues.
531+
number >= expected - 0.1 && number <= expected + 1,
532532
`${metric} sanity check failed (increase CPU load above with more ticks): ${number} ${expected}`
533533
)
534534
totalPercent = number

0 commit comments

Comments
 (0)