Adding build property to indicate whether auth is enabled, and adjust…

…ing expected response codes in IT based on that property
fcrepo-exts · Apr 29, 2015 · 8376a7a · 8376a7a
1 parent bd8b713
commit 8376a7a
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -35,7 +35,7 @@ mvn install -P xacml
 This profile builds webapp that includes the [fcrepo-audit](https://github.com/fcrepo4-labs/fcrepo-audit) module that provides internal auditing capability.
 
 ```
-mvn install -P audit -DskipTests=true
+mvn install -P audit
 ```
 
 

diff --git a/pom.xml b/pom.xml
@@ -220,6 +220,10 @@
                   <name>fcrepo.xacml.initial.root.policy.file</name>
                   <value>${project.build.directory}/policies/GlobalRolesPolicySet.xml</value>
                 </systemProperty>
+                <systemProperty>
+                  <name>auth.enabled</name>
+                  <value>${auth.enabled:false}</value>
+                </systemProperty>
 
               </systemProperties>
 
@@ -237,6 +241,20 @@
           </execution>
         </executions>
       </plugin>
+      <plugin>
+        <artifactId>maven-failsafe-plugin</artifactId>
+        <configuration>
+          <argLine>-XX:MaxPermSize=128m ${jacoco.agent.it.arg} -Dauth.enabled=${auth.enabled}</argLine>
+        </configuration>
+        <executions>
+          <execution>
+            <id>default-integration-test</id>
+            <goals>
+              <goal>integration-test</goal>
+            </goals>
+          </execution>
+        </executions>
+      </plugin>
     </plugins>
 
   </build>
@@ -273,6 +291,7 @@
       <id>xacml</id>
       <properties>
         <profile.src.dir>${project.basedir}/src/xacml</profile.src.dir>
+        <auth.enabled>true</auth.enabled>
       </properties>
       <dependencies>
         <dependency>
@@ -345,6 +364,7 @@
       <id>rbacl</id>
       <properties>
         <profile.src.dir>${project.basedir}/src/rbacl</profile.src.dir>
+        <auth.enabled>true</auth.enabled>
       </properties>
       <activation>
         <activeByDefault>true</activeByDefault>

diff --git a/src/test/java/org/fcrepo/integration/SanityIT.java b/src/test/java/org/fcrepo/integration/SanityIT.java
@@ -50,10 +50,18 @@ public class SanityIT {
     private static final String CONTEXT_PATH = System.getProperty("fcrepo.test.context.path");
 
     protected Logger logger;
+    private static int noAuthExpectedResponse;
 
     @Before
     public void setLogger() {
         logger = LoggerFactory.getLogger(this.getClass());
+
+        logger.debug("auth.enabled: {}", System.getProperty("auth.enabled"));
+        if ("true".equals(System.getProperty("auth.enabled"))) {
+            noAuthExpectedResponse = 401;
+        } else {
+            noAuthExpectedResponse = 200;
+        }
     }
 
     protected static final String HOSTNAME = "localhost";
@@ -83,7 +91,7 @@ public void doASanityCheck() throws IOException {
     @Test
     public void doASanityCheckNoAuth() throws IOException {
         final HttpGet get = new HttpGet(serverAddress + "rest/");
-        assertEquals(401, getStatus(get));
+        assertEquals(noAuthExpectedResponse, getStatus(get));
     }
 
     protected int getStatus(final HttpUriRequest method) throws IOException {