mosaicstack/stack
25
0 Fork 1
Code Issues 404 Pull Requests 13 Packages Projects Releases 15 Wiki Activity

test(cli): json-redaction pins follow the label refusals

Browse Source 
The review round changed the config-command relay and the org document
refusal to name the file by label. cli/tests/json-redaction.test.mjs still
required the host path in those two CLI refusals. They now require the
label and the path's absence. The direct scripts/mosaic-config.mjs checks
and the loadSystem check (PL1, deferred) still require the path.
This commit was merged in pull request #1926.
This commit is contained in:
jason.woltje committed 2026-11-11 01:04:15 -06:00
1 parent d085b16136
commit be698e602b
1 file changed
+9 -4
Download Patch File Download Diff File Expand all files Collapse all files
packages/cli/tests/json-redaction.test.mjs
+9 -4
View File
@@ -21,10 +21,15 @@ function fixture(t) {
}; };
return { root, file, env, run, boot }; return { root, file, env, run, boot };
} }
function redacted(result, file, status) { // label: the CLI refusal names the file by this label and never by host path (PR #1916).
// Without it, the refusal comes from scripts/mosaic-config.mjs, which still names the file.
function redacted(result, file, status, label) {
assert.equal(result.status, status, result.stderr); assert.equal(result.status, status, result.stderr);
assert.equal(result.stdout, ''); assert.equal(result.stdout, '');
assert.ok(result.stderr.includes(file), result.stderr); if (label) {
assert.ok(!result.stderr.includes(file), result.stderr);
assert.ok(result.stderr.includes(label), result.stderr);
} else assert.ok(result.stderr.includes(file), result.stderr);
assert.match(result.stderr, /not valid JSON/); assert.match(result.stderr, /not valid JSON/);
assert.doesNotMatch(result.stderr, /sk-test|Unexpected token/); assert.doesNotMatch(result.stderr, /sk-test|Unexpected token/);
} }
@@ -40,7 +45,7 @@ test('system config parse refusal and CLI relay omit parser bytes', t => {
assert.doesNotMatch(error.message, /sk-test|Unexpected token/); assert.doesNotMatch(error.message, /sk-test|Unexpected token/);
return true; return true;
}); });
redacted(f.run(cli, 'org', 'config', 'validate', 'alpha'), f.file, 3); redacted(f.run(cli, 'org', 'config', 'validate', 'alpha'), f.file, 3, 'configuration is not valid JSON; expected readable JSON: the system config MOSAIC_CONFIG names');
}); });
test('named instance marker parse refusal omits parser bytes', t => { test('named instance marker parse refusal omits parser bytes', t => {
@@ -55,5 +60,5 @@ for (const operation of ['validate', 'create', 'update']) test(`org ${operation}
const file = join(f.root, 'alpha.json'); writeFileSync(file, secret); const file = join(f.root, 'alpha.json'); writeFileSync(file, secret);
const args = operation === 'validate' ? ['validate', file] const args = operation === 'validate' ? ['validate', file]
: [operation, 'alpha', '--from', file, ...(operation === 'update' ? ['--if-revision', '1'] : [])]; : [operation, 'alpha', '--from', file, ...(operation === 'update' ? ['--if-revision', '1'] : [])];
redacted(f.run(cli, 'org', ...args), file, 2); redacted(f.run(cli, 'org', ...args), file, 2, 'mosaic: cannot read organization document: not valid JSON\n');
}); });