Skip to content

Commit cc2a346

Browse files
committed
fixup! module: exclude node:ffi from builtinModules when flag is disabled
1 parent 999efeb commit cc2a346

1 file changed

Lines changed: 11 additions & 11 deletions

File tree

test/ffi/test-ffi-module.js

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
// Flags: --experimental-ffi
22
'use strict';
33
const common = require('../common');
4+
const { spawnSyncAndAssert } = require('../common/child_process');
45
const assert = require('node:assert');
56
const { spawnSync } = require('node:child_process');
67
const { test } = require('node:test');
@@ -32,17 +33,16 @@ test('ffi builtin is unavailable when disabled', () => {
3233
});
3334

3435
test('ffi builtin is listed', () => {
35-
const { stdout, stderr, status, signal } = spawnSync(process.execPath, [
36-
'-p',
37-
'require("node:module").builtinModules.includes("node:ffi")',
38-
], {
39-
encoding: 'utf8',
40-
});
41-
42-
assert.strictEqual(stdout.trim(), 'true');
43-
assert.strictEqual(stderr, '');
44-
assert.strictEqual(status, 0);
45-
assert.strictEqual(signal, null);
36+
for (const [flag, stdout] of Object.entries({
37+
'--experimental-ffi': 'true\n',
38+
'--no-experimental-ffi': 'false\n',
39+
})) {
40+
spawnSyncAndAssert(process.execPath, [
41+
flag,
42+
'-p',
43+
'require("node:module").builtinModules.includes("node:ffi")',
44+
], { stdout });
45+
}
4646
});
4747

4848
test('ffi can be imported from ESM', () => {

0 commit comments

Comments
 (0)