[java-idp-integration-tests] 02/03: Log to debug rather than info

Tom Zeller tzeller at dragonacea.biz
Fri Aug 21 00:08:28 UTC 2020


This is an automated email from the git hooks/post-receive script.

tzeller pushed a commit to branch main
in repository java-idp-integration-tests.

View the commit online:
http://git.shibboleth.net/view/?p=java-idp-integration-tests.git;a=commit;h=8f04ac1052aa6f4a2db16546366d7b8a33a97543

commit 8f04ac1052aa6f4a2db16546366d7b8a33a97543
Author: Tom Zeller <tzeller at dragonacea.biz>
AuthorDate: Wed Aug 19 17:39:42 2020 -0500

    Log to debug rather than info
---
 .../net/shibboleth/idp/test/BaseIntegrationTest.java   | 18 ++++++++----------
 .../idp/test/CustomSauceOnDemandTestListener.java      |  2 +-
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/src/test/java/net/shibboleth/idp/test/BaseIntegrationTest.java b/src/test/java/net/shibboleth/idp/test/BaseIntegrationTest.java
index b54d7b6..774593c 100644
--- a/src/test/java/net/shibboleth/idp/test/BaseIntegrationTest.java
+++ b/src/test/java/net/shibboleth/idp/test/BaseIntegrationTest.java
@@ -77,7 +77,6 @@ import org.springframework.util.FileSystemUtils;
 import org.springframework.util.SocketUtils;
 import org.testng.Assert;
 import org.testng.ITestResult;
-import org.testng.Reporter;
 import org.testng.annotations.AfterClass;
 import org.testng.annotations.AfterMethod;
 import org.testng.annotations.BeforeClass;
@@ -395,7 +394,7 @@ public abstract class BaseIntegrationTest
         // Path to per-test idp.home
         final String timestamp = DateTimeFormatter.ofPattern(idpHomePattern).format(LocalDateTime.now());
         pathToIdPHome = pathToDistIdPHome.getParent().resolve(timestamp);
-        log.info("Path to idp.home '{}'", pathToIdPHome.toAbsolutePath());
+        log.debug("Path to idp.home '{}'", pathToIdPHome.toAbsolutePath());
         Assert.assertFalse(pathToIdPHome.toAbsolutePath().toFile().exists(), "Path to idp.home already exists");
 
         // Copy idp.home directory from distribution to new per test directory
@@ -634,16 +633,16 @@ public abstract class BaseIntegrationTest
         final Iterator<Integer> iterator = ports.iterator();
 
         port = iterator.next();
-        log.info("Selecting port '{}' for non-secure endpoints", port);
+        log.debug("Selecting port '{}' for non-secure endpoints", port);
 
         securePort = iterator.next();
-        log.info("Selecting port '{}' for secure endpoints", securePort);
+        log.debug("Selecting port '{}' for secure endpoints", securePort);
 
         backchannelPort = iterator.next();
-        log.info("Selecting port '{}' for backchannel endpoint", backchannelPort);
+        log.debug("Selecting port '{}' for backchannel endpoint", backchannelPort);
 
         ldapPort = iterator.next();
-        log.info("Selecting port '{}' for LDAP", ldapPort);
+        log.debug("Selecting port '{}' for LDAP", ldapPort);
         serverCommands.add("-D" + TEST_LDAP_PORT_PROPERTY + "=" + Integer.toString(ldapPort));
     }
 
@@ -657,14 +656,14 @@ public abstract class BaseIntegrationTest
         urlBuilder.setHost(address);
         urlBuilder.setPort(port);
         baseURL = urlBuilder.buildURL();
-        log.info("URL '{}' is the base URL which clients should connect to.", baseURL);
+        log.debug("URL '{}' is the base URL which clients should connect to.", baseURL);
 
         final URLBuilder secureUrlBuilder = new URLBuilder();
         secureUrlBuilder.setScheme("https");
         secureUrlBuilder.setHost(secureAddress);
         secureUrlBuilder.setPort(securePort);
         secureBaseURL = secureUrlBuilder.buildURL();
-        log.info("URL '{}' is the secure base URL which clients should connect to.", secureBaseURL);
+        log.debug("URL '{}' is the secure base URL which clients should connect to.", secureBaseURL);
     }
 
     /**
@@ -1344,7 +1343,6 @@ public abstract class BaseIntegrationTest
         }
 
         log.debug("Desired capabilities '{}'", desiredCapabilities);
-        Reporter.log("Desired capabilities '" + desiredCapabilities + "'", true);
     }
 
     /**
@@ -1423,7 +1421,7 @@ public abstract class BaseIntegrationTest
     public void setUpSauceLabsClientIPRange() {
         if (BaseIntegrationTest.isRemote()) {
             clientIPRange = SAUCE_LABS_IP_RANGE;
-            log.info("Setting client IP range to '{}'", clientIPRange);
+            log.debug("Setting client IP range to '{}'", clientIPRange);
         }
     }
 
diff --git a/src/test/java/net/shibboleth/idp/test/CustomSauceOnDemandTestListener.java b/src/test/java/net/shibboleth/idp/test/CustomSauceOnDemandTestListener.java
index 24eee79..3ab5354 100644
--- a/src/test/java/net/shibboleth/idp/test/CustomSauceOnDemandTestListener.java
+++ b/src/test/java/net/shibboleth/idp/test/CustomSauceOnDemandTestListener.java
@@ -54,7 +54,7 @@ public class CustomSauceOnDemandTestListener extends SauceOnDemandTestListener {
     @Override
     public void onStart(ITestContext testContext) {
         if (!BaseIntegrationTest.isRemote()) {
-            log.info("Setting system property '{}' to 'true'", BaseIntegrationTest.SELENIUM_IS_LOCAL);
+            log.debug("Setting system property '{}' to 'true'", BaseIntegrationTest.SELENIUM_IS_LOCAL);
             System.setProperty(BaseIntegrationTest.SELENIUM_IS_LOCAL, "true");
         }
         super.onStart(testContext);

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list