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

增加流量标签透传的UT,完善流量标签透传拦截器继承结构 #1263

Merged
merged 3 commits into from
Aug 14, 2023

Conversation

TangLeDaily
Copy link
Collaborator

【修复issue】#1244

【修改内容】
增加了HttpClient的UT
增加了HttpServlet的UT
增加了KafkaConsumerRecord的UT
增加了KafkaProducer的UT
增加了RocketmqConsumer的UT
增加了RocketmqProducer的UT

修改了HttpServletInterceptor的继承结构
修改了RocketmqConsumerInterceptor的继承结构
修改了RocketmqProducerInterceptor的继承结构

【用例描述】增加了流量标签透传模块的UT

【自测情况】本地编译测试通过

【影响范围】无

@TangLeDaily TangLeDaily self-assigned this Jul 27, 2023
@TangLeDaily TangLeDaily changed the title 【UT】增加流量标签透传的UT,完善流量标签透传拦截器继承结构 增加流量标签透传的UT,完善流量标签透传拦截器继承结构 Jul 27, 2023
@TangLeDaily TangLeDaily added kind/unit-test Issues or PRs releated to sermant test area/plugin Issues or PRs releated to sermant plugin service labels Jul 27, 2023
@TangLeDaily TangLeDaily force-pushed the tagTransmissionUT_a branch 2 times, most recently from fd9d848 to 1874a5c Compare July 27, 2023 07:22
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.31% ⚠️

Comparison is base (c8fca81) 42.65% compared to head (4497e5b) 42.34%.
Report is 3 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #1263      +/-   ##
=============================================
- Coverage      42.65%   42.34%   -0.31%     
  Complexity      3230     3230              
=============================================
  Files            816      821       +5     
  Lines          17762    17895     +133     
  Branches        1935     1947      +12     
=============================================
+ Hits            7576     7578       +2     
- Misses          9234     9365     +131     
  Partials         952      952              
Flag Coverage Δ
unittests 42.34% <ø> (-0.31%) ⬇️

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

see 26 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@TangLeDaily TangLeDaily force-pushed the tagTransmissionUT_a branch 2 times, most recently from 3a111b0 to cac61b6 Compare August 2, 2023 02:02
@luanwenfei-venus luanwenfei-venus added area/plugin Issues or PRs releated to sermant plugin service and removed area/plugin Issues or PRs releated to sermant plugin service labels Aug 10, 2023
lilai23
lilai23 previously approved these changes Aug 11, 2023
@Sherlockhan Sherlockhan merged commit 815d7b7 into sermant-io:develop Aug 14, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/plugin Issues or PRs releated to sermant plugin service kind/unit-test Issues or PRs releated to sermant test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants