ignoreLines feature for filtering the input file

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/IgnoreLines/Bug1Expected.kt

@@ -0,0 +1,21 @@
+package org.cqfn.save.IgnoreLinesTest.IgnoreLines
+
+class D {
+    val x = 0
+
+    /**
+     * @return
+     */
+    fun bar(): Bar {
+        val qux = 42
+        return Bar(qux)
+    }
+}
+
+/**
+ * @param foo
+ */
+fun readFile(foo: Foo) {
+    var bar: Bar
+}
+// ;warn:0: [TEST] JUST_A_TEST

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/IgnoreLines/Bug1Test.kt

@@ -0,0 +1,11 @@
+package test.smoke.src.main.kotlin
+
+fun readFile(foo: Foo) {
+    var bar: Bar
+}
+
+class D {val x = 0
+fun bar(): Bar {val qux = 42; return Bar(qux)}
+}
+// ;warn:0: [TEST] JUST_A_TEST
+// IGNORE_ME

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/IgnoreLines/save.toml

@@ -0,0 +1,7 @@
+[general]
+    tags = ["FixIgnoreLines"]
+    description = "Test for ignoreLines option in Fix"
+    suiteName = "Autofix: Smoke Tests"
+
+[fix]
+    ignoreLines = ["// IGNORE_ME"]

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/IgnoreLinesIsEmpty/Bug1Expected.kt

@@ -0,0 +1,22 @@
+package org.cqfn.save.IgnoreLinesTest.IgnoreLinesIsEmpty
+
+class D {
+    val x = 0
+
+    /**
+     * @return
+     */
+    fun bar(): Bar {
+        val qux = 42
+        return Bar(qux)
+    }
+}
+
+/**
+ * @param foo
+ */
+fun readFile(foo: Foo) {
+    var bar: Bar
+}
+// ;warn:0: [TEST] JUST_A_TEST
+// IGNORE_ME

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/IgnoreLinesIsEmpty/Bug1Test.kt

@@ -0,0 +1,11 @@
+package test.smoke.src.main.kotlin
+
+fun readFile(foo: Foo) {
+    var bar: Bar
+}
+
+class D {val x = 0
+fun bar(): Bar {val qux = 42; return Bar(qux)}
+}
+// ;warn:0: [TEST] JUST_A_TEST
+// IGNORE_ME

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/IgnoreLinesIsEmpty/save.toml

@@ -0,0 +1,7 @@
+[general]
+    tags = ["smoke"]
+    description = "Smoke tests for diktat"
+    suiteName = "Autofix: Smoke Tests"
+
+[fix]
+    ignoreLines = []

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/NoIgnoreLines/Bug1Expected.kt

@@ -0,0 +1,21 @@
+package org.cqfn.save.IgnoreLinesTest.NoIgnoreLines
+
+class D {
+    val x = 0
+
+    /**
+     * @return
+     */
+    fun bar(): Bar {
+        val qux = 42
+        return Bar(qux)
+    }
+}
+
+/**
+ * @param foo
+ */
+fun readFile(foo: Foo) {
+    var bar: Bar
+}
+// IGNORE_ME

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/NoIgnoreLines/Bug1Test.kt

@@ -0,0 +1,11 @@
+package test.smoke.src.main.kotlin
+
+fun readFile(foo: Foo) {
+    var bar: Bar
+}
+
+class D {val x = 0
+fun bar(): Bar {val qux = 42; return Bar(qux)}
+}
+// ;warn:0: [TEST] JUST_A_TEST
+// IGNORE_ME

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/NoIgnoreLines/save.toml

@@ -0,0 +1,6 @@
+[general]
+    tags = ["smoke"]
+    description = "Smoke tests for diktat"
+    suiteName = "Autofix: Smoke Tests"
+
+[fix]

examples/kotlin-diktat/fix/smoke/src/main/kotlin/org/cqfn/save/chapter1/save.toml

@@ -0,0 +1,6 @@
+[general]
+    tags = ["smoke"]
+    description = "Smoke tests for diktat"
+    suiteName = "Autofix: Smoke Tests"
+
+[fix]

examples/kotlin-diktat/save.toml

@@ -2,6 +2,4 @@
     execCmd = "java -jar ktlint -R diktat.jar"
     description = "Test for diktat - linter and formatter for Kotlin"
     # this is the default value, you don't need to add it explicitly, but can be useful, if you have different pattern:
