Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add Redis Component in integrate test #412

Merged
merged 4 commits into from
Mar 29, 2022

Conversation

zhenjunMa
Copy link
Contributor

@zhenjunMa zhenjunMa commented Mar 14, 2022

What this PR does:

add integrate test for runtime.

  1. Component List:
  • redis
  1. Api List (not all features) :
  • hello api
  • state api
  • lock api
  • sequencer api

@codecov
Copy link

codecov bot commented Mar 14, 2022

Codecov Report

Merging #412 (b645614) into main (f431199) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #412   +/-   ##
=======================================
  Coverage   60.18%   60.19%           
=======================================
  Files         117      117           
  Lines        6252     6253    +1     
=======================================
+ Hits         3763     3764    +1     
  Misses       2128     2128           
  Partials      361      361           
Impacted Files Coverage Δ
sdk/go-sdk/client/hello.go 71.42% <100.00%> (+4.76%) ⬆️

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 f431199...b645614. Read the comment docs.

@zhenjunMa zhenjunMa changed the title add Redis Component in integrate test feat:add Redis Component in integrate test Mar 29, 2022
@zhenjunMa zhenjunMa changed the title feat:add Redis Component in integrate test feat: add Redis Component in integrate test Mar 29, 2022
Copy link
Member

@seeflood seeflood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@zhenjunMa zhenjunMa merged commit 498bacb into mosn:main Mar 29, 2022
@zhenjunMa zhenjunMa deleted the integrate-test branch March 29, 2022 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants