search

jina-ai / jcloud

Simplify deploying and managing Jina projects on Jina Cloud
https://docs.jina.ai/concepts/jcloud/
Apache License 2.0
298 stars 13 forks source link

test: add integration test to cover wolf's jina version support #55

Closed zac-li closed 2 years ago

zac-li commented 2 years ago

Goal

A part of https://github.com/jina-ai/wolf/issues/117, we need to make sure e2e tests starting from client side cover the new feature as well.

@jina-ai/team-wolf

codecov[bot] commented 2 years ago

Codecov Report

Merging #55 (345cd7d) into main (d674cf0) will decrease coverage by 0.41%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
- Coverage   55.13%   54.72%   -0.42%     
==========================================
  Files          15       15              
  Lines         974      974              
==========================================
- Hits          537      533       -4     
- Misses        437      441       +4
Flag Coverage Δ
jcloud 54.72% <ø> (-0.42%) :arrow_down:

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

Impacted Files Coverage Δ
jcloud/flow.py 88.46% <0.00%> (-1.16%) :arrow_down:
jcloud/helper.py 75.23% <0.00%> (-0.96%) :arrow_down:

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 d674cf0...345cd7d. Read the comment docs.

zac-li commented 2 years ago

Integration test https://github.com/jina-ai/jcloud/actions/runs/2472380259 against this branch always failed due to 3 tests, can't figure out why, maybe this branch is in some weird state. Let's merge it anyway and see if it will resolve by itself.