Update dependency io.github.microutils:kotlin-logging-jvm to v3

diktat-common/pom.xml

@@ -38,20 +38,6 @@
             <groupId>com.pinterest.ktlint</groupId>
             <artifactId>ktlint-core</artifactId>
         </dependency>
-        <dependency>
-            <groupId>org.slf4j</groupId>
-            <artifactId>slf4j-api</artifactId>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-core</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-slf4j-impl</artifactId>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter</artifactId>

diktat-common/src/main/kotlin/org/cqfn/diktat/common/config/reader/JsonResourceConfigReader.kt

@@ -1,8 +1,8 @@
 package org.cqfn.diktat.common.config.reader
 
 import org.cqfn.diktat.common.utils.loggerWithKtlintConfig
+import mu.KLogger
 import mu.KotlinLogging
-import org.slf4j.Logger
 
 import java.io.BufferedReader
 import java.io.IOException
@@ -61,6 +61,6 @@ abstract class JsonResourceConfigReader<T> {
         /**
          * A [Logger] that can be used
          */
-        val log: Logger = KotlinLogging.loggerWithKtlintConfig(JsonResourceConfigReader::class)
+        val log: KLogger = KotlinLogging.loggerWithKtlintConfig(JsonResourceConfigReader::class)
     }
 }

diktat-common/src/main/kotlin/org/cqfn/diktat/common/config/rules/RulesConfigReader.kt

@@ -10,8 +10,8 @@ import org.cqfn.diktat.common.utils.loggerWithKtlintConfig
 
 import com.charleskorn.kaml.Yaml
 import com.charleskorn.kaml.YamlConfiguration
+import mu.KLogger
 import mu.KotlinLogging
-import org.slf4j.Logger
 
 import java.io.BufferedReader
 import java.io.File
@@ -111,7 +111,7 @@ open class RulesConfigReader(override val classLoader: ClassLoader) : JsonResour
         /**
          * A [Logger] that can be used
          */
-        val log: Logger = KotlinLogging.loggerWithKtlintConfig(RulesConfigReader::class)
+        val log: KLogger = KotlinLogging.loggerWithKtlintConfig(RulesConfigReader::class)
     }
 }
 

diktat-common/src/main/kotlin/org/cqfn/diktat/common/utils/LoggingUtils.kt

@@ -6,7 +6,6 @@ package org.cqfn.diktat.common.utils
 
 import com.pinterest.ktlint.core.initKtLintKLogger
 import mu.KotlinLogging
-import org.slf4j.LoggerFactory
 import kotlin.reflect.KClass
 
 /**
@@ -25,4 +24,4 @@ fun KotlinLogging.loggerWithKtlintConfig(func: () -> Unit) =
  * @return a logger
  */
 fun KotlinLogging.loggerWithKtlintConfig(clazz: KClass<*>) =
-    logger(LoggerFactory.getLogger(clazz.java)).initKtLintKLogger()
+    logger(clazz.java.name).initKtLintKLogger()

diktat-rules/pom.xml

@@ -49,20 +49,6 @@
             <groupId>io.github.microutils</groupId>
             <artifactId>kotlin-logging-jvm</artifactId>
         </dependency>
-        <dependency>
-            <groupId>org.slf4j</groupId>
-            <artifactId>slf4j-api</artifactId>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-core</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-slf4j-impl</artifactId>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter</artifactId>

diktat-rules/src/main/kotlin/org/cqfn/diktat/ruleset/rules/chapter3/EmptyBlock.kt

@@ -9,6 +9,7 @@ import org.cqfn.diktat.ruleset.utils.*
 
 import com.pinterest.ktlint.core.ast.ElementType
 import com.pinterest.ktlint.core.ast.ElementType.CALL_EXPRESSION
+import com.pinterest.ktlint.core.ast.ElementType.DOT_QUALIFIED_EXPRESSION
 import com.pinterest.ktlint.core.ast.ElementType.FUNCTION_LITERAL
 import com.pinterest.ktlint.core.ast.ElementType.IDENTIFIER
 import com.pinterest.ktlint.core.ast.ElementType.LAMBDA_EXPRESSION
