bentoml / bentoctl

Fast model deployment on any cloud 🚀
https://bentoml.com
Other
175 stars 30 forks source link

test: adding tests #119

Closed jjmachan closed 2 years ago

jjmachan commented 2 years ago

Description

Motivation and Context

How Has This Been Tested?

Checklist:

codecov[bot] commented 2 years ago

Codecov Report

Merging #119 (ed7fdcd) into main (67f51c5) will increase coverage by 34.97%. The diff coverage is 91.66%.

@@             Coverage Diff             @@
##             main     #119       +/-   ##
===========================================
+ Coverage   38.14%   73.12%   +34.97%     
===========================================
  Files          16       16               
  Lines         894      878       -16     
===========================================
+ Hits          341      642      +301     
+ Misses        553      236      -317     
Flag Coverage Δ
unit-tests 73.12% <91.66%> (+34.97%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
bentoctl/deployment_config.py 66.66% <ø> (+3.23%) :arrow_up:
bentoctl/cli/interactive.py 91.30% <80.00%> (+91.30%) :arrow_up:
bentoctl/cli/__init__.py 86.90% <100.00%> (+86.90%) :arrow_up:
bentoctl/docker_utils.py 20.00% <0.00%> (+20.00%) :arrow_up:
bentoctl/utils.py 73.68% <0.00%> (+36.84%) :arrow_up:
bentoctl/cli/operator_management.py 40.27% <0.00%> (+40.27%) :arrow_up:
bentoctl/cli/helper_scripts.py 45.00% <0.00%> (+45.00%) :arrow_up:
bentoctl/console.py 92.30% <0.00%> (+61.53%) :arrow_up:
... and 1 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

jjmachan commented 2 years ago

Merging this in. Looking forward to pairing up sometime @yubozhao and getting it to the 80+ coverage mark