impress/impress.js

Test coverage for all plugins

Open

#659 aperta il 15 gen 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (6605 fork)batch import
good first issuehelp wanted

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Scegli un plugin non testato, controlla se esiste già un file di test nella sua directory e scrivi un test di base utilizzando il framework di test esistente (probabilmente Jasmine o Mocha) per verificare la funzionalità del plugin.
Tech stack
javascript
Dominio
frontend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
GitJavaScript testing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65