Skip to content

lib: reset the cwd cache before execution #49684

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
nodejs-github-bot merged 11 commits into from
Sep 21, 2023
Merged

lib: reset the cwd cache before execution #49684

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
d6d6eab
lib: reset the cwd cache before execution
arcanis Sep 17, 2023
7b2cfac
test: add a cwd cache bust test
arcanis Sep 17, 2023
f02a6ab
fix lint
arcanis Sep 17, 2023
851b29a
leverage addSerializeCallback
arcanis Sep 18, 2023
a74d68d
leverage spawnSyncAndExitWithoutError
arcanis Sep 18, 2023
3736efe
use trim
arcanis Sep 18, 2023
9b9b520
Update test/parallel/test-snapshot-cwd.js
arcanis Sep 18, 2023
d4b132b
Update test/parallel/test-snapshot-cwd.js
arcanis Sep 18, 2023
12eab3d
Update test/parallel/test-snapshot-cwd.js
arcanis Sep 18, 2023
e2109f7
Update test/parallel/test-snapshot-cwd.js
arcanis Sep 18, 2023
f02b286
Update test-snapshot-cwd.js
arcanis Sep 18, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: add a cwd cache bust test
  • Loading branch information
@arcanis
arcanis committed Sep 17, 2023
commit 7b2cfac4fc18a36e108c58256dfc188c86d94fd0
10 changes: 10 additions & 0 deletions test/fixtures/snapshot/cwd.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
const {
setDeserializeMainFunction,
} = require('v8').startupSnapshot;

// To make sure the cwd is present in the cache
process.cwd();

setDeserializeMainFunction(() => {
console.log(process.cwd());
});
49 changes: 49 additions & 0 deletions test/parallel/test-snapshot-cwd.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
'use strict';

// This tests that user land snapshots works when the instance restored from
// the snapshot is launched with -p and -e

Comment thread
arcanis marked this conversation as resolved.
Outdated
require('../common');
const assert = require('assert');
const { spawnSync } = require('child_process');
Comment thread
aduh95 marked this conversation as resolved.
Outdated
const tmpdir = require('../common/tmpdir');
const fixtures = require('../common/fixtures');
const fs = require('fs');
const path = require('path');

tmpdir.refresh();
const blobPath = tmpdir.resolve('snapshot.blob');
const file = fixtures.path('snapshot', 'cwd.js');

const subdir = path.join(tmpdir.path, 'foo');
Comment thread
arcanis marked this conversation as resolved.
Outdated
fs.mkdirSync(subdir);

{
// Create the snapshot.
const child = spawnSync(process.execPath, [
'--snapshot-blob',
blobPath,
'--build-snapshot',
file,
], {
cwd: tmpdir.path,
encoding: `utf8`
});

assert.strictEqual(child.status, 0);
}

{
// Check a custom works.
Comment thread
arcanis marked this conversation as resolved.
Outdated
Comment thread
arcanis marked this conversation as resolved.
Outdated
const child = spawnSync(process.execPath, [
'--snapshot-blob',
blobPath,
file,
], {
cwd: subdir,
encoding: `utf8`
});

assert.strictEqual(child.status, 0);
assert.strictEqual(child.stdout, `${subdir}\n`);
}