Skip to content

Commit a3ee187

Browse files
joyeecheungnodejs-github-bot
joyeecheung
authored and
nodejs-github-bot
committed
test: log more information in SEA tests
- Use spawnSyncAndExitWithoutError to log more information on error. - Use NODE_DEBUG_NATIVE to log internals - Skip the test when available disk space < 120MB PR-URL: #50759 Refs: #50740 Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
1 parent 973f30b commit a3ee187

7 files changed

+106
-35
lines changed

‎test/common/sea.js

+9
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
const common = require('../common');
44
const fixtures = require('../common/fixtures');
5+
const tmpdir = require('../common/tmpdir');
56

67
const { readFileSync } = require('fs');
78
const {
@@ -43,6 +44,14 @@ function skipIfSingleExecutableIsNotSupported() {
4344
common.skip('On s390x, postject fails with `memory access out of bounds`.');
4445
}
4546
}
47+
48+
tmpdir.refresh();
49+
50+
// The SEA tests involve making a copy of the executable and writing some fixtures
51+
// to the tmpdir. To be safe, ensure that at least 120MB disk space is available.
52+
if (!tmpdir.hasEnoughSpace(120 * 1024 * 1024)) {
53+
common.skip('Available disk space < 120MB');
54+
}
4655
}
4756

4857
function injectAndCodeSign(targetExecutable, resource) {

‎test/sequential/test-single-executable-application-disable-experimental-sea-warning.js

+17-8
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,8 @@ skipIfSingleExecutableIsNotSupported();
1515
const fixtures = require('../common/fixtures');
1616
const tmpdir = require('../common/tmpdir');
1717
const { copyFileSync, writeFileSync, existsSync } = require('fs');
18-
const { execFileSync } = require('child_process');
18+
const { spawnSyncAndExitWithoutError } = require('../common/child_process');
1919
const { join } = require('path');
20-
const { strictEqual } = require('assert');
2120
const assert = require('assert');
2221

2322
const inputFile = fixtures.path('sea.js');
@@ -44,17 +43,27 @@ writeFileSync(configFile, `
4443

4544
// Copy input to working directory
4645
copyFileSync(inputFile, tmpdir.resolve('sea.js'));
47-
execFileSync(process.execPath, ['--experimental-sea-config', 'sea-config.json'], {
48-
cwd: tmpdir.path
49-
});
46+
spawnSyncAndExitWithoutError(
47+
process.execPath,
48+
['--experimental-sea-config', 'sea-config.json'],
49+
{ cwd: tmpdir.path },
50+
{});
5051

5152
assert(existsSync(seaPrepBlob));
5253

5354
copyFileSync(process.execPath, outputFile);
5455
injectAndCodeSign(outputFile, seaPrepBlob);
5556

56-
const singleExecutableApplicationOutput = execFileSync(
57+
spawnSyncAndExitWithoutError(
5758
outputFile,
5859
[ '-a', '--b=c', 'd' ],
59-
{ env: { COMMON_DIRECTORY: join(__dirname, '..', 'common') } });
60-
strictEqual(singleExecutableApplicationOutput.toString(), 'Hello, world! 😊\n');
60+
{
61+
env: {
62+
COMMON_DIRECTORY: join(__dirname, '..', 'common'),
63+
NODE_DEBUG_NATIVE: 'SEA',
64+
...process.env,
65+
}
66+
},
67+
{
68+
stdout: 'Hello, world! 😊\n'
69+
});

‎test/sequential/test-single-executable-application-empty.js

+14-5
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ skipIfSingleExecutableIsNotSupported();
1414

1515
const tmpdir = require('../common/tmpdir');
1616
const { copyFileSync, writeFileSync, existsSync } = require('fs');
17-
const { execFileSync } = require('child_process');
17+
const { spawnSyncAndExitWithoutError } = require('../common/child_process');
1818
const assert = require('assert');
1919

2020
const configFile = tmpdir.resolve('sea-config.json');
@@ -31,13 +31,22 @@ writeFileSync(configFile, `
3131
}
3232
`);
3333

34-
execFileSync(process.execPath, ['--experimental-sea-config', 'sea-config.json'], {
35-
cwd: tmpdir.path
36-
});
34+
spawnSyncAndExitWithoutError(
35+
process.execPath,
36+
['--experimental-sea-config', 'sea-config.json'],
37+
{ cwd: tmpdir.path });
3738

3839
assert(existsSync(seaPrepBlob));
3940

4041
copyFileSync(process.execPath, outputFile);
4142
injectAndCodeSign(outputFile, seaPrepBlob);
4243

43-
execFileSync(outputFile);
44+
spawnSyncAndExitWithoutError(
45+
outputFile,
46+
{
47+
env: {
48+
NODE_DEBUG_NATIVE: 'SEA',
49+
...process.env,
50+
}
51+
},
52+
{});

‎test/sequential/test-single-executable-application-snapshot-and-code-cache.js

+16-5
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,11 @@ const outputFile = join(tmpdir.path, process.platform === 'win32' ? 'sea.exe' :
4949
process.execPath,
5050
['--experimental-sea-config', 'sea-config.json'],
5151
{
52-
cwd: tmpdir.path
52+
cwd: tmpdir.path,
53+
env: {
54+
NODE_DEBUG_NATIVE: 'SEA',
55+
...process.env,
56+
},
5357
},
5458
{
5559
stderr: /"useCodeCache" is redundant when "useSnapshot" is true/
@@ -61,8 +65,15 @@ const outputFile = join(tmpdir.path, process.platform === 'win32' ? 'sea.exe' :
6165
copyFileSync(process.execPath, outputFile);
6266
injectAndCodeSign(outputFile, seaPrepBlob);
6367

64-
spawnSyncAndExitWithoutError(outputFile, {
65-
stdout: 'Hello from snapshot',
66-
trim: true,
67-
});
68+
spawnSyncAndExitWithoutError(
69+
outputFile,
70+
{
71+
env: {
72+
NODE_DEBUG_NATIVE: 'SEA,MKSNAPSHOT',
73+
...process.env,
74+
}
75+
}, {
76+
stdout: 'Hello from snapshot',
77+
trim: true,
78+
});
6879
}

‎test/sequential/test-single-executable-application-snapshot.js

+11-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,11 @@ const outputFile = tmpdir.resolve(process.platform === 'win32' ? 'sea.exe' : 'se
7373
process.execPath,
7474
['--experimental-sea-config', 'sea-config.json'],
7575
{
76-
cwd: tmpdir.path
76+
cwd: tmpdir.path,
77+
env: {
78+
NODE_DEBUG_NATIVE: 'SEA',
79+
...process.env,
80+
},
7781
},
7882
{
7983
stderr: /Single executable application is an experimental feature/
@@ -86,6 +90,12 @@ const outputFile = tmpdir.resolve(process.platform === 'win32' ? 'sea.exe' : 'se
8690

8791
spawnSyncAndExitWithoutError(
8892
outputFile,
93+
{
94+
env: {
95+
NODE_DEBUG_NATIVE: 'SEA,MKSNAPSHOT',
96+
...process.env,
97+
}
98+
},
8999
{
90100
trim: true,
91101
stdout: 'Hello from snapshot',

‎test/sequential/test-single-executable-application-use-code-cache.js

+22-8
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,8 @@ skipIfSingleExecutableIsNotSupported();
1515
const fixtures = require('../common/fixtures');
1616
const tmpdir = require('../common/tmpdir');
1717
const { copyFileSync, writeFileSync, existsSync } = require('fs');
18-
const { execFileSync } = require('child_process');
18+
const { spawnSyncAndExitWithoutError } = require('../common/child_process');
1919
const { join } = require('path');
20-
const { strictEqual } = require('assert');
2120
const assert = require('assert');
2221

2322
const inputFile = fixtures.path('sea.js');
@@ -44,17 +43,32 @@ writeFileSync(configFile, `
4443

4544
// Copy input to working directory
4645
copyFileSync(inputFile, tmpdir.resolve('sea.js'));
47-
execFileSync(process.execPath, ['--experimental-sea-config', 'sea-config.json'], {
48-
cwd: tmpdir.path
49-
});
46+
spawnSyncAndExitWithoutError(
47+
process.execPath,
48+
['--experimental-sea-config', 'sea-config.json'],
49+
{
50+
cwd: tmpdir.path,
51+
env: {
52+
NODE_DEBUG_NATIVE: 'SEA',
53+
...process.env,
54+
},
55+
});
5056

5157
assert(existsSync(seaPrepBlob));
5258

5359
copyFileSync(process.execPath, outputFile);
5460
injectAndCodeSign(outputFile, seaPrepBlob);
5561

56-
const singleExecutableApplicationOutput = execFileSync(
62+
spawnSyncAndExitWithoutError(
5763
outputFile,
5864
[ '-a', '--b=c', 'd' ],
59-
{ env: { COMMON_DIRECTORY: join(__dirname, '..', 'common') } });
60-
strictEqual(singleExecutableApplicationOutput.toString(), 'Hello, world! 😊\n');
65+
{
66+
env: {
67+
COMMON_DIRECTORY: join(__dirname, '..', 'common'),
68+
NODE_DEBUG_NATIVE: 'SEA',
69+
...process.env,
70+
}
71+
},
72+
{
73+
stdout: 'Hello, world! 😊\n'
74+
});

‎test/sequential/test-single-executable-application.js

+17-8
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,8 @@ skipIfSingleExecutableIsNotSupported();
1414
const fixtures = require('../common/fixtures');
1515
const tmpdir = require('../common/tmpdir');
1616
const { copyFileSync, writeFileSync, existsSync } = require('fs');
17-
const { execFileSync } = require('child_process');
17+
const { spawnSyncAndExitWithoutError } = require('../common/child_process');
1818
const { join } = require('path');
19-
const { strictEqual } = require('assert');
2019
const assert = require('assert');
2120

2221
const inputFile = fixtures.path('sea.js');
@@ -43,17 +42,27 @@ writeFileSync(configFile, `
4342

4443
// Copy input to working directory
4544
copyFileSync(inputFile, tmpdir.resolve('sea.js'));
46-
execFileSync(process.execPath, ['--experimental-sea-config', 'sea-config.json'], {
47-
cwd: tmpdir.path
48-
});
45+
spawnSyncAndExitWithoutError(
46+
process.execPath,
47+
['--experimental-sea-config', 'sea-config.json'],
48+
{ cwd: tmpdir.path },
49+
{});
4950

5051
assert(existsSync(seaPrepBlob));
5152

5253
copyFileSync(process.execPath, outputFile);
5354
injectAndCodeSign(outputFile, seaPrepBlob);
5455

55-
const singleExecutableApplicationOutput = execFileSync(
56+
spawnSyncAndExitWithoutError(
5657
outputFile,
5758
[ '-a', '--b=c', 'd' ],
58-
{ env: { COMMON_DIRECTORY: join(__dirname, '..', 'common') } });
59-
strictEqual(singleExecutableApplicationOutput.toString(), 'Hello, world! 😊\n');
59+
{
60+
env: {
61+
COMMON_DIRECTORY: join(__dirname, '..', 'common'),
62+
NODE_DEBUG_NATIVE: 'SEA',
63+
...process.env,
64+
}
65+
},
66+
{
67+
stdout: 'Hello, world! 😊\n'
68+
});

0 commit comments

Comments
 (0)