sealerio/sealer

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

オープン

#1,488 opened on 2022/06/08

GitHub で見るコントリビューターガイド
 (1 件のコメント) (0 件のリアクション) (0 人の担当者)Go (357 件のフォーク)github user discovery
good first issuehelp wantedkind/featuretest

Repository metrics

Stars
 (2,096 個のスター)
PR merge metrics
 (30d に merged PR はありません)

説明

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.

コントリビューターガイド

調査方針
コードベース内のコメントアウトされた単体テスト（例: build/layerutils/charts/charts test.go、pkg/config/config test.go）を見つけ、コメントを解除し、テストを実行し、失敗を修正します。
技術スタック
go
領域
backend
Issue 種別
テスト
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
GoTesting
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
80