hound-search/hound

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

Open

#363 aperta il 30 ott 2020

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (606 fork)batch import
buggood first issuehelp wanted

Metriche repository

Star
 (5470 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Scopri come escludere la directory `db/` dalle esecuzioni di `jest` e `go test`. Per jest, aggiungi `testPathIgnorePatterns` o `modulePathIgnorePatterns` nella configurazione jest. Per go test, modifica il comando di test per filtrare le directory usando `go list` con `grep v` o usa un pattern ` run`. Aggiorna anche `make clean` per includere `db/`.
Tech stack
javascriptgo
Dominio
backendtesting qadeveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitGoNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70