test: remove the DebugLogConfigurator from the exclusion list of Timber logging, present in the ValidateLogger#no class should use Timber logging quality test

shamim-emon · shamim-emon · commit feb8c23b614f · 2025-09-25T20:50:14.000+06:00
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|DebugLogConfigurator".toRegex())
+                it.hasNameMatching("LogFileWriter|FileLoggerTree|K9".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\|DebugLogConfigurator".toRegex())`
	`33`	`+ it.hasNameMatching("LogFileWriter\|FileLoggerTree\|K9".toRegex())`
`34`	`34`	`}`
`35`	`35`	`.assertFalse(`
`36`	`36`	`additionalMessage = "No class should use timber.log.Timber import, use net.thunderbird.core.logging.Logger instead."`