diff options
| author | Albin <albin@mullvad.net> | 2024-06-25 16:45:43 +0200 |
|---|---|---|
| committer | Albin <albin@mullvad.net> | 2024-06-26 12:59:29 +0200 |
| commit | 0316f672c58f5558ce9880af698098c3718591b6 (patch) | |
| tree | 87bd36bbb036990544ce2e6c3614598e439a180c /android | |
| parent | aa272071729e8fe11e8224d5438aabd38f08b245 (diff) | |
| download | mullvadvpn-0316f672c58f5558ce9880af698098c3718591b6.tar.xz mullvadvpn-0316f672c58f5558ce9880af698098c3718591b6.zip | |
Add konsist rule disallowing direct android log calls
Diffstat (limited to 'android')
| -rw-r--r-- | android/test/arch/src/test/kotlin/net/mullvad/mullvadvpn/test/arch/LogTest.kt | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/android/test/arch/src/test/kotlin/net/mullvad/mullvadvpn/test/arch/LogTest.kt b/android/test/arch/src/test/kotlin/net/mullvad/mullvadvpn/test/arch/LogTest.kt new file mode 100644 index 0000000000..1c46c322df --- /dev/null +++ b/android/test/arch/src/test/kotlin/net/mullvad/mullvadvpn/test/arch/LogTest.kt @@ -0,0 +1,24 @@ +package net.mullvad.mullvadvpn.test.arch + +import com.lemonappdev.konsist.api.Konsist +import com.lemonappdev.konsist.api.verify.assertFalse +import java.util.stream.Stream +import org.junit.jupiter.api.DynamicTest +import org.junit.jupiter.api.TestFactory + +class LogTest { + @TestFactory + fun `ensure no usage of disallowed loggers`(): Stream<DynamicTest> = + DISALLOWED_LOGGER_PACKAGE_NAMES.stream().map { disallowedLoggerPackageName -> + val testName = "ensure no usage of: $disallowedLoggerPackageName" + DynamicTest.dynamicTest(testName) { + Konsist.scopeFromProject().imports.assertFalse { import -> + import.name == disallowedLoggerPackageName + } + } + } + + companion object { + private val DISALLOWED_LOGGER_PACKAGE_NAMES = listOf("android.util.Log") + } +} |