-    expectedWarningsPattern = "// ;warn:?(.*):(\\d*): (.+)"
-
-
+    expectedWarningsPattern = "// ;warn:?(.*):(\\d*): (.+)"

examples/kotlin-diktat/warn/chapter1/IgnoreLinesTest/EnumTestDetection.kt

@@ -0,0 +1,16 @@
+package org.cqfn.diktat.test.resources.test.paragraph1.naming.enum_
+
+// ;warn:3:1: [MISSING_KDOC_TOP_LEVEL] all public and internal top-level classes and functions should have Kdoc: EnumTestDetection (cannot be auto-corrected)
+// ;warn:30: [WRONG_DECLARATIONS_ORDER] declarations of constants and enum members should be sorted alphabetically: enum entries order is incorrect
+// ;warn:10:5: [ENUMS_SEPARATED] enum is incorrectly formatted: enums must end with semicolon
+enum class EnumTestDetection {
+    // ;warn:$line+1:5: [ENUM_VALUE] enum values should be{{ in }}selected UPPER_CASE snake/PascalCase format: paSC_SAl_l
+    paSC_SAl_l,
+
+    // ;warn:5: [ENUM_VALUE] enum values{{ should }}be in selected{{ UPPER_CASE }}snake/PascalCase format: PascAsl_f
+    PascAsl_f
+    // ;warn:$line-2:5: [ENUMS_SEPARATED] enum is incorrectly formatted: last enum entry must end with a comma
+
+    // ;warn:1:9: {{.*}}[PACKAGE_NAME_INCORRECT_PREFIX] package name should start from company's domain: org.cqfn.save{{.*}}
+}
+// ;warn:0:0: [YOU SHOULD NOT SEE THIS] this warning should not be shown

examples/kotlin-diktat/warn/chapter1/IgnoreLinesTest/save.toml

@@ -0,0 +1,7 @@
+[general]
+    tags = ["WarnIgnoreLines"]
+    description = "Test suite that checks if filtering by regex work."
+
+[warn]
+    testNameRegex = "^E.*T.*D.*"
+    ignoreLines = [".*// ;warn:0:0: \\[YOU SHOULD NOT SEE THIS\\] this warning should not be shown.*"]

examples/kotlin-diktat/warn/save.toml

@@ -17,4 +17,4 @@
     messageCaptureGroupOut = 4
     exactWarningsMatch = false
     warningTextHasColumn = true
-    warningTextHasLine = true
+    warningTextHasLine = true

save-common/src/commonMain/kotlin/org/cqfn/save/core/plugin/PluginConfig.kt

@@ -25,6 +25,11 @@ interface PluginConfig {
      */
     val type: TestConfigSections
 
+    /**
+     * list of regexes to be ignored
+     */
+    var ignoreLinesPatterns: MutableList<Regex>
+
     /**
      * Location of the toml config
      */
@@ -69,6 +74,7 @@ data class GeneralConfig(
     val runConfigPattern: Regex? = null,
 ) : PluginConfig {
     override val type = TestConfigSections.GENERAL
+    override var ignoreLinesPatterns: MutableList<Regex> = mutableListOf()
 
     @Transient
     override var configLocation: Path = "undefined_toml_location".toPath()
@@ -88,8 +94,10 @@ data class GeneralConfig(
             this.suiteName ?: other.suiteName,
             this.excludedTests ?: other.excludedTests,
             this.expectedWarningsPattern ?: other.expectedWarningsPattern,
-            this.runConfigPattern ?: other.runConfigPattern,
-        ).also { it.configLocation = this.configLocation }
+            this.runConfigPattern ?: other.runConfigPattern
+        ).also {
+            it.configLocation = this.configLocation
+        }
     }
 
     override fun validateAndSetDefaults(): GeneralConfig {
@@ -113,7 +121,9 @@ data class GeneralConfig(
             excludedTests ?: emptyList(),
             expectedWarningsPattern ?: defaultExpectedWarningPattern,
             runConfigPattern ?: defaultRunConfigPattern,
-        ).also { it.configLocation = this.configLocation }
+        ).also {
+            it.configLocation = this.configLocation
+        }
     }
 
     private fun errorMsgForRequireCheck(field: String) =

save-core/src/commonNonJsTest/kotlin/org/cqfn/save/core/integration/ClassicFixTest.kt

