= Adds a test that demonstrates how to integrate tracee-springhttpclient by danielwegener · Pull Request #126 · tracee/tracee · GitHub
Skip to content
This repository was archived by the owner on Feb 4, 2024. It is now read-only.

= Adds a test that demonstrates how to integrate tracee-springhttpclient#126

Merged
SvenBunge merged 1 commit into
tracee:masterfrom
danielwegener:springhttpclient-xmlcontext-test
Jan 12, 2016
Merged

= Adds a test that demonstrates how to integrate tracee-springhttpclient#126
SvenBunge merged 1 commit into
tracee:masterfrom
danielwegener:springhttpclient-xmlcontext-test

Conversation

@danielwegener

Copy link
Copy Markdown
Member

…with xml configuration

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

merge="true" is not needed. As far as I know it's only used in conjunction with parent="parentList".

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Does it not merge this list <list> with the list of already existing interceptors on the RestTemplate? If now? how would that be possible?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hmm, as far as I know a property simple calls the setXX(..)-Method and if you use merge it is a bit tricky: you've to add a parent to the bean definition that points to the other definition. See: http://javarticles.com/2015/05/spring-collection-merging-example.html

@SvenBunge

Copy link
Copy Markdown
Member

At all: looks good. 👍

@danielwegener danielwegener force-pushed the springhttpclient-xmlcontext-test branch 2 times, most recently from 80358bf to 63e1e02 Compare January 6, 2016 17:36
@danielwegener

Copy link
Copy Markdown
Member Author

Fixed & rebased

@danielwegener danielwegener force-pushed the springhttpclient-xmlcontext-test branch from 63e1e02 to 96e7291 Compare January 6, 2016 17:38
@SvenBunge

Copy link
Copy Markdown
Member

SvenBunge added a commit that referenced this pull request Jan 12, 2016
…t-test

Adds a test that demonstrates how to integrate tracee-springhttpclient
@SvenBunge SvenBunge merged commit 7cfebd2 into tracee:master Jan 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants