From: Aron Roberts <aron@socrates.berkeley.edu>
Date: Sat, 30 Jan 2010 01:29:50 +0000 (+0000)
Subject: CSPACE-685: ServiceLayerTest now uses a BaseServiceClient-derived client, to facilita... 
X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=7824fb1ca248acc578b6814dfb5d514ba80cde1d;p=tmp%2Fjakarta-migration.git

CSPACE-685: ServiceLayerTest now uses a BaseServiceClient-derived client, to facilitate testing with authentication enabled.
---

diff --git a/services/client/src/test/java/org/collectionspace/services/client/test/ServiceLayerTest.java b/services/client/src/test/java/org/collectionspace/services/client/test/ServiceLayerTest.java
index 458cb0df3..61fcbe99a 100644
--- a/services/client/src/test/java/org/collectionspace/services/client/test/ServiceLayerTest.java
+++ b/services/client/src/test/java/org/collectionspace/services/client/test/ServiceLayerTest.java
@@ -33,9 +33,11 @@ import org.apache.commons.httpclient.methods.HeadMethod;
 import org.apache.commons.httpclient.methods.OptionsMethod;
 import org.apache.commons.httpclient.methods.TraceMethod;
 import org.collectionspace.services.client.TestServiceClient;
+
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.testng.Assert;
+import org.testng.annotations.BeforeSuite;
 import org.testng.annotations.Test;
 
 /**
@@ -48,8 +50,25 @@ import org.testng.annotations.Test;
 public class ServiceLayerTest {
 
     final Logger logger = LoggerFactory.getLogger(ServiceTest.class);
-    private HttpClient httpClient = new HttpClient();
     private TestServiceClient serviceClient = new TestServiceClient();
+    private HttpClient httpClient = serviceClient.getHttpClient();
+
+    @BeforeSuite void printServiceClientProperties() {
+        if(logger.isDebugEnabled()){
+            logger.debug("Client properties read from the properties path;\n"
+                +"possibly from the command line or a properties file:");
+            logger.debug("url = " +
+                serviceClient.getProperty(serviceClient.URL_PROPERTY));
+            logger.debug("secure (SSL) = " +
+                serviceClient.getProperty(serviceClient.SSL_PROPERTY));
+            logger.debug("useAuth = " +
+                serviceClient.getProperty(serviceClient.AUTH_PROPERTY));
+            logger.debug("user = " +
+                serviceClient.getProperty(serviceClient.USER_PROPERTY));
+            logger.debug("password = " +
+                serviceClient.getProperty(serviceClient.PASSWORD_PROPERTY));
+        }
+    }
 
     @Test
     public void servicesExist() {
@@ -57,6 +76,7 @@ public class ServiceLayerTest {
         String url = serviceClient.getBaseURL() + "idgenerators";
         OptionsMethod method = new OptionsMethod(url);
         try{
+            serviceClient = new TestServiceClient();
             int statusCode = httpClient.executeMethod(method);
             if(logger.isDebugEnabled()){
                 logger.debug("servicesExist url=" + url + " status=" + statusCode);
@@ -77,7 +97,7 @@ public class ServiceLayerTest {
 
     @Test
     public void methodNotAllowed() {
-        //delete is not allowed on the root URL of the id service
+        // Delete is not allowed on the root URL of the id service
         String url = serviceClient.getBaseURL() + "idgenerators";
         DeleteMethod method = new DeleteMethod(url);
         try{
@@ -100,14 +120,13 @@ public class ServiceLayerTest {
     }
 
     @Test
-    public void noService() {
-
-        String url = serviceClient.getBaseURL() + "fake-service";
+    public void nonexistentService() {
+        String url = serviceClient.getBaseURL() + "nonexistent-service";
         GetMethod method = new GetMethod(url);
         try{
             int statusCode = httpClient.executeMethod(method);
             if(logger.isDebugEnabled()){
-                logger.debug("noService url=" + url + " status=" + statusCode);
+                logger.debug("nonexistentService url=" + url + " status=" + statusCode);
             }
             Assert.assertEquals(statusCode, HttpStatus.SC_NOT_FOUND,
                     "expected " + HttpStatus.SC_NOT_FOUND);
@@ -131,8 +150,11 @@ public class ServiceLayerTest {
         }
         String url = serviceClient.getBaseURL() + "collectionobjects";
         GetMethod method = new GetMethod(url);
+        // This vanilla HTTP client does not contain credentials or any other
+        // properties of the serviceClient.
+        HttpClient noCredentialsHttpClient = new HttpClient();
         try{
-            int statusCode = httpClient.executeMethod(method);
+            int statusCode = noCredentialsHttpClient.executeMethod(method);
             if(logger.isDebugEnabled()){
                 logger.debug("serviceSecure url=" + url + " status=" + statusCode);
             }