-
Notifications
You must be signed in to change notification settings - Fork 130
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
Winduprule 363 and 142 spring remote services #1340
base: master
Are you sure you want to change the base?
Winduprule 363 and 142 spring remote services #1340
Conversation
…orter bean and the Class from the implementation bean
created Arquillian test for Spring RMI rule
Added a POC of including info in a different section ( JAX-WS )
…operly work Added all non RMI services to JaxWS services while waiting the category to be put
…rvice Reports and TechnologyIdentified to show on Technology Usage report
…eport template ) Added new Model and Service to cover Spring Remote Services Modified the creation of the Remote Services report to consider new Model
…-on-services-report
…duplicated code Created rule to discover Spring Rest services
…bean of a remote service in any XML file
…o compile because of the dependent projects
I haven't been able to run the Test classes because of problems with my local Arquillian configuration but I'm happy with the code. If someone else can certify the tests then great. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests run successfully
set always the implementation class from outside, or find it as it was before
…s considering the new bean definition
@mrizzi I've reviewed and tested the latest changes successfully so this is ready to merge as far as I'm concerned |
…nceModel inject returnType on the ClassReference on the analyze process use returnType on the DiscoverySpringBeanMethodAnnotationsRuleProvider
…, otherwise are excluded from the analysis
changed test files to packages out of the regular ones added annotation to the interested types
Can one of the admins verify this patch? |
PR With the Spring Remote Services features :
Also contains Spring REST Services detection on Java classes
To allow this it was needed to create rules to detect Spring Beans on Java classes and being added to the SpringBeanService
Jira tickets covered :
WINDUPRULE-363
WINDUPRULE-142