Merge pull request #573 from yschimke/output_error

Apply suppressions to AccessibilityViewCheckResult instead of filtering

roborazzi-accessibility-check/src/main/java/com/github/takahirom/roborazzi/RoborazziATFAccessibilityChecker.kt

@@ -194,8 +194,24 @@ data class RoborazziATFAccessibilityChecker(
 
     val results = viewChecker.runChecksOnView(ImmutableSet.copyOf(checks), view, parameters)
 
-    return results.filter {
-      !suppressions.matches(it)
+    return results.applySuppressions(suppressions)
+  }
+
+  private fun List<AccessibilityViewCheckResult>.applySuppressions(
+    suppressions: Matcher<in AccessibilityViewCheckResult>
+  ): List<AccessibilityViewCheckResult> {
+    val ranTypes = listOf(
+      AccessibilityCheckResultType.ERROR,
+      AccessibilityCheckResultType.WARNING,
+      AccessibilityCheckResultType.INFO
+    )
+
+    return map { checkResult ->
+      if (suppressions.matches(checkResult) && ranTypes.contains(checkResult.type)) {
+        checkResult.suppressedResultCopy
+      } else {
+        checkResult
+      }
     }
   }
 
@@ -210,6 +226,9 @@ data class RoborazziATFAccessibilityChecker(
         AccessibilityCheckResultType.WARNING -> roborazziErrorLog(
           "Warning: $check"
         )
+        AccessibilityCheckResultType.SUPPRESSED -> roborazziReportLog(
+          "Suppressed: $check"
+        )
 
         AccessibilityCheckResultType.INFO -> roborazziReportLog(
           "Info: $check"