chore: Add unit tests for options functions and cli#2792
chore: Add unit tests for options functions and cli#2792inferrinizzard wants to merge 18 commits into
Conversation
* fix lint errors * remove gratuitous lodash from test.ts * resolve lint issues in index
* fix lint errors * remove gratuitous lodash from test.ts * resolve lint issues in index * split CLIOptions interface to own file * split cli options, inference, option definitions, utils to own files * split sources to own file * move input, usage to own file * move quicktype options to own file * ♻️
10efa0c to
ff8bdf8
Compare
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Thanks for this — splitting 1. Inference flags default to OFF for every invocation (functional bug). In 2. 3. 4. Real conflicts with current master in 8 files ( 5. This overlaps almost entirely with your other open PR, #2761, which does the same CLI refactor/split. We'd rather not merge both — could you consolidate into a single PR? Given the overlap and the master drift, would you be up for redoing this as one refactor branched off current master, folding in #2761? The bug-1 fix and the mock-test rewrite are hard requirements, and we'll need a green fixture-test CI run (all languages, not fail-fast-cancelled) before merge. |
Description
makeQuicktypeOptionsRelated Issue
#2753
Motivation and Context
Previously testing coverage for CLI functionality was lacking
Previous Behaviour / Output
N/a
New Behaviour / Output
How Has This Been Tested?
vitestlocally, vitest version^3.1.4Screenshots (if appropriate):