A bit of test cleanup

This commit is contained in:
jamesagnew 2018-03-23 07:15:22 -04:00
parent d5a1c089b8
commit e5903f79f0
12 changed files with 9 additions and 11 deletions

View File

@ -18,7 +18,7 @@ public class PublicSecurityInterceptor extends AuthorizationInterceptor {
public PublicSecurityInterceptor() { public PublicSecurityInterceptor() {
String passwordsString = System.getProperty("fhir.tdlpass"); String passwordsString = System.getProperty("fhir.tdlpass");
String[] passwords = passwordsString.split(","); String[] passwords = passwordsString.split(",");
myTokens = new HashSet<String>(Arrays.asList(passwords)); myTokens = new HashSet<>(Arrays.asList(passwords));
ourLog.info("We have {} valid security tokens", myTokens.size()); ourLog.info("We have {} valid security tokens", myTokens.size());
} }
@ -27,7 +27,6 @@ public class PublicSecurityInterceptor extends AuthorizationInterceptor {
public List<IAuthRule> buildRuleList(RequestDetails theRequestDetails) { public List<IAuthRule> buildRuleList(RequestDetails theRequestDetails) {
String authHeader = theRequestDetails.getHeader("Authorization"); String authHeader = theRequestDetails.getHeader("Authorization");
//@formatter:off
if (isBlank(authHeader)) { if (isBlank(authHeader)) {
return new RuleBuilder() return new RuleBuilder()
.deny().operation().named(BaseJpaSystemProvider.MARK_ALL_RESOURCES_FOR_REINDEXING).onServer().andThen() .deny().operation().named(BaseJpaSystemProvider.MARK_ALL_RESOURCES_FOR_REINDEXING).onServer().andThen()
@ -35,7 +34,6 @@ public class PublicSecurityInterceptor extends AuthorizationInterceptor {
.allowAll() .allowAll()
.build(); .build();
} }
//@formatter:off
if (!authHeader.startsWith("Bearer ")) { if (!authHeader.startsWith("Bearer ")) {
throw new ForbiddenOperationException("Invalid bearer token, must be in the form \"Authorization: Bearer [token]\""); throw new ForbiddenOperationException("Invalid bearer token, must be in the form \"Authorization: Bearer [token]\"");

View File

@ -1954,8 +1954,6 @@
<module>hapi-deployable-pom</module> <module>hapi-deployable-pom</module>
<module>hapi-fhir-base</module> <module>hapi-fhir-base</module>
<module>hapi-fhir-utilities</module> <module>hapi-fhir-utilities</module>
<module>hapi-fhir-base-test-mindeps-client</module>
<module>hapi-fhir-base-test-mindeps-server</module>
<module>hapi-tinder-plugin</module> <module>hapi-tinder-plugin</module>
<module>hapi-tinder-test</module> <module>hapi-tinder-test</module>
<module>hapi-fhir-client</module> <module>hapi-fhir-client</module>
@ -1990,6 +1988,8 @@
<module>example-projects/hapi-fhir-standalone-overlay-example</module> <module>example-projects/hapi-fhir-standalone-overlay-example</module>
<module>example-projects/hapi-fhir-jpaserver-cds-example</module> <module>example-projects/hapi-fhir-jpaserver-cds-example</module>
<module>example-projects/hapi-fhir-jpaserver-dynamic</module> <module>example-projects/hapi-fhir-jpaserver-dynamic</module>
<module>tests/hapi-fhir-base-test-mindeps-client</module>
<module>tests/hapi-fhir-base-test-mindeps-server</module>
<module>hapi-fhir-jacoco</module> <module>hapi-fhir-jacoco</module>
<module>hapi-fhir-igpacks</module> <module>hapi-fhir-igpacks</module>
<module>hapi-fhir-spring-boot</module> <module>hapi-fhir-spring-boot</module>

View File

@ -5,10 +5,10 @@
<groupId>ca.uhn.hapi.fhir</groupId> <groupId>ca.uhn.hapi.fhir</groupId>
<artifactId>hapi-fhir</artifactId> <artifactId>hapi-fhir</artifactId>
<version>3.3.0-SNAPSHOT</version> <version>3.3.0-SNAPSHOT</version>
<relativePath>../pom.xml</relativePath> <relativePath>../../pom.xml</relativePath>
</parent> </parent>
<artifactId>hapi-fhir-base-testmindeps-client</artifactId> <artifactId>hapi-fhir-base-test-mindeps-client</artifactId>
<packaging>jar</packaging> <packaging>jar</packaging>
<name>HAPI FHIR - Minimal Dependency Test - Client</name> <name>HAPI FHIR - Minimal Dependency Test - Client</name>

View File

@ -1,4 +1,4 @@
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" <project xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://maven.apache.org/POM/4.0.0"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion> <modelVersion>4.0.0</modelVersion>
@ -6,10 +6,10 @@
<groupId>ca.uhn.hapi.fhir</groupId> <groupId>ca.uhn.hapi.fhir</groupId>
<artifactId>hapi-fhir</artifactId> <artifactId>hapi-fhir</artifactId>
<version>3.3.0-SNAPSHOT</version> <version>3.3.0-SNAPSHOT</version>
<relativePath>../pom.xml</relativePath> <relativePath>../../pom.xml</relativePath>
</parent> </parent>
<artifactId>hapi-fhir-base-testmindeps-server</artifactId> <artifactId>hapi-fhir-base-test-mindeps-server</artifactId>
<packaging>jar</packaging> <packaging>jar</packaging>
<name>HAPI FHIR - Minimal Dependency Test - Server</name> <name>HAPI FHIR - Minimal Dependency Test - Server</name>