Write test cases for all suppression levels - #6

detekt-rules/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/KT.kt

@@ -20,7 +20,9 @@ enum class Case(val file: String) {
 	NestedClasses("/cases/NestedClasses.kt"),
 	NestedLongMethods("/cases/NestedLongMethods.kt"),
 	FeatureEnvy("/cases/FeatureEnvy.kt"),
-	SuppressedElements("/SuppressedElements.kt");
+	SuppressedElements("/SuppressedByElementAnnotation.kt"),
+	SuppressedElementsByFile("/SuppressedElementsByFileAnnotation.kt"),
+	SuppressedElementsByClass("/SuppressedElementsByClassAnnotation.kt");
 
 	fun path(): Path {
 		val resource = Case::class.java.getResource(file)

detekt-rules/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/SuppressingSpec.kt

@@ -14,7 +14,7 @@ import org.jetbrains.spek.api.dsl.it
  */
 class SuppressingSpec : Spek({
 
-	it("all findings are suppressed on different levels") {
+	it("all findings are suppressed on element levels") {
 		val ktFile = compileForTest(Case.SuppressedElements.path())
 		val ruleSet = RuleSet("Test", listOf(LongMethod(), LongParameterList(), ComplexCondition()))
 		val findings = ruleSet.accept(ktFile)
@@ -24,4 +24,24 @@ class SuppressingSpec : Spek({
 		assertThat(findings).hasSize(0)
 	}
 
+	it("all findings are suppressed on file levels") {
+		val ktFile = compileForTest(Case.SuppressedElementsByFile.path())
+		val ruleSet = RuleSet("Test", listOf(LongMethod(), LongParameterList(), ComplexCondition()))
+		val findings = ruleSet.accept(ktFile)
+		findings.forEach {
+			println(it.compact())
+		}
+		assertThat(findings).hasSize(0)
+	}
+
+	it("all findings are suppressed on class levels") {
+		val ktFile = compileForTest(Case.SuppressedElementsByClass.path())
+		val ruleSet = RuleSet("Test", listOf(LongMethod(), LongParameterList(), ComplexCondition()))
+		val findings = ruleSet.accept(ktFile)
+		findings.forEach {
+			println(it.compact())
+		}
+		assertThat(findings).hasSize(0)
+	}
+
 })

detekt-rules/src/test/resources/SuppressedElements.kt → detekt-rules/src/test/resources/SuppressedByElementAnnotation.kt

@@ -1,3 +1,5 @@
+@file:Suppress("unused")
+
 /**
  * @author Artur Bosch
  */

detekt-rules/src/test/resources/SuppressedElementsByClassAnnotation.kt

@@ -0,0 +1,42 @@
+/**
+ * @author Artur Bosch
+ */
+@Suppress("unused", "LongMethod", "LongParameterList", "ComplexCondition")
+class SuppressedElements3 {
+
+	fun lpl(a: Int, b: Int, c: Int, d: Int, e: Int, f: Int) = (a + b + c + d + e + f).apply {
+		assert(false) { "FAILED TEST" }
+	}
+
+	fun cc() {
+		if (this is SuppressedElements3 && this !is Any && this is Nothing && this is SuppressedElements3) {
+			assert(false) { "FAIL" }
+		}
+	}
+
+	fun lm() {
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		assert(false) { "FAILED TEST" }
+	}
+
+}

detekt-rules/src/test/resources/SuppressedElementsByFileAnnotation.kt

@@ -0,0 +1,48 @@
+@file:Suppress("unused", "LongMethod", "LongParameterList", "ComplexCondition")
+
+/**
+ * @author Artur Bosch
+ */
+
+fun lpl2(a: Int, b: Int, c: Int, d: Int, e: Int, f: Int) = (a + b + c + d + e + f).apply {
+	assert(false) { "FAILED TEST" }
+}
+
+class SuppressedElements2 {
+
+	fun lpl(a: Int, b: Int, c: Int, d: Int, e: Int, f: Int) = (a + b + c + d + e + f).apply {
+		assert(false) { "FAILED TEST" }
+	}
+
+	fun cc() {
+		if (this is SuppressedElements2 && this !is Any && this is Nothing && this is SuppressedElements2) {
+			assert(false) { "FAIL" }
+		}
+	}
+
+	fun lm() {
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		lpl(1, 2, 3, 4, 5, 6)
+		assert(false) { "FAILED TEST" }
+	}
+
+}