sealerio/sealer

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

Ouverte

#1 488 ouverte le 8 juin 2022

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Go (358 forks)github user discovery
good first issuehelp wantedkind/featuretest

Métriques du dépôt

Stars
 (2 096 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Trouvez les tests unitaires commentés dans la base de code (par exemple, build/layerutils/charts/charts test.go et pkg/config/config test.go), décommentez les, exécutez les tests et corrigez les échecs.
Stack technique
go
Domaine
backend
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.
Une demi journée
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.
Claire
Prérequis
GoTesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80