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.

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

Open
wants to merge 85 commits into
base: master
Choose a base branch
from

Conversation

jonathanvila
Copy link
Member

PR With the Spring Remote Services features :

  • detected on XML files
  • detected on Java annotated classes ( methods and classes )

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

jonathan added 30 commits September 27, 2018 11:31
…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
…duplicated code

Created rule to discover Spring Rest services
…o compile because of the dependent projects
@m-brophy
Copy link
Contributor

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.

m-brophy
m-brophy previously approved these changes Nov 30, 2018
Copy link
Contributor

@m-brophy m-brophy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests run successfully

@m-brophy
Copy link
Contributor

@mrizzi I've reviewed and tested the latest changes successfully so this is ready to merge as far as I'm concerned

jonathan added 19 commits January 21, 2019 10:51
…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
@PhilipCattanach PhilipCattanach added On Hold Further checks required before merging with Master and removed Ready to Review labels Apr 18, 2019
@windupgithubbot1
Copy link

Can one of the admins verify this patch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Hold Further checks required before merging with Master
Projects
None yet
4 participants