ipfs/ipfs-desktop

Improve Test Suite

Open

#1 121 ouverte le 13 sept. 2019

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (912 forks)batch import
P2effort/daysexp/intermediatehelp wantedkind/maintenance

Métriques du dépôt

Stars
 (5 632 stars)
Métriques de merge PR
 (Merge moyen 8j 12h) (6 PRs mergées en 30 j)

Description

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).

Guide contributeur

Direction de recherche
Commencez par écrire des tests unitaires pour des fichiers individuels, en vous concentrant sur le mock des dépendances. Ensuite, essayez d'ajouter des tests E2E pour l'interaction avec la barre de menu, les raccourcis globaux, l'ajout d'IPFS au PATH et les gestionnaires de protocole en utilisant Spectron ou des méthodes alternatives.
Stack technique
javascript
Domaine
desktop
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaScriptJestSpectron
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60