@@ -18,14 +18,46 @@ class ClassicFixTest {
             runTestsWithDiktat(
                 listOf(
                     "fix/save.toml"
-                ), 2
+                ), 5
             )
 
     @Test
     fun `executing fix plugin on parental save-toml file`() =
             runTestsWithDiktat(
                 listOf(
                     "fix/smoke/save.toml"
-                ), 2
+                ), 5
+            )
+
+    @Test
+    fun `execute fix plugin on folder`() =
+            runTestsWithDiktat(
+                listOf(
+                    "fix/smoke/src/main/kotlin/org/cqfn/save/"
+                ), 5
+            )
+
+    @Test
+    fun `check NoIgnoreLines`() =
+            runTestsWithDiktat(
+                listOf(
+                    "fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/NoIgnoreLines"
+                ), 1
+            )
+
+    @Test
+    fun `check IgnoreLinesIsEmpty`() =
+            runTestsWithDiktat(
+                listOf(
+                    "fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/IgnoreLinesIsEmpty"
+                ), 1
+            )
+
+    @Test
+    fun `check IgnoreLines`() =
+            runTestsWithDiktat(
+                listOf(
+                    "fix/smoke/src/main/kotlin/org/cqfn/save/IgnoreLinesTest/IgnoreLines"
+                ), 1
             )
 }

save-core/src/commonNonJsTest/kotlin/org/cqfn/save/core/integration/ClassicWarnTest.kt

@@ -38,7 +38,7 @@ class ClassicWarnTest {
             runTestsWithDiktat(
                 listOf(
                     "warn/chapter1"
-                ), 6
+                ), 7
             )
 
     @Test
@@ -49,6 +49,14 @@ class ClassicWarnTest {
                 ), 1
             )
 
+    @Test
+    fun `lines that match ignoreLines should be ignored`() =
+            runTestsWithDiktat(
+                listOf(
+                    "warn/chapter1/IgnoreLinesTest"
+                ), 1
+            )
+
     @Test
     fun `test output file set`() =
             runTestsWithDiktat(
@@ -73,15 +81,15 @@ class ClassicWarnTest {
             runTestsWithDiktat(
                 listOf(
                     "warn/save.toml"
-                ), 6
+                ), 7
             )
 
     @Test
     fun `executing warn plugin on parental save-toml file`() =
             runTestsWithDiktat(
                 listOf(
                     "warn/chapter1/save.toml"
-                ), 6
+                ), 7
             )
 
     @Test

save-plugins/fix-and-warn-plugin/src/commonMain/kotlin/org/cqfn/save/plugins/fixandwarn/FixAndWarnPlugin.kt

@@ -121,7 +121,6 @@ class FixAndWarnPlugin(
         files.forEach { file ->
             val fileData = fs.readLines(file)
             filesAndTheirWarningsMap[file] = mutableListOf()
-
             val fileDataWithoutWarnings = fileData.filterIndexed { index, line ->
                 val isLineWithWarning = (generalConfig.expectedWarningsPattern!!.find(line)?.groups != null)
                 if (isLineWithWarning) {

save-plugins/fix-and-warn-plugin/src/commonMain/kotlin/org/cqfn/save/plugins/fixandwarn/FixAndWarnPluginConfig.kt

@@ -25,14 +25,19 @@ data class FixAndWarnPluginConfig(
     @Transient
     override var configLocation: Path = "undefined_toml_location".toPath()
 
+    @Transient
+    override var ignoreLinesPatterns: MutableList<Regex> = defaultIgnoreLines
+
     override fun mergeWith(otherConfig: PluginConfig): PluginConfig {
         val other = otherConfig as FixAndWarnPluginConfig
         val mergedFixPluginConfig = fix.mergeWith(other.fix)
         val mergedWarnPluginConfig = warn.mergeWith(other.warn)
         return FixAndWarnPluginConfig(
             mergedFixPluginConfig as FixPluginConfig,
             mergedWarnPluginConfig as WarnPluginConfig
-        ).also { it.configLocation = this.configLocation }
+        ).also {
+            it.configLocation = this.configLocation
+        }
     }
 
     override fun validateAndSetDefaults(): PluginConfig {
@@ -48,6 +53,11 @@ data class FixAndWarnPluginConfig(
         return FixAndWarnPluginConfig(
             fix.validateAndSetDefaults(),
             warn.validateAndSetDefaults()
-        ).also { it.configLocation = this.configLocation }
+        ).also {
+            it.configLocation = this.configLocation
+        }
+    }
+    companion object {
+        internal val defaultIgnoreLines: MutableList<Regex> = mutableListOf()
     }
 }