Refactor some cmd.js logic out into its own file for easier tests

Offen
#71 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
35/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
javascript
Bereich
cli

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.

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

Vorherrschende Sprache
JavaScript
Sterne
55
Forks
62
Ø Merge
4 T. 9 Std.
Gemergte PRs (30 T.)
1

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus nodejs/core-validate-commit

Alle Issues in nodejs/core-validate-commit

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.