Skip to content

Expose attributes in MongoHttpSessionConfiguration #492

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

Closed
wants to merge 1 commit into from

Conversation

eddumelendez
Copy link
Contributor

  • I have signed the CLA

Attributes collectionName and maxInactiveIntervalInSeconds can be set
now.

See gh-489

* @since 1.2.0
*/
@ContextConfiguration
public class MongoHttpSessionConfigurationITests extends AbstractMongoRepositoryITests {
Copy link
Contributor

Choose a reason for hiding this comment

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

This shouldn't be an integration test.

@eddumelendez eddumelendez force-pushed the gh-489 branch 3 times, most recently from fa517a9 to 7d39eb7 Compare April 18, 2016 21:40
@rwinch rwinch added the status: waiting-for-feedback We need additional information before we can continue label Apr 19, 2016
@eddumelendez
Copy link
Contributor Author

@vpavic thanks for feedback.

@rwinch I have updated the PR.

@snicoll
Copy link
Member

snicoll commented Apr 19, 2016

LGTM

public class MongoHttpSessionConfigurationTest {

private AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext();

Copy link
Member

Choose a reason for hiding this comment

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

Should have an @After that closes the context

Attributes collectionName and maxInactiveIntervalInSeconds can be set
now.

See spring-projectsgh-489
@eddumelendez
Copy link
Contributor Author

@rwinch fixed

@rwinch
Copy link
Member

rwinch commented Apr 20, 2016

Thanks for the PR! This is merged via edbf8bf

@rwinch rwinch closed this Apr 20, 2016
@rwinch rwinch self-assigned this Apr 20, 2016
@rwinch rwinch added status: duplicate A duplicate of another issue type: enhancement A general enhancement and removed status: waiting-for-feedback We need additional information before we can continue labels Apr 20, 2016
@rwinch rwinch added this to the 1.2.0.RC3 milestone Apr 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: duplicate A duplicate of another issue type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants