nodejs / nodejs/core-validate-commit
Refactor some cmd.js logic out into its own file for easier tests
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 55
- Forks
- 62
- Ø Merge
- 4 T. 9 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
While working on issue #69 I started to come across places where it was a little difficult to test if just a particular flag was called. I think it might help if some of the logic that is in cmd.js is broken out into its own file, so it would be easier to test if a particular flag was specified, and not necessarily test what that flag actually does. That could be done in its own separate test, where it would be easier to mock certain functions.
@richardlau @Trott thoughts?
I can send a PR so we can talk about it also
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
Lies zunächst cmd.js und identifiziere die Logik, die bestimmt, ob bestimmte Flags angegeben wurden. Lagere diese Logik in eine eigene Datei aus, damit sie unabhängig getestet werden kann, bei Bedarf mit gemockten Funktionen. Die Aufgabe ist erledigt, wenn die Flag-Erkennung fokussierte Tests hat, die nicht gleichzeitig testen, was jedes Flag bewirkt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- cli
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100