ipfs/ipfs-desktop

Improve Test Suite

Open

#1121 aperta il 13 set 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (912 fork)batch import
P2effort/daysexp/intermediatehelp wantedkind/maintenance

Metriche repository

Star
 (5632 star)
Metriche merge PR
 (Merge medio 8g 12h) (6 PR mergiate in 30 g)

Descrizione

Right now, we have quite a few unit tests and some end-to-end tests for the launching process. We should aim to improve this and get a better coverage of the code.

Unit Tests

We are aiming for one unit test file per file. The biggest issue with unit tests is the amount of mocking we need to do to make sure we don't have side-effects.

E2E

We are using Spectron for our end-to-end tests and it can sometimes be limiting since we have a menubar application and Spectron does not allow to test menus.

Some interesting E2E tests that we should aim to make (if possible):

  • Test menubar interaction.
  • Test global shortcuts functionality.
  • Test adding ipfs to PATH.
  • Test Protocol Handlers (macOS and Windows only).

Guida contributor

Direzione di ricerca
Inizia scrivendo test unitari per singoli file, concentrandoti sulla simulazione delle dipendenze. Quindi tenta di aggiungere test E2E per l'interazione con la barra dei menu, le scorciatoie globali, l'aggiunta di IPFS a PATH e i gestori di protocollo utilizzando Spectron o metodi alternativi.
Tech stack
javascript
Dominio
desktop
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaScriptJestSpectron
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60