regression: CLI processing of globs
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 1.2k
- Forks
- 242
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
The prior version of HTTPSnippet allowed processing of globs on the CLI as follows:
httpsnippet ./*.json --target node --client request --output ./snippets
$ tree snippets/
snippets/
├── endpoint-1.js
├── endpoint-2.js
└── endpoint-3.js
I wasn't aware of this prior functionality (and there were no tests covering it) so it was missed.
Shouldn't be too terrible to add back. Let's have some tests for it this time, too.
Reproduction steps
run the following:
./bin/httpsnippet src/fixtures/requests/*.json --target node --client unirest --output ./example
And it returns this input:
Unknown arguments: src/fixtures/requests/application-json.json, src/fixtures/requests/cookies.json, src/fixtures/requests/custom-method.json, src/fixtures/requests/full.json, src/fixtures/requests/headers.json, src/fixtures/requests/https.json, src/fixtures/requests/jsonObj-multiline.json, src/fixtures/requests/jsonObj-null-value.json, src/fixtures/requests/multipart-data.json, src/fixtures/requests/multipart-file.json, src/fixtures/requests/multipart-form-data.json, src/fixtures/requests/multipart-form-data-no-params.json, src/fixtures/requests/nested.json, src/fixtures/requests/query.json, src/fixtures/requests/short.json, src/fixtures/requests/text-plain.json
When it should, instead, process all of those files.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem CLI-Einstiegspunkt unter bin/httpsnippet und reproduziere den Fehler mit dem glob-Befehl für src/fixtures/requests/*.json. Verfolge, wie Argumente geparst werden und wie eine einzelne Eingabe verarbeitet wird, und füge anschließend CLI-Tests hinzu, die mehrere per glob gefundene Dateien abdecken, und überprüfe, dass jede Eingabe eine Ausgabe erzeugt, statt einen Fehler wegen unbekannter Argumente zu verursachen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- cli, testing
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100