From 23518bf0f1bb37b97e0e75f35f66412e20a80699 Mon Sep 17 00:00:00 2001 From: John Blum Date: Tue, 6 Jun 2017 20:05:25 -0700 Subject: [PATCH] Polish and source code formatting. --- ...sionGemFireCustomIndexingIntegrationTests.java | 13 ++++++++----- ...ttpSessionGemFireIndexingIntegrationTests.java | 15 ++++++++++----- 2 files changed, 18 insertions(+), 10 deletions(-) diff --git a/docs/src/test/java/docs/gemfire/indexing/HttpSessionGemFireCustomIndexingIntegrationTests.java b/docs/src/test/java/docs/gemfire/indexing/HttpSessionGemFireCustomIndexingIntegrationTests.java index 031a5bb..15266e5 100644 --- a/docs/src/test/java/docs/gemfire/indexing/HttpSessionGemFireCustomIndexingIntegrationTests.java +++ b/docs/src/test/java/docs/gemfire/indexing/HttpSessionGemFireCustomIndexingIntegrationTests.java @@ -29,13 +29,14 @@ import org.springframework.session.ExpiringSession; import org.springframework.session.data.gemfire.GemFireOperationsSessionRepository; import org.springframework.session.data.gemfire.config.annotation.web.http.EnableGemFireHttpSession; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; +import org.springframework.test.context.junit4.SpringRunner; /** * @author Rob Winch * @author John Blum */ -@RunWith(SpringJUnit4ClassRunner.class) +@SuppressWarnings("unused") +@RunWith(SpringRunner.class) @ContextConfiguration(classes = HttpSessionGemFireCustomIndexingIntegrationTests.TestConfiguration.class) public class HttpSessionGemFireCustomIndexingIntegrationTests { @@ -44,19 +45,21 @@ public class HttpSessionGemFireCustomIndexingIntegrationTests { @Test public void findByIndexName() { + ExpiringSession session = this.sessionRepository.createSession(); - String attrValue = "HttpSessionGemFireCustomIndexingIntegrationTests-findByIndexName"; + String indexValue = "HttpSessionGemFireCustomIndexingIntegrationTests-findByIndexName"; // tag::findbyindexname-set[] String indexName = "name1"; - session.setAttribute(indexName, attrValue); + + session.setAttribute(indexName, indexValue); // end::findbyindexname-set[] this.sessionRepository.save(session); // tag::findbyindexname-get[] Map idToSessions = - this.sessionRepository.findByIndexNameAndIndexValue(indexName, attrValue); + this.sessionRepository.findByIndexNameAndIndexValue(indexName, indexValue); // end::findbyindexname-get[] assertThat(idToSessions.keySet()).containsOnly(session.getId()); diff --git a/docs/src/test/java/docs/gemfire/indexing/HttpSessionGemFireIndexingIntegrationTests.java b/docs/src/test/java/docs/gemfire/indexing/HttpSessionGemFireIndexingIntegrationTests.java index 67a728e..b4fbe25 100644 --- a/docs/src/test/java/docs/gemfire/indexing/HttpSessionGemFireIndexingIntegrationTests.java +++ b/docs/src/test/java/docs/gemfire/indexing/HttpSessionGemFireIndexingIntegrationTests.java @@ -41,8 +41,9 @@ import org.springframework.test.context.junit4.SpringRunner; * @author Rob Winch * @author John Blum */ +@SuppressWarnings("unused") @RunWith(SpringRunner.class) -@ContextConfiguration(classes = HttpSessionGemFireIndexingIntegrationTests.Config.class) +@ContextConfiguration(classes = HttpSessionGemFireIndexingIntegrationTests.TestConfiguration.class) public class HttpSessionGemFireIndexingIntegrationTests { @Autowired @@ -50,6 +51,7 @@ public class HttpSessionGemFireIndexingIntegrationTests { @Test public void findByIndexName() { + ExpiringSession session = this.sessionRepository.createSession(); String username = "HttpSessionGemFireIndexingIntegrationTests-findByIndexName-username"; @@ -74,18 +76,21 @@ public class HttpSessionGemFireIndexingIntegrationTests { @Test @WithMockUser("HttpSessionGemFireIndexingIntegrationTests-findBySpringSecurityIndexName") public void findBySpringSecurityIndexName() { + ExpiringSession session = this.sessionRepository.createSession(); // tag::findbyspringsecurityindexname-context[] - SecurityContext context = SecurityContextHolder.getContext(); - Authentication authentication = context.getAuthentication(); + SecurityContext securityContext = SecurityContextHolder.getContext(); + Authentication authentication = securityContext.getAuthentication(); // end::findbyspringsecurityindexname-context[] - session.setAttribute(HttpSessionSecurityContextRepository.SPRING_SECURITY_CONTEXT_KEY, context); + session.setAttribute(HttpSessionSecurityContextRepository.SPRING_SECURITY_CONTEXT_KEY, securityContext); + this.sessionRepository.save(session); // tag::findbyspringsecurityindexname-get[] String indexName = FindByIndexNameSessionRepository.PRINCIPAL_NAME_INDEX_NAME; + Map idToSessions = this.sessionRepository.findByIndexNameAndIndexValue(indexName, authentication.getName()); // end::findbyspringsecurityindexname-get[] @@ -97,6 +102,6 @@ public class HttpSessionGemFireIndexingIntegrationTests { @PeerCacheApplication(name = "HttpSessionGemFireIndexingIntegrationTests", logLevel = "error") @EnableGemFireHttpSession(regionName = "HttpSessionGemFireIndexingTestRegion") - static class Config { + static class TestConfiguration { } }