open-duelyst/duelyst

[P3] Repair and re-enable disabled SDK unit tests

Aperta

#25 aperta il 10 set 2022

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (526 fork)batch import
backendenhancementfrontendhelp wanted

Metriche repository

Star
 (3443 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Several unit tests have been disabled until we have time to go back and update them. search for Test disabled in the repo to find them:

% git grep -l 'Test disabled'
test/unit/sdk/ai/ai.js
test/unit/sdk/ai/bosses.js
test/unit/sdk/cards/bloodstorm/faction6.js
test/unit/sdk/cards/bloodstorm/neutrals.js
test/unit/sdk/cards/core/faction2/artifacts.js
test/unit/sdk/cards/core/faction3/minions.js
test/unit/sdk/cards/core/faction5/spells.js
test/unit/sdk/cards/core/neutrals/epics.js
test/unit/sdk/cards/coreshatter/faction1.js
test/unit/sdk/cards/misc/bloodborn_spells.js
test/unit/sdk/cards/monthlies/month7.js
test/unit/sdk/cards/monthlies/month9.js
test/unit/sdk/cards/monthlies/seven_sisters.js
test/unit/sdk/cards/shimzar/battle_pets.js
test/unit/sdk/cards/shimzar/faction1.js
test/unit/sdk/cards/shimzar/faction2.js
test/unit/sdk/cards/shimzar/faction4.js
test/unit/sdk/cards/shimzar/faction5.js
test/unit/sdk/cards/unity/faction5.js
test/unit/sdk/cards/unity/faction6.js
test/unit/sdk/cards/wartech/faction2.js
test/unit/sdk/cards/wartech/faction3.js
test/unit/sdk/cards/wartech/faction4.js
test/unit/sdk/cards/wartech/faction5.js
test/unit/sdk/game/basic.js

We should revisit these and turn them back on once we get them working.

Guida contributor

Direzione di ricerca
Innanzitutto, utilizza git grep per individuare i file contenenti 'Test disabled' come elencati nell'issue (ad esempio, test/unit/sdk/ai/ai.js). Per ogni file, esamina il test disabilitato e comprendine la logica. Cerca commenti o modifiche recenti che hanno causato il fallimento del test. Aggiorna il codice del test per risolvere eventuali problemi e rimuovi il marcatore 'Test disabled'. Assicurati che tutti i test passino. Il repository utilizza JavaScript e probabilmente un framework di test come Mocha o Jest—controlla package.json per i dettagli. Non esistono PR collegati o assegnatari, quindi questo compito è aperto per essere preso in carico.
Tech stack
javascript
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60