openyurtio / yurt-edgex-manager

yurt-edgex-manager is an operator for managing EdgeX lifecycle(including deploy, delete, update) in OpenYurt cluster.
Apache License 2.0
18 stars 24 forks source link

Add API test for yurt-edgex-manager controller #40

Closed LindaYu17 closed 2 years ago

LindaYu17 commented 2 years ago

run: make test

Signed-off-by: Linda Yu linda.yu@intel.com

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line: /kind bug /kind documentation /kind enhancement /kind good-first-issue /kind feature /kind question /kind design /sig ai /sig iot /sig network /sig storage

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

LindaYu17 commented 2 years ago

ok github.com/openyurtio/yurt-edgex-manager/controllers 15.623s coverage: 37.3% of statements ? github.com/openyurtio/yurt-edgex-manager/controllers/utils [no test files] ? github.com/openyurtio/yurt-edgex-manager/pkg/webhook/edgex/validating [no test files]

gnunu commented 2 years ago

/LGTM

codecov-commenter commented 2 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@54be695). Click here to learn what that means. The diff coverage is n/a.

:exclamation: Current head 3a6f75a differs from pull request most recent head 56affa7. Consider uploading reports for the commit 56affa7 to get more accurate results

@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage        ?   29.84%           
=======================================
  Files           ?        1           
  Lines           ?      315           
  Branches        ?        0           
=======================================
  Hits            ?       94           
  Misses          ?      205           
  Partials        ?       16           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.