Spring/spring-boot-data-geode
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Refactor the dataReadNotAllowed() test case method to use JUnit 4 Exception handling (try-catch with expected Exception) rather than JUnit 5's (Jupiter) assertThrows(..) method.

Browse Source
This commit is contained in:
John Blum
2021-03-29 23:57:55 -07:00
parent ce971d004c
commit 59b91376b0
1 changed files with 16 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
spring-geode-samples/boot/security/src/test/java/example/app/security/BootGeodeSecurityClientApplicationIntegrationTests.java
View File

@@ -16,7 +16,6 @@
package example.app.security;
import static org.assertj.core.api.AssertionsForInterfaceTypes.assertThat;
import static org.junit.jupiter.api.Assertions.assertThrows;
import java.io.IOException;
@@ -77,20 +76,25 @@ public class BootGeodeSecurityClientApplicationIntegrationTests extends ForkingC
@SuppressWarnings("unused")
private GemfireTemplate customersTemplate;
@Test
@Test(expected = DataAccessResourceFailureException.class)
public void dataReadNotAllowed() {
Exception exception = assertThrows(DataAccessResourceFailureException.class,
() -> this.customersTemplate.get(2L));
try {
this.customersTemplate.get(2L);
}
catch (DataAccessResourceFailureException expected) {
assertThat(exception).hasCauseInstanceOf(ServerOperationException.class);
assertThat(exception.getCause()).hasMessageContaining("remote server");
assertThat(exception.getCause()).hasCauseInstanceOf(NotAuthorizedException.class);
assertThat(exception.getCause().getCause()).hasMessageContaining("jdoe not authorized for DATA:READ");
assertThat(exception.getCause().getCause()).hasCauseInstanceOf(UnauthorizedException.class);
assertThat(exception.getCause().getCause().getCause())
.hasMessageContaining("Subject does not have permission [DATA:READ");
assertThat(exception.getCause().getCause().getCause()).hasNoCause();
assertThat(expected).hasCauseInstanceOf(ServerOperationException.class);
assertThat(expected.getCause()).hasMessageContaining("remote server");
assertThat(expected.getCause()).hasCauseInstanceOf(NotAuthorizedException.class);
assertThat(expected.getCause().getCause()).hasMessageContaining("jdoe not authorized for DATA:READ");
assertThat(expected.getCause().getCause()).hasCauseInstanceOf(UnauthorizedException.class);
assertThat(expected.getCause().getCause().getCause())
.hasMessageContaining("Subject does not have permission [DATA:READ");
assertThat(expected.getCause().getCause().getCause()).hasNoCause();
throw expected;
}
}
@Test