sealerio/sealer

[unit-test] Correct commented unit tests to take them back

Aperta

#1488 aperta il 8 giu 2022

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Go (357 fork)github user discovery
good first issuehelp wantedkind/featuretest

Metriche repository

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

Descrizione

In pull request https://github.com/sealerio/sealer/pull/1476, we have succeeded in making unit-test work. While after that pull request, we found that we have very few unit test coverage, and this will definitely lead to the poor quality of sealer.

So I think we should try to gradually add all those commented unit tests back.

And we encourage the all community participants could feel free to help and contribute.

The example comment test case is like :

https://github.com/sealerio/sealer/blob/main/build/layerutils/charts/charts_test.go#L17-L47

https://github.com/sealerio/sealer/blob/main/pkg/config/config_test.go#L24-L106

and so on.

Guida contributor

Direzione di ricerca
Trova i test unitari commentati nel codebase (es. build/layerutils/charts/charts test.go e pkg/config/config test.go), decommentali, esegui i test e correggi eventuali errori.
Tech stack
go
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoTesting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80