-
Notifications
You must be signed in to change notification settings - Fork 172
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
add rocketmq gray plugin workflow test #1660
base: develop
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Flags with carried forward coverage won't be shown. Click here to find out more. |
18a1251
to
5c338d0
Compare
* 启动类 | ||
* | ||
* @param args 进程启动入参 | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English comments are recommended.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
if (offSetTable.get(mq) != null) { | ||
return offSetTable.get(mq); | ||
} | ||
return 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return map.getOrDefault(mq, 0L)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
* 构造函数 | ||
* | ||
* @param restTemplate 请求器 | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English comments are recommended.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
using: "composite" | ||
steps: | ||
- name: Set up JDK ${{ env.javaVersion }} | ||
uses: actions/setup-java@v3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use v4 version for setup-java、cache、checkout action
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
strategy: | ||
matrix: | ||
test-model: [ 'PLUGIN_ENABLED_FALSE','CONSUMER_BASE_ONLY','CONSUMER_BASE_GRAY' ] | ||
rocketMqClientVersion: ["4.8.0", "4.9.0", "4.9.1", "4.9.2", "4.9.3", "4.9.4", "4.9.5", "4.9.6", "4.9.7", "4.9.8", "5.0.0"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5.1.x version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SubscriptionData class path changed from 5.1.x,and bug fixed later.
d5e3acb
to
cba46fb
Compare
Signed-off-by: chengyouling <[email protected]>
eea18b1
to
922ad63
Compare
What type of PR is this?
UT tests
What this PR does / why we need it?
Function coverage test for mq grayscale.
Which issue(s) this PR fixes?
#1659
Does this PR introduce a user-facing change?
No