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

[Task] ShenYu Admin test case #4540

Closed
38 of 40 tasks
moremind opened this issue Apr 9, 2023 · 21 comments
Closed
38 of 40 tasks

[Task] ShenYu Admin test case #4540

moremind opened this issue Apr 9, 2023 · 21 comments
Assignees
@hudongdong129 @whenelse @devinsong77 @yeshadoo @zuobiao-zhou @sachin10fi @lijay7674
Labels
good first issue Good for newcomers status: volunteer wanted Extra attention is needed

Comments

@moremind
Copy link
Member

moremind commented Apr 9, 2023 •
edited
Loading

Description

Add test cases for shenyu-admin

Task List

  1. Add test case for DocParameter @zuobiao-zhou
  2. Add test case for UpstreamInstance @zuobiao-zhou
  • service:
  1. Add test case for DocManagerImpl @lijay7674
  2. Add test case for LoadServiceDocEntryImpl @lijay7674
  3. Add test case for ServiceDocManagerImpl @lijay7674
  4. Add test case for SwaggerDocParser @lijay7674
  5. Add test case for ShenyuClientRegisterBrpcServiceImpl @sachin10fi
  6. Add test case for AbstractContextPathRegisterService @sachin10fi
  7. Add test case for AbstractShenyuClientRegisterServiceImpl @sachin10fi
  8. Add test case for FallbackShenyuClientRegisterService @sachin10fi
  9. Add test case for ShenyuClientRegisterWebSocketServiceImpl @sachin10fi
  1. Add test case for BatchPluginHandleChangedEvent @whenelse
  2. Add test case for PluginHandleChangedEvent @whenelse
  3. Add test case for BatchMetaDataChangedEvent @whenelse
  4. Add test case for BatchMetaDataDeletedEvent @whenelse
  5. Add test case for MetaDataChangedEvent @whenelse
  6. Add test case for MetaDataCreatedEvent @whenelse
  7. Add test case for MetadataUpdatedEvent @whenelse
  8. Add test case for BatchPluginChangedEvent @whenelse
  9. Add test case for BatchPluginDeletedEvent @whenelse
  10. Add test case for PluginChangedEvent
  11. Add test case for PluginCreatedEvent
  12. Add test case for public class BatchResourceDeletedEvent extends BatchChangedEvent
  13. Add test case for BatchRoleDeletedEvent
  14. Add test case for RoleChangedEvent
  15. Add test case for RoleCreatedEvent
  16. Add test case for RoleUpdatedEvent
  17. Add test case for BatchRuleDeletedEvent
  18. Add test case for RuleChangedEvent
  19. Add test case for RuleCreatedEvent
  20. Add test case for RuleUpdatedEvent
  21. Add test case for BatchSelectorDeletedEvent
  22. Add test case for SelectorChangedEvent
  23. Add test case for SelectorCreatedEvent
  24. Add test case for SelectorUpdatedEvent
  25. Add test case for BatchUserDeletedEvent
  26. Add test case for UserChangedEvent
  27. Add test case for UserCreatedEvent
  28. Add test case for UserUpdatedEvent
  • utils
  1. Add test case for HttpUtils @hudongdong129
@moremind moremind added status: volunteer wanted Extra attention is needed good first issue Good for newcomers labels Apr 9, 2023
@hudongdong129
Copy link
Contributor

please Assign this task to me for Add test case for HttpUtils

@moremind
Copy link
Member Author

moremind commented Apr 9, 2023

please Assign this task to me for Add test case for HttpUtils

assigned

@zuobiao-zhou
Copy link
Contributor

please assign the task of add test case for model.

@moremind
Copy link
Member Author

please assign the task of add test case for model.

assigned

@zuobiao-zhou zuobiao-zhou mentioned this issue Apr 11, 2023
Merged
3 tasks
@lijay7674
Copy link
Contributor

I want to try to complete these tasks,please Assign this task to me for Add test case for service

@moremind
Copy link
Member Author

