-
Notifications
You must be signed in to change notification settings - Fork 88
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
PermissionCheckTest not working without Use-Cases #156
Comments
Was initially from devonfw/my-thai-star#113 |
As nobody catched up, I have to move to the next release. :( |
If still unclear the issue is about this test: When using Component-Facade without Use-Cases: |
BTW: We could have a discussion with the Spanish collegues who suggested this "Component Facade without Use-Cases" approach in devon4j. An alternative for simplication is still "Use-Cases without Component Facade" that makes IMHO more sense. Further I am unsure if there are projects left that actually use "Component Facade without Use-Cases". If not used anywhere we can also close this issue as wontfix. |
As a devon4j developer/architect I have the option to omit use-cases and only have component facades with implementation, I want
PermissionCheckTest
to still work in that case so that security is still verified.See
https://github.com/devonfw/devon4j/blob/master/documentation/guide-component-facade.asciidoc#implementation
vs.
https://github.com/devonfw/devon4j/blob/master/documentation/guide-usecase.asciidoc#usecase
We can also create a rule in sonar-devon(4j)-plugin that does this check and remove the Junit implemnation (also ArchUnit was in discussion):
https://github.com/devonfw/sonar-devon4j-plugin/#sonar-devon4j-plugin
The text was updated successfully, but these errors were encountered: