search

AmadeusITGroup / workflow-controller

Kubernetes workflow controller
Apache License 2.0
24 stars 15 forks source link

add test cases to TestControllerSyncWorkflow #36

Closed alexei-led closed 6 years ago

alexei-led commented 6 years ago

I try to learn workflow-controller codebase through testing, so adding some tests now

codecov-io commented 6 years ago

Codecov Report

Merging #36 into master will increase coverage by 10.9%. The diff coverage is 58.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   25.92%   36.82%   +10.9%     
==========================================
  Files           9        9              
  Lines         841      850       +9     
==========================================
+ Hits          218      313      +95     
+ Misses        605      511      -94     
- Partials       18       26       +8
Impacted Files Coverage Δ
pkg/controller/controller.go 36.93% <58.82%> (+19.96%) :arrow_up:
pkg/controller/controller_util.go 48.21% <0%> (+48.21%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6fc8f5a...f7536ff. Read the comment docs.