test: tmpdir include build id

test/common/tmpdir.js

@@ -14,6 +14,7 @@ const testRoot = process.env.NODE_TEST_DIR ?
 // Using a `.` prefixed name, which is the convention for "hidden" on POSIX,
 // gets tools to ignore it by default or by simple rules, especially eslint.
 const tmpdirName = '.tmp.' +
+  (process.env.BUILD_ID || process.pid + '.') +
   (process.env.TEST_SERIAL_ID || process.env.TEST_THREAD_ID || '0');
 const tmpPath = path.join(testRoot, tmpdirName);
 

test/sequential/test-debugger-heap-profiler.mjs

@@ -0,0 +1,36 @@
+import { skipIfInspectorDisabled } from '../common/index.mjs';
+import { createRequire } from 'module';
+import path from 'path';
+
+const require = createRequire(import.meta.url);
+const fixtures = require('../common/fixtures');
+const startCLI = require('../common/debugger');
+const tmpdir = require('../common/tmpdir');
+
+skipIfInspectorDisabled();
+
+tmpdir.refresh();
+
+const { readFileSync } = require('fs');
+
+const filename = path.join(tmpdir.path, 'node.heapsnapshot');
+
+// Heap profiler take snapshot.
+const opts = { cwd: tmpdir.path };
+const cli = startCLI([fixtures.path('debugger/empty.js')], [], opts);
+
+try {
+  // Check that the snapshot is valid JSON.
+  await cli.waitForInitialBreak();
+  await cli.waitForPrompt();
+  await cli.command('takeHeapSnapshot()');
+  await JSON.parse(readFileSync(filename, 'utf8'));
+  // Check that two simultaneous snapshots don't step all over each other.
+  // Refs: https://github.com/nodejs/node/issues/39555
+  await cli.command('takeHeapSnapshot(); takeHeapSnapshot()');
+  await JSON.parse(readFileSync(filename, 'utf8'));
+  await cli.quit();
+} catch (e) {
+  cli.quit();
+  throw e;
+}