You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mi...@apache.org on 2022/01/15 20:50:54 UTC
[maven-site-plugin] branch master updated: [MSITE-880] AbstractSiteDeployWebDavTest should not log
This is an automated email from the ASF dual-hosted git repository.
michaelo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-site-plugin.git
The following commit(s) were added to refs/heads/master by this push:
new db0ed41 [MSITE-880] AbstractSiteDeployWebDavTest should not log
db0ed41 is described below
commit db0ed41b00f17f3756338e2516512254caf8cacd
Author: Michael Osipov <mi...@apache.org>
AuthorDate: Sat Jan 15 21:50:08 2022 +0100
[MSITE-880] AbstractSiteDeployWebDavTest should not log
---
.../site/deploy/AbstractSiteDeployWebDavTest.java | 24 +++++-----------------
1 file changed, 5 insertions(+), 19 deletions(-)
diff --git a/src/test/java/org/apache/maven/plugins/site/deploy/AbstractSiteDeployWebDavTest.java b/src/test/java/org/apache/maven/plugins/site/deploy/AbstractSiteDeployWebDavTest.java
index 1af75b1..7e22aa2 100644
--- a/src/test/java/org/apache/maven/plugins/site/deploy/AbstractSiteDeployWebDavTest.java
+++ b/src/test/java/org/apache/maven/plugins/site/deploy/AbstractSiteDeployWebDavTest.java
@@ -43,8 +43,6 @@ import org.junit.Before;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.junit.runners.JUnit4;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
/**
* @author Olivier Lamy
@@ -57,8 +55,6 @@ public abstract class AbstractSiteDeployWebDavTest
File siteTargetPath = new File( getBasedir() + File.separator + "target" + File.separator + "siteTargetDeploy" );
- private Logger log = LoggerFactory.getLogger( getClass() );
-
@Before
public void setUp()
throws Exception
@@ -158,18 +154,12 @@ public abstract class AbstractSiteDeployWebDavTest
assertContentInFiles();
assertTrue( requestsContainsProxyUse( simpleDavServerHandler.httpRequests ) );
-
- for ( HttpRequest rq : simpleDavServerHandler.httpRequests )
- {
- log.info( rq.toString() );
- }
-
}
finally
{
simpleDavServerHandler.stop();
- }
-
+ }
+
}
@Test
@@ -184,7 +174,7 @@ public abstract class AbstractSiteDeployWebDavTest
AuthAsyncProxyServlet servlet = new AuthAsyncProxyServlet( authentications, siteTargetPath );
- SimpleDavServerHandler simpleDavServerHandler = new SimpleDavServerHandler( servlet );
+ SimpleDavServerHandler simpleDavServerHandler = new SimpleDavServerHandler( servlet );
try
{
File pluginXmlFile = getTestFile( "src/test/resources/unit/deploy-dav/pom.xml" );
@@ -242,16 +232,12 @@ public abstract class AbstractSiteDeployWebDavTest
assertContentInFiles();
assertTrue( requestsContainsProxyUse( servlet.httpRequests ) );
assertAtLeastOneRequestContainsHeader( servlet.httpRequests, "Proxy-Authorization" );
- for ( HttpRequest rq : servlet.httpRequests )
- {
- log.info( rq.toString() );
- }
}
finally
{
simpleDavServerHandler.stop();
- }
- }
+ }
+ }
private void assertContentInFiles()
throws Exception