language-servers/deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2025-10-02 07:01:16 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

chore(cli): measure performance against wall-clock (#18357)

Browse source 
Fixes test flake by requiring perf to be <150% of wall-clock time rather
than <500ms.
This commit is contained in:
Matt Mastracci 2023-03-22 12:06:29 -06:00 • committed by GitHub
parent 79fb3b1f35
commit 03e236b614
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
cli/tests/unit/performance_test.ts
View file

@ -81,9 +81,12 @@ Deno.test(function performanceMeasure() {
const measureName1 = "measure1"; const measureName1 = "measure1";
const measureName2 = "measure2"; const measureName2 = "measure2";
const mark1 = performance.mark(markName1); const mark1 = performance.mark(markName1);
// Measure against the inaccurate-but-known-good wall clock
const now = new Date().valueOf();
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
setTimeout(() => { setTimeout(() => {
try { try {
const later = new Date().valueOf();
const measure1 = performance.measure(measureName1, markName1); const measure1 = performance.measure(measureName1, markName1);
const measure2 = performance.measure( const measure2 = performance.measure(
measureName2, measureName2,
@ -103,8 +106,10 @@ Deno.test(function performanceMeasure() {
`duration below 100ms: ${measure1.duration}`, `duration below 100ms: ${measure1.duration}`,
); );
assert( assert(
measure1.duration < 500, measure1.duration < (later - now) * 1.50,
`duration exceeds 500ms: ${measure1.duration}`, `duration exceeds 150% of wallclock time: ${measure1.duration}ms vs ${
later - now
}ms`,
); );
const entries = performance.getEntries(); const entries = performance.getEntries();
assert(entries[entries.length - 1] === measure2); assert(entries[entries.length - 1] === measure2);