@@ -46,7 +47,10 @@ class EmptyBlock(configRules: List<RulesConfig>) : DiktatRule(
 
     @Suppress("UnsafeCallOnNullableType", "TOO_LONG_FUNCTION")
     private fun checkEmptyBlock(node: ASTNode, configuration: EmptyBlockStyleConfiguration) {
-        if (node.treeParent.isOverridden() || isAnonymousSamClass(node) || isLambdaUsedAsFunction(node)) {
+        if (node.treeParent.isOverridden() ||
+            isAnonymousSamClass(node) ||
+            isLambdaUsedAsFunction(node) ||
+            isKotlinLogging(node)) {
             return
         }
         if (node.isBlockEmpty()) {
@@ -118,6 +122,13 @@ class EmptyBlock(configRules: List<RulesConfig>) : DiktatRule(
         }
     }
 
+    private fun isKotlinLogging(node: ASTNode): Boolean = node.findParentNodeWithSpecificType(DOT_QUALIFIED_EXPRESSION)
+        ?.text
+        ?.replace(" ", "")
+        .let {
+            it == "KotlinLogging.logger{}"
+        }
+
     /**
      * [RuleConfiguration] for empty blocks formatting
      */

diktat-rules/src/main/kotlin/org/cqfn/diktat/ruleset/utils/KtlintUtils.kt

@@ -13,7 +13,6 @@ import com.pinterest.ktlint.core.RuleSetProvider
 import mu.KotlinLogging
 import org.intellij.lang.annotations.Language
 
-@Suppress("EMPTY_BLOCK_STRUCTURE_ERROR")
 private val log = KotlinLogging.loggerWithKtlintConfig {}
 
 @Suppress("TYPE_ALIAS")

diktat-rules/src/main/kotlin/org/cqfn/diktat/ruleset/utils/StringCaseUtils.kt

@@ -8,7 +8,6 @@ import mu.KotlinLogging
 
 import java.util.Locale
 
-@Suppress("EMPTY_BLOCK_STRUCTURE_ERROR")
 private val log = KotlinLogging.loggerWithKtlintConfig {}
 
 /**

diktat-rules/src/test/kotlin/org/cqfn/diktat/ruleset/chapter3/EmptyBlockWarnTest.kt

@@ -238,4 +238,19 @@ class EmptyBlockWarnTest : LintTestBase(::EmptyBlock) {
             rulesConfigList = rulesConfigListEmptyBlockExist
         )
     }
+
+    @Test
+    @Tag(WarningNames.EMPTY_BLOCK_STRUCTURE_ERROR)
+    fun `should not trigger on KotlinLogging logger`() {
+        lintMethod(
+            """
+                |import mu.KotlinLogging
+                |
+                |fun some() {
+                |    val log = KotlinLogging.logger {}
+                |    log.info { "test" }
+                |}
+            """.trimMargin(),
+        )
+    }
 }

diktat-ruleset/pom.xml

@@ -69,16 +69,6 @@
             <artifactId>kotlin-compiler-embeddable</artifactId>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-core</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-slf4j-impl</artifactId>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter</artifactId>

diktat-test-framework/pom.xml

@@ -30,14 +30,6 @@
             <groupId>commons-io</groupId>
             <artifactId>commons-io</artifactId>
         </dependency>
-        <dependency>
-            <groupId>org.slf4j</groupId>
-            <artifactId>slf4j-api</artifactId>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-core</artifactId>
-        </dependency>
         <dependency>
             <groupId>org.apache.logging.log4j</groupId>
             <artifactId>log4j-slf4j-impl</artifactId>

diktat-test-framework/src/main/java/org/cqfn/diktat/test/framework/common/StreamGobbler.java

@@ -1,30 +1,34 @@
 package org.cqfn.diktat.test.framework.common;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.nio.charset.Charset;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.function.BiConsumer;
 
 public class StreamGobbler extends Thread {
-    private static final Logger log = LoggerFactory.getLogger(StreamGobbler.class);
     private final InputStream inputStream;
     private final String streamType;
+    private final BiConsumer<Exception, String> exceptionHandler;
     private final ArrayList<String> result;
     private volatile boolean isStopped = false;
 
     /**
      * @param inputStream the InputStream to be consumed
-     * @param streamType  the stream type (should be OUTPUT or ERROR)
+     * @param streamType the stream type (should be OUTPUT or ERROR)
+     * @param exceptionHandler the exception handler
      */
-    public StreamGobbler(final InputStream inputStream, final String streamType) {
+    public StreamGobbler(
+            final InputStream inputStream,
+            final String streamType,
+            final BiConsumer<Exception, String> exceptionHandler
+    ) {
         this.inputStream = inputStream;
         this.streamType = streamType;
+        this.exceptionHandler = exceptionHandler;
         this.result = new ArrayList<>();
     }
 
@@ -43,7 +47,7 @@ synchronized public void run() {
                 this.result.add(line);
             }
         } catch (IOException ex) {
-                log.error("Failed to consume and display the input stream of type " + streamType + ".", ex);
+                exceptionHandler.accept(ex, "Failed to consume and display the input stream of type " + streamType + ".");
         } finally {
             this.isStopped = true;
             notify();
@@ -55,7 +59,7 @@ synchronized public List<String> getContent() {
                 try {
                     wait();
                 } catch (InterruptedException e) {
-                    log.error("Cannot get content of output stream", e);
+                    exceptionHandler.accept(e, "Cannot get content of output stream");
                 }
         }
         return this.result;

diktat-test-framework/src/main/kotlin/org/cqfn/diktat/test/framework/common/LocalCommandExecutor.kt

@@ -1,6 +1,7 @@
 package org.cqfn.diktat.test.framework.common
 
-import org.slf4j.LoggerFactory
+import org.cqfn.diktat.common.utils.loggerWithKtlintConfig
+import mu.KotlinLogging
 
 import java.io.IOException
 
@@ -15,14 +16,18 @@ class LocalCommandExecutor internal constructor(private val command: String) {
      */
     fun executeCommand(): ExecutionResult {
         try {
-            log.info("Executing command: {}", command)
+            log.info { "Executing command: $command" }
             val process = Runtime.getRuntime().exec(command)
             process.outputStream.close()
             val inputStream = process.inputStream
-            val outputGobbler = StreamGobbler(inputStream, "OUTPUT")
+            val outputGobbler = StreamGobbler(inputStream, "OUTPUT") { msg, ex ->
+                log.error(ex, msg)
+            }
             outputGobbler.start()
             val errorStream = process.errorStream
-            val errorGobbler = StreamGobbler(errorStream, "ERROR")
+            val errorGobbler = StreamGobbler(errorStream, "ERROR") { msg, ex ->
+                log.error(ex, msg)
+            }
             errorGobbler.start()
             return ExecutionResult(outputGobbler.content, errorGobbler.content)
         } catch (ex: IOException) {
@@ -32,6 +37,6 @@ class LocalCommandExecutor internal constructor(private val command: String) {
     }
 
     companion object {
-        private val log = LoggerFactory.getLogger(LocalCommandExecutor::class.java)
+        private val log = KotlinLogging.loggerWithKtlintConfig {}
     }
 }

diktat-test-framework/src/main/kotlin/org/cqfn/diktat/test/framework/config/TestArgumentsReader.kt

@@ -2,14 +2,15 @@ package org.cqfn.diktat.test.framework.config
 
 import org.cqfn.diktat.common.cli.CliArgument
 import org.cqfn.diktat.common.config.reader.JsonResourceConfigReader
+import org.cqfn.diktat.common.utils.loggerWithKtlintConfig
+import mu.KotlinLogging
 
 import org.apache.commons.cli.CommandLine
 import org.apache.commons.cli.CommandLineParser
 import org.apache.commons.cli.DefaultParser
 import org.apache.commons.cli.HelpFormatter
 import org.apache.commons.cli.Options
 import org.apache.commons.cli.ParseException
-import org.slf4j.LoggerFactory
 
 import java.io.BufferedReader
 import java.io.IOException
@@ -94,6 +95,6 @@ class TestArgumentsReader(
     }
 
     companion object {
-        private val log = LoggerFactory.getLogger(TestArgumentsReader::class.java)
+        private val log = KotlinLogging.loggerWithKtlintConfig {}
     }
 }

diktat-test-framework/src/main/kotlin/org/cqfn/diktat/test/framework/processing/FileComparator.kt

@@ -1,9 +1,10 @@
 package org.cqfn.diktat.test.framework.processing
 
+import org.cqfn.diktat.common.utils.loggerWithKtlintConfig
 import io.github.petertrr.diffutils.diff
 import io.github.petertrr.diffutils.patch.ChangeDelta
 import io.github.petertrr.diffutils.text.DiffRowGenerator
-import org.slf4j.LoggerFactory
+import mu.KotlinLogging
 
 import java.io.File
 import java.io.IOException
@@ -96,7 +97,7 @@ class FileComparator(
     }
 
     companion object {
-        private val log = LoggerFactory.getLogger(FileComparator::class.java)
+        private val log = KotlinLogging.loggerWithKtlintConfig {}
 
         /**
          * @param file file where to write these list to, separated with newlines.

diktat-test-framework/src/main/kotlin/org/cqfn/diktat/test/framework/processing/TestCheckWarn.kt

@@ -1,14 +1,16 @@
 package org.cqfn.diktat.test.framework.processing
 
+import org.cqfn.diktat.common.utils.loggerWithKtlintConfig
 import org.cqfn.diktat.test.framework.config.TestConfig
-import org.slf4j.Logger
-import org.slf4j.LoggerFactory
+import mu.KLogger
+import mu.KotlinLogging
 
 /**
  * [TestCompare] that uses stderr as tests output stream
  */
 class TestCheckWarn : TestCompare() {
-    @Suppress("MISSING_KDOC_CLASS_ELEMENTS") override val log: Logger = LoggerFactory.getLogger(TestCheckWarn::class.java)
+    @Suppress("EMPTY_BLOCK_STRUCTURE_ERROR", "MISSING_KDOC_CLASS_ELEMENTS")
+    override val log: KLogger = KotlinLogging.loggerWithKtlintConfig {}
 
     @Suppress(
         "UnusedPrivateMember",

diktat-test-framework/src/main/kotlin/org/cqfn/diktat/test/framework/processing/TestComparatorUnit.kt

@@ -1,8 +1,7 @@
 package org.cqfn.diktat.test.framework.processing
 
-import org.slf4j.Logger
-import org.slf4j.LoggerFactory
-
+import org.cqfn.diktat.common.utils.loggerWithKtlintConfig
+import mu.KotlinLogging
 import java.io.IOException
 import java.nio.file.Path
 import java.nio.file.Paths
@@ -115,7 +114,7 @@ class TestComparatorUnit(
     }
 
     private companion object {
-        private val log: Logger = LoggerFactory.getLogger(TestComparatorUnit::class.java)
+        private val log = KotlinLogging.loggerWithKtlintConfig {}
 
         /**
          * @param file the file whose content is to be read.