test: fix flaky test-perf-hooks-histogram

santigimeno · guangwong · commit 579fae8d572c · 2022-10-10T11:48:55.000+08:00
Make sure the histogram containing event loop delay info is posted only when at least one datapoint is collected. Fixes: nodejs/node#43503 PR-URL: nodejs/node#43567 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/test/parallel/test-perf-hooks-histogram.js b/test/parallel/test-perf-hooks-histogram.js
@@ -89,7 +89,12 @@ const { inspect } = require('util');
     strictEqual(data.enable, undefined);
     mc.port1.close();
   });
-  setTimeout(() => mc.port2.postMessage(e), 100);
+  const interval = setInterval(() => {
+    if (e.count > 0) {
+      clearInterval(interval);
+      mc.port2.postMessage(e);
+    }
+  }, 50);
 }
 
 {

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,12 @@ const { inspect } = require('util');`
`89`	`89`	`strictEqual(data.enable, undefined);`
`90`	`90`	`mc.port1.close();`
`91`	`91`	`});`
`92`		`- setTimeout(() => mc.port2.postMessage(e), 100);`
	`92`	`+ const interval = setInterval(() => {`
	`93`	`+ if (e.count > 0) {`
	`94`	`+ clearInterval(interval);`
	`95`	`+ mc.port2.postMessage(e);`
	`96`	`+ }`
	`97`	`+ }, 50);`
`93`	`98`	`}`
`94`	`99`
`95`	`100`	`{`