search

openyurtio / yurt-app-manager

The workload controller manager from NodePool level in OpenYurt cluster
Apache License 2.0
6 stars 1 forks source link

[unit test] improve unit test coverage for yurtappset #74

Closed rambohe-ch closed 2 years ago

rambohe-ch commented 2 years ago

What would you like to be added: improve unit test coverage for yurtappset controller:

target coverage: 70%

Why is this needed: improve unit test coverage for OpenYurt project.

others /kind feature

River-sh commented 2 years ago

/assign

Congrool commented 2 years ago

We commented yurtappset_controller_test.go under pkg/yurtappmanager/controller/yurtappset in pr #72. We'd better take care of it in this pr.

rambohe-ch commented 2 years ago

We commented yurtappset_controller_test.go under pkg/yurtappmanager/controller/yurtappset in pr #72. We'd better take care of it in this pr.

@River-sh FYI

rambohe-ch commented 2 years ago

Fixes by #113