Skip to content

Commit 8c12209

Browse files
committed
chore: remove the snapshot test
we're going to have automation for generating the proto file anyways, there is no need to have the snapshot test for now so that it won't fail when we add a new CLI flag.
1 parent 0840ff0 commit 8c12209

2 files changed

Lines changed: 0 additions & 60 deletions

File tree

tests/telemetry/flag-utils.test.js.snapshot

Lines changed: 0 additions & 37 deletions
This file was deleted.

tests/telemetry/flag-utils.test.ts

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -104,27 +104,4 @@ describe('computeFlagUsage', () => {
104104
assert.equal(usage.string_flag_present, false);
105105
});
106106
});
107-
108-
it('matches snapshot for all current CLI options', async t => {
109-
// Import the real options to test against the actual CLI definition
110-
const {cliOptions} = await import('../../src/cli.js');
111-
112-
const mockArgs: Record<string, unknown> = {};
113-
for (const [key, config] of Object.entries(cliOptions)) {
114-
if ('choices' in config && config.choices) {
115-
mockArgs[key] = config.choices[0];
116-
} else if (config.type === 'boolean') {
117-
mockArgs[key] = true;
118-
} else if (config.type === 'string') {
119-
mockArgs[key] = '/mock/path';
120-
} else if (config.type === 'array') {
121-
mockArgs[key] = ['--mock-arg'];
122-
} else {
123-
mockArgs[key] = 'mock-value';
124-
}
125-
}
126-
127-
const usage = computeFlagUsage(mockArgs, cliOptions);
128-
t.assert.snapshot(JSON.stringify(usage, null, 2));
129-
});
130107
});

0 commit comments

Comments
 (0)