Merge pull request #9584 from rafaeltonholo/fix/validator-logger-quality-test

wmontwe · web-flow · commit fbcbbff3b72a · 2025-08-07T17:16:53.000+02:00
chore(quality): add DebugLogConfigurator into the exclusion list of Timber logging
diff --git a/quality/konsist/src/test/kotlin/net/thunderbird/quality/ValidateLogger.kt b/quality/konsist/src/test/kotlin/net/thunderbird/quality/ValidateLogger.kt
@@ -30,7 +30,7 @@ class ValidateLogger {
             .filterNot { it.hasNameMatching("ConsoleLogSink.android|ConsoleLogSinkTest.android".toRegex()) }
             .filterNot {
                 // Exclude legacy code that still uses Timber
-                it.hasNameMatching("LogFileWriter|FileLoggerTree|K9".toRegex())
+                it.hasNameMatching("LogFileWriter|FileLoggerTree|K9|DebugLogConfigurator".toRegex())
             }
             .assertFalse(
                 additionalMessage = "No class should use timber.log.Timber import, use net.thunderbird.core.logging.Logger instead."

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ class ValidateLogger {`
`30`	`30`	`.filterNot { it.hasNameMatching("ConsoleLogSink.android\|ConsoleLogSinkTest.android".toRegex()) }`
`31`	`31`	`.filterNot {`
`32`	`32`	`// Exclude legacy code that still uses Timber`
`33`		`- it.hasNameMatching("LogFileWriter\|FileLoggerTree\|K9".toRegex())`
	`33`	`+ it.hasNameMatching("LogFileWriter\|FileLoggerTree\|K9\|DebugLogConfigurator".toRegex())`
`34`	`34`	`}`
`35`	`35`	`.assertFalse(`
`36`	`36`	`additionalMessage = "No class should use timber.log.Timber import, use net.thunderbird.core.logging.Logger instead."`