Test runner cannot find snapshot with `\r` in title

[melusc]

Version

23.7.0

Platform

- my PC: Microsoft Windows NT 10.0.26100.0 x64
- ubuntu-latest on GitHub Actions
- windows-latest on GitHub Actions

Subsystem

Test runner

What steps will reproduce the bug?

test.ts:

import {test} from 'node:test';

test('\r', t => {
	t.assert.snapshot({json: 'value'});
});

$ node --test-update-snapshots test.ts
$ node --test test.ts

How often does it reproduce? Is there a required condition?

Always

What is the expected behavior? Why is that the expected behavior?

It should be able to select the correct snapshot that it has just written.

What do you see instead?

✖ failing tests:

test at test.ts:3:1
 (2.8696ms)
 1' not found in '[...]\node-test-snapshot-cr\test.ts.snapshot.' Missing snapshots can be generated by rerunning the command with the --test-update-snapshots flag.
      at SnapshotFile.getSnapshot (node:internal/test_runner/snapshot:69:19)
      at TestContext.snapshotAssertion (node:internal/test_runner/snapshot:206:41)
      at assert.<computed> [as snapshot] (node:internal/test_runner/test:254:18)
      at TestContext.<anonymous> (file:///[...]/node-test-snapshot-cr/test.ts:4:11)
      at Test.runInAsyncScope (node:async_hooks:211:14)
      at Test.run (node:internal/test_runner/test:980:25)
      at Test.start (node:internal/test_runner/test:878:17)
      at startSubtestAfterBootstrap (node:internal/test_runner/harness:297:17) {
    code: 'ERR_INVALID_STATE',
    snapshot: '\r 1',
    filename: '[...]\\node-test-snapshot-cr\\test.ts.snapshot'
  }

Additional information

• https://github.com/melusc/node-test-snapshot-cr
• https://github.com/melusc/node-test-snapshot-cr/actions/runs/13065943568

[melusc]

For fun, I tested all char codes 0 - 65536. Besides \r, it fails for all characters with codes in the range 55296-57343 (incl.)

E.g. with 57343:

// $ node --test-update-snapshots test.ts
// $ node --test test.ts

import {test} from 'node:test';

test(String.fromCharCode(57343), t => {
	t.assert.snapshot({key: 'value'});
});

I have omitted the error for brevity but it is the same error as above

[cjihrig]

Thanks for reporting this. The issue is that vm.runInContext() is used to evaluate the snapshot files, and it replaces characters:

import { createContext, runInContext } from 'node:vm';
const context = { __proto__: null, exports: { __proto__: null } };
const source = 'exports[`\rfoo`] = 1;';

createContext(context);
runInContext(source, context);
console.log(context.exports);
// Logs: [Object: null prototype] { '\nfoo': 1 }

I think we just need some additional escaping for the snapshot keys in this file.

[Ceres6]

I'll take a look