[MDEP-923] Remove plexus logger from DependencySilentLog

apache · Apr 24, 2024 · 20a12ab · 20a12ab
1 parent fda1184
commit 20a12ab
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 175 deletions.
diff --git a/src/main/java/org/apache/maven/plugins/dependency/utils/DependencySilentLog.java b/src/main/java/org/apache/maven/plugins/dependency/utils/DependencySilentLog.java
@@ -19,14 +19,13 @@
 package org.apache.maven.plugins.dependency.utils;
 
 import org.apache.maven.plugin.logging.Log;
-import org.codehaus.plexus.logging.Logger;
 
 /**
  * This logger implements both types of logs currently in use and turns off logs.
  *
  * @author <a href="mailto:brianf@apache.org">Brian Fox</a>
  */
-public class DependencySilentLog implements Log, Logger {
+public class DependencySilentLog implements Log {
     /**
      * @return <code>false</code>
      * @see org.apache.maven.plugin.logging.Log#isDebugEnabled()
@@ -184,148 +183,4 @@ public void error(CharSequence content, Throwable error) {
     public void error(Throwable error) {
         // nop
     }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#debug(java.lang.String)
-     */
-    @Override
-    public void debug(String message) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#debug(java.lang.String, java.lang.Throwable)
-     */
-    @Override
-    public void debug(String message, Throwable throwable) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#info(java.lang.String)
-     */
-    @Override
-    public void info(String message) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#info(java.lang.String, java.lang.Throwable)
-     */
-    @Override
-    public void info(String message, Throwable throwable) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#warn(java.lang.String)
-     */
-    @Override
-    public void warn(String message) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#warn(java.lang.String, java.lang.Throwable)
-     */
-    @Override
-    public void warn(String message, Throwable throwable) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#error(java.lang.String)
-     */
-    @Override
-    public void error(String message) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#error(java.lang.String, java.lang.Throwable)
-     */
-    @Override
-    public void error(String message, Throwable throwable) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#fatalError(java.lang.String)
-     */
-    @Override
-    public void fatalError(String message) {
-        // nop
-    }
-
-    /**
-     * By default, do nothing.
-     *
-     * @see org.codehaus.plexus.logging.Logger#fatalError(java.lang.String, java.lang.Throwable)
-     */
-    @Override
-    public void fatalError(String message, Throwable throwable) {
-        // nop
-    }
-
-    /**
-     * @return <code>false</code>
-     * @see org.codehaus.plexus.logging.Logger#isFatalErrorEnabled()
-     */
-    @Override
-    public boolean isFatalErrorEnabled() {
-        return false;
-    }
-
-    /**
-     * @return <code>null</code>
-     * @see org.codehaus.plexus.logging.Logger#getChildLogger(java.lang.String)
-     */
-    @Override
-    public Logger getChildLogger(String name) {
-        return null;
-    }
-
-    /**
-     * @return <code>0</code>
-     * @see org.codehaus.plexus.logging.Logger#getThreshold()
-     */
-    @Override
-    public int getThreshold() {
-        return 0;
-    }
-
-    /**
-     * By default, do nothing
-     */
-    @Override
-    public void setThreshold(int threshold) {
-        // nop
-    }
-
-    /**
-     * @return <code>null</code>
-     * @see org.codehaus.plexus.logging.Logger#getName()
-     */
-    @Override
-    public String getName() {
-        return null;
-    }
 }
diff --git a/src/test/java/org/apache/maven/plugins/dependency/utils/TestSilentLog.java b/src/test/java/org/apache/maven/plugins/dependency/utils/TestSilentLog.java
@@ -18,12 +18,12 @@
  */
 package org.apache.maven.plugins.dependency.utils;
 
-import junit.framework.TestCase;
 import org.apache.maven.plugin.logging.Log;
-import org.codehaus.plexus.logging.Logger;
+import org.junit.Test;
 
-public class TestSilentLog extends TestCase {
+public class TestSilentLog {
 
+    @Test
     public void testLog() {
         Log log = new DependencySilentLog();
         String text = "Text";
@@ -45,30 +45,4 @@ public void testLog() {
         log.isWarnEnabled();
         log.isInfoEnabled();
     }
-
-    public void testLogger() {
-        Logger log = new DependencySilentLog();
-        String text = "Text";
-        Throwable e = new RuntimeException();
-
-        log.debug(text);
-        log.debug(text, e);
-        log.error(text);
-        log.error(text, e);
-        log.warn(text);
-        log.warn(text, e);
-        log.info(text);
-        log.info(text, e);
-
-        log.fatalError(text);
-        log.fatalError(text, e);
-        log.getChildLogger(text);
-        log.getName();
-        log.getThreshold();
-        log.isDebugEnabled();
-        log.isErrorEnabled();
-        log.isFatalErrorEnabled();
-        log.isInfoEnabled();
-        log.isWarnEnabled();
-    }
 }