BE: RBAC: Implement RBAC tests

[zambrinf]

• Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)
Added tests to cober AccessControlService
Closes #729

Is there anything you'd like reviewers to focus on?

How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

• No need to
• Manually (please, describe, if necessary)
• Unit checks
• Integration checks
• Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
• My changes generate no new warnings (e.g. Sonar is happy)
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged

Check out Contributing and Code of Conduct

A picture of a cute animal (not mandatory but encouraged)

[github-actions]

Hi zambrinf! 👋

Welcome, and thank you for opening your first PR in the repo!

Please wait for triaging by our maintainers.

Please take a look at our contributing guide.

[Haarolean]

@zambrinf could we please add some tests to cover these cases?

• RBAC: Topic create permission issues #76
• There was an error updating the application config #461
• RBAC: Actions granted for cluster A are mistakenly available for cluster B #274
• perhaps RBAC: Can't edit topic while having inherited edit permissions #260 if you're willing to investigate this

If the tests fail due to faulty business logic, we can disable them until the issues are resolved.

[zambrinf]

@Haarolean Done. I added a test class RbacEditTopicTest specific to #260 but I couldn't reproduce a RBAC error. And the test for #461 is disabled because it's failing.

[zambrinf]

@Haarolean Are you planning on merging this PR?

[Haarolean]

@Haarolean Are you planning on merging this PR?

absolutely! Sorry for the delay, I'll be taking a look at this soon - it just takes more cognitive load than other issues/PRs :))

[Haarolean]

Thank you very much for your efforts and sorry for the delay. Left a few inline comments, PTAL.

[Haarolean]

@zambrinf thank you very much for your contribution! This is a quite important enhancement for us despite being "just tests", as we had a decent amount of issues with RBAC due to lack of the tests. Much appreciated!

[Haarolean]

@zambrinf
any idea how (and did it even) build for you?

/home/runner/work/kafka-ui/kafka-ui/api/src/test/java/io/kafbat/ui/service/rbac/RbacEditTopicTest.java:115: error: incompatible types: String cannot be converted to Provider
    sub.setProvider("ldap");

[zambrinf]

@Haarolean apparently my branch wasn't fully up to date with the main branch, there was a commit 3074abcdc9f760eb22e5f9056a4f634905ee2441 on March 20th that changed the setter method of the Subject class, saw that you already fixed it

[Haarolean]

@zambrinf ah I see, I've recently disabled 'branch should be up to date' merge check for some quick merges.