Skip to content

chore: Add unit tests for options functions and cli #2792

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

Jump to bottom
Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

chore: Add unit tests for options functions and cli #2792

wants to merge 17 commits into from

Conversation

inferrinizzard
Copy link
Contributor

@inferrinizzard inferrinizzard commented Jun 24, 2025 •
edited
Loading

Description

Related Issue

#2753

Motivation and Context

Previously testing coverage for CLI functionality was lacking

Previous Behaviour / Output

N/a

New Behaviour / Output

image

How Has This Been Tested?

vitest locally, vitest version ^3.1.4

Screenshots (if appropriate):

@inferrinizzard
chore: Lint and cleanup CLI package (glideapps#2758)
9f82f03 
* fix lint errors

* remove gratuitous lodash from test.ts

* resolve lint issues in index
@inferrinizzard
fix read name of undefined
fdf7c98
@inferrinizzard
Merge branch 'master' into release/bin/revamp
537254a
@inferrinizzard
fix ci
66bcaee
@inferrinizzard
chore: Split CLI index into multiple files (glideapps#2762)
37864fa 
* 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

* ♻️
@inferrinizzard
install vitest as devDep
24876e9
@inferrinizzard
use in instead of hasOwnProperty for src options
b71a838
@inferrinizzard
add basic unit test for cli args
8089fef
@inferrinizzard
add test to parse cli renderer options for all fixture languages
edecab5
@inferrinizzard
strictly generic type inference flag names
07f542c
@inferrinizzard
add test for infer cli options function
b6431ab
@inferrinizzard
rename usage
97f6f6c
@inferrinizzard
rename misc
3da8b90
@inferrinizzard
read both positive and negated inference flag values
ecc99ba
@inferrinizzard
add generated unit test for make quicktype options
62484a8
@inferrinizzard
add unit tests to test cmd
5b4e2f0
@inferrinizzard
add unit test to gh action instead
ff8bdf8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@inferrinizzard