I want to try to complete these tasks,please Assign this task to me for Add test case for service

which task?

yu199195 pushed a commit that referenced this issue Apr 12, 2023
@zuobiao-zhou @moremind
[type:test] add test cases for shenyu-admin.model (#4540) (#4550)
5149a14 
* [type:test] add test for shenyu-admin.model (#4540)

* [type:test] add test for shenyu-admin.model (#4540)

---------

Co-authored-by: moremind <hefengen@hotmail.com>
@lijay7674
Copy link
Contributor

The first four of the service

@moremind
Copy link
Member Author

The first four of the service

assigned

@lijay7674 lijay7674 removed their assignment Apr 12, 2023
@lijay7674
Copy link
Contributor

The first four of the service

assigned

I'm sorry, accidentally click wrong

@sachin10fi
Copy link
Contributor

Can you please assign me the rest of the service classes

@moremind
Copy link
Member Author

Can you please assign me the rest of the service classes

assigned

moremind pushed a commit that referenced this issue Apr 23, 2023
@sachin10fi @li-keguo
[ISSUE #4540] Added service test cases for admin (#4579)
5c2760f 
* Added Service Test Cases

* Fixing Style Issues

---------

Co-authored-by: likeguo <33576070+li-keguo@users.noreply.github.com>
yu199195 pushed a commit to yu199195/shenyu that referenced this issue May 5, 2023
@sachin10fi @li-keguo
[ISSUE apache#4540] Added service test cases for admin (apache#4579)
2f31bec 
* Added Service Test Cases

* Fixing Style Issues

---------

Co-authored-by: likeguo <33576070+li-keguo@users.noreply.github.com>
@devinsong77
Copy link
Contributor

hi @moremind，can you assign the task add test case for BatchRoleDeletedEvent , RoleChangedEvent, RoleCreatedEvent, RoleUpdatedEvent to me, thank you

@yeshadoo
Copy link
Contributor

yeshadoo commented Aug 2, 2023

hi, @moremind, please assign BatchRuleDeletedEvent、RoleChangedEvent、RoleCreatedEvent、RoleUpdatedEvent、BatchSelectorDeletedEvent、SelectorChangedEvent、SelectorCreatedEvent、SelectorUpdatedEvent for me , thanks.

@moremind
Copy link
Member Author

moremind commented Aug 4, 2023

hi @moremind，can you assign the task add test case for BatchRoleDeletedEvent , RoleChangedEvent, RoleCreatedEvent, RoleUpdatedEvent to me, thank you

assigned

@moremind
Copy link
Member Author

moremind commented Aug 4, 2023

hi, @moremind, please assign BatchRuleDeletedEvent、RoleChangedEvent、RoleCreatedEvent、RoleUpdatedEvent、BatchSelectorDeletedEvent、SelectorChangedEvent、SelectorCreatedEvent、SelectorUpdatedEvent for me , thanks.

assigned,

devinsong77 added a commit to devinsong77/shenyu that referenced this issue Aug 6, 2023
@devinsong77
[type:test][ISSUE apache#4540] add test cases for shenyu.admin.model.…
46fd343 
…event.role (apache#4540)
@devinsong77 devinsong77 mentioned this issue Aug 6, 2023
Merged
3 tasks
devinsong77 added a commit to devinsong77/shenyu that referenced this issue Aug 11, 2023
@devinsong77
[type:test][ISSUE apache#4540] add test cases for shenyu.admin.model.…
5104d00 
…event.role (apache#4540)
847850277 pushed a commit that referenced this issue Aug 12, 2023
@devinsong77
[type:test][ISSUE #4540] add test cases for shenyu.admin.model.event.…
fa5adf3 
…role (#4540) (#4965)
@whenelse
Copy link
Contributor

please Assign this task to me for Add test case for PluginChangedEvent and PluginCreatedEvent

@whenelse
Copy link
Contributor

hi, @moremind, please assign BatchPluginHandleChangedEvent、PluginHandleChangedEvent、BatchPluginChangedEvent、BatchPluginDeletedEvent for me , thanks.

@moremind
Copy link
Member Author

hi, @moremind, please assign BatchPluginHandleChangedEvent、PluginHandleChangedEvent、BatchPluginChangedEvent、BatchPluginDeletedEvent for me , thanks.

assigned

whenelse pushed a commit to whenelse/shenyu that referenced this issue Oct 17, 2023
[type:test][ISSUE apache#4540] add plugin test cases for admin
4736fca
@whenelse
Copy link
Contributor

hi, @moremind also need to assign the above PluginChangedEvent and PluginCreatedEvent

whenelse pushed a commit to whenelse/shenyu that referenced this issue Oct 17, 2023
[type:test][ISSUE apache#4540] add plugin test cases for admin
73960b2
@whenelse whenelse mentioned this issue Oct 17, 2023
Merged
3 tasks
whenelse pushed a commit to whenelse/shenyu that referenced this issue Oct 17, 2023
[type:test][ISSUE apache#4540] add Apache License for plugin test cas…
c671a84 
…es for admin
whenelse pushed a commit to whenelse/shenyu that referenced this issue Oct 17, 2023
[type:test][ISSUE apache#4540] add CheckStyle for plugin test cases f…
63718ac 
…or admin
whenelse pushed a commit to whenelse/shenyu that referenced this issue Oct 19, 2023
[type:test][ISSUE apache#4540] add batch plugin test cases for admin
c20eb2f
whenelse pushed a commit to whenelse/shenyu that referenced this issue Oct 19, 2023
[type:test][ISSUE apache#4540] add plugin handle test cases for admin
8414979
@whenelse
Copy link
Contributor

please assign all event tasks to me, thanks

@moremind
Copy link
Member Author

please assign all event tasks to me, thanks

assigned

moremind added a commit that referenced this issue Oct 24, 2023
@whenelse @moremind
[type:test][ISSUE #4540] add plugin test cases for admin (#5231)
648c415 
* [type:test][ISSUE #4540] add plugin test cases for admin

* [type:test][ISSUE #4540] add Apache License for plugin test cases for admin

* [type:test][ISSUE #4540] add CheckStyle for plugin test cases for admin

* [type:test][ISSUE #4540] add batch plugin test cases for admin

* [type:test][ISSUE #4540] add plugin handle test cases for admin

---------

Co-authored-by: Shibo Xia <xiashibo@rd.netease.com>
Co-authored-by: moremind <hefengen@apache.org>
@whenelse whenelse mentioned this issue Oct 31, 2023
Merged
3 tasks
moremind added a commit that referenced this issue Nov 3, 2023
@whenelse @moremind
[type:test][ISSUE #4540] add all event test cases for admin (#5263)
4e264b1 
* [type:test][ISSUE #4540] add plugin test cases for admin

* [type:test][ISSUE #4540] add Apache License for plugin test cases for admin

* [type:test][ISSUE #4540] add CheckStyle for plugin test cases for admin

* [type:test][ISSUE #4540] add batch plugin test cases for admin

* [type:test][ISSUE #4540] add plugin handle test cases for admin

* [type:test]add metadata test case for admin

* [type:refactor]refactor metadata comment and format

* [type:test]add dict test case for admin

* [type:test]add resource and user test case for admin

* [type:fix]resource test case checkstyle

---------

Co-authored-by: Shibo Xia <xiashibo@rd.netease.com>
Co-authored-by: moremind <hefengen@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hudongdong129 hudongdong129

@whenelse whenelse

@devinsong77 devinsong77

@yeshadoo yeshadoo

@zuobiao-zhou zuobiao-zhou

@sachin10fi sachin10fi

@lijay7674 lijay7674

Labels
good first issue Good for newcomers status: volunteer wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@moremind @hudongdong129 @whenelse @devinsong77 @yeshadoo @zuobiao-zhou @sachin10fi @lijay7674