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

[Enhancement] Add some test cases for org.apache.rocketmq.acl #8519

Closed
1 task done
HardX8 opened this issue Aug 11, 2024 · 1 comment · Fixed by #8508
Closed
1 task done

[Enhancement] Add some test cases for org.apache.rocketmq.acl #8519

HardX8 opened this issue Aug 11, 2024 · 1 comment · Fixed by #8508
Labels
soc Summer of Code, hosted by Google, Alibaba, Chinese Academy of Sciences and so on type/enhancement

Comments

@HardX8
Copy link
Contributor

HardX8 commented Aug 11, 2024

Before Creating the Enhancement Request

  • I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary

Add test for:

org.apache.rocketmq.acl.common.AuthorizationHeader
org.apache.rocketmq.acl.plain.PlainPermissionChecker

Motivation

for issue #8262 , add test cases for acl module

Describe the Solution You'd Like

This modification is assisted by the Tongyi Lingcode
356165609-c7933630-2385-4d0f-91bb-2ed6623c1224

Describe Alternatives You've Considered

NULL

Additional Context

The test cases have all passed
356165448-e766c43b-9125-4c23-84a3-728c7e059217
356165457-8ff513be-970a-4437-b217-03816e0cff0c
@HardX8
Copy link
Contributor Author

HardX8 commented Aug 11, 2024

link #8262

@ShannonDing ShannonDing added the soc Summer of Code, hosted by Google, Alibaba, Chinese Academy of Sciences and so on label Aug 12, 2024
@ShannonDing ShannonDing mentioned this issue Aug 12, 2024
Merged
ShannonDing pushed a commit that referenced this issue Aug 16, 2024
@HardX8
[ISSUE #8519] Add test case for rocketmq acl module. (#8508)
9059fcd 
* add AuthorizationHeaderTest

* add PlainPermissionCheckerTest

* add license header for AuthorizationHeaderTest and PlainPermissionCheckerTest

* Update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
soc Summer of Code, hosted by Google, Alibaba, Chinese Academy of Sciences and so on type/enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE #8519] Add test case for rocketmq acl module. HardX8/rocketmq
2 participants
@ShannonDing @HardX8