hound-search/hound

`make test` fails if `db/` has contents.

Open

#363 geöffnet am 30. Okt. 2020

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (606 Forks)batch import
buggood first issuehelp wanted

Repository-Metriken

Stars
 (5.470 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

If you start the houndd server, /db becomes populated with the contents of other repos. Both jest and go test currently crawl over /db and get confused by it, usually because there are naming collisions.

I think three things would make this a little better:

  1. make clean should also empty db/.
  2. jest should ignore db/, which seems easy to do in a jest-config.js file.
  3. go test should ignore db, which seems like we have to do manually, by using something like go test 'go list ./... | grep -v directoriesToSkip\` (I got that example from stack overflow).

A band-aid could be to just do 1 above and then run it before we run tests. That doesn't seem ideal, though, because we'd have to re-install node_modules between make clean and running the tests.

Contributor Guide

Research-Richtung
Untersuchen Sie, wie das Verzeichnis `db/` von den Ausführungen von `jest` und `go test` ausgeschlossen werden kann. Fügen Sie für jest `testPathIgnorePatterns` oder `modulePathIgnorePatterns` in der Jest Konfiguration hinzu. Passen Sie für go test den Testbefehl an, um Verzeichnisse mit `go list` und `grep v` zu filtern oder ein ` run` Muster zu verwenden. Aktualisieren Sie auch `make clean`, um `db/` einzuschließen.
Tech Stack
javascriptgo
Domain
backendtesting qadeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitGoNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70