perf_hooks: add warning when too many entries in the timeline

PR-URL: https://github.com/nodejs/node/pull/18087
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

1 files changed, 29 insertions, 0 deletions

diff --git a/test/parallel/test-performance-warning.js b/test/parallel/test-performance-warning.js
new file mode 100644
index 0000000000..f3104677a7
--- /dev/null
+++ b/test/parallel/test-performance-warning.js
@@ -0,0 +1,29 @@
+// Flags: --no-warnings
+'use strict';
+
+const common = require('../common');
+const { performance } = require('perf_hooks');
+const assert = require('assert');
+
+assert.strictEqual(performance.length, 1);
+assert.strictEqual(performance.maxEntries, 150);
+
+performance.maxEntries = 1;
+
+[-1, 0xffffffff + 1, '', null, undefined, Infinity].forEach((i) => {
+  common.expectsError(
+    () => performance.maxEntries = i,
+    {
+      code: 'ERR_INVALID_ARG_TYPE',
+      type: TypeError
+    }
+  );
+});
+
+common.expectWarning('Warning', [
+  'Possible perf_hooks memory leak detected. There are 2 entries in the ' +
+  'Performance Timeline. Use the clear methods to remove entries that are no ' +
+  'longer needed or set performance.maxEntries equal to a higher value ' +
+  '(currently the maxEntries is 1).']);
+
+performance.mark('test');