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.

Sign up for GitHub

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

Jump to bottom

Add UT for the PR RedisRegistry #3312

Closed
2 tasks
khanimteyaz opened this issue Jan 22, 2019 · 1 comment
Closed
2 tasks

Add UT for the PR RedisRegistry #3312

khanimteyaz opened this issue Jan 22, 2019 · 1 comment

Comments

@khanimteyaz
Copy link
Contributor

Add missing UT for RedisRegistry which got identified as part of PR review

  • I have searched the issues of this repository and believe that this is not a duplicate.
  • I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

  • Dubbo version: xxx
  • Operating System version: xxx
  • Java version: xxx

Steps to reproduce this issue

  1. xxx
  2. xxx
  3. xxx

Pls. provide [GitHub address] to reproduce this issue.

Expected Result

What do you expected from the above steps？

Actual Result

What actually happens?

If there is an exception, please attach the exception trace:

Just put your stack trace here!
@khanimteyaz khanimteyaz mentioned this issue Jan 22, 2019
Merged
6 tasks
@kun-song kun-song assigned kun-song and unassigned kun-song Jan 29, 2019
@lixiaojiee
Copy link
Contributor

It seems that this issue has been resolved, and it can be closed now. @ralf0131

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@kun-song @kexianjun @khanimteyaz @lixiaojiee