impress/impress.js

Test coverage for all plugins

Open

#659 ouverte le 15 janv. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (6 605 forks)batch import
good first issuehelp wanted

Métriques du dépôt

Stars
 (38 178 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

As of this writing, the core impress.js API has ok test coverage, as does the navigation plugin. All other plugins were essentially implemented without tests. (Sorry, historical reasons.)

Now that they're merged to upstream impress.js though, we do have a goal of adding test coverage for all of them. Basically, it would be nice to have at least one src/plugins/pluginname/pluginname_tests.js file for each plugin.

This is a catch all ticket. If you want to help in creating tests for a plugin, please file a new issue with a title like "Add tests for plugin pluginname"

Related: #657

Guide contributeur

Direction de recherche
Choisissez un plugin non testé, vérifiez si un fichier de test existe déjà dans son répertoire et écrivez un test de base en utilisant le framework de test existant (probablement Jasmine ou Mocha) pour vérifier la fonctionnalité du plugin.
Stack technique
javascript
Domaine
frontend
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.
2
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
GitJavaScript testing
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65