Browse Source

Konsist: add rule obout test class name, and fix existing issue

pull/1552/head
Benoit Marty 11 months ago committed by Benoit Marty
parent
commit
7389748f3e
  1. 10
      app/src/test/kotlin/io/element/android/app/KonsistTestTest.kt
  2. 2
      features/call/src/test/kotlin/io/element/android/features/call/CallIntentDataParserTest.kt
  3. 2
      features/login/impl/src/test/kotlin/io/element/android/features/login/impl/error/ErrorFormatterTest.kt
  4. 2
      libraries/core/src/test/kotlin/io/element/android/libraries/core/extensions/ResultTest.kt
  5. 2
      libraries/eventformatter/impl/src/test/kotlin/io/element/android/libraries/eventformatter/impl/DefaultRoomLastMessageFormatterTest.kt
  6. 2
      libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/auth/AuthErrorCodeTest.kt
  7. 2
      libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/MatrixToConverterTest.kt
  8. 2
      libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/PermalinkBuilderTest.kt
  9. 2
      libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/PermalinkParserTest.kt
  10. 2
      libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/auth/AuthenticationExceptionMappingTest.kt
  11. 2
      libraries/mediapickers/api/src/test/kotlin/io/element/android/libraries/mediapickers/PickerTypeTest.kt

10
app/src/test/kotlin/io/element/android/app/KonsistTestTest.kt

@ -18,11 +18,21 @@ package io.element.android.app @@ -18,11 +18,21 @@ package io.element.android.app
import com.lemonappdev.konsist.api.Konsist
import com.lemonappdev.konsist.api.ext.list.modifierprovider.withoutOverrideModifier
import com.lemonappdev.konsist.api.ext.list.withFunction
import com.lemonappdev.konsist.api.ext.list.withReturnType
import com.lemonappdev.konsist.api.verify.assertTrue
import org.junit.Test
class KonsistTestTest {
@Test
fun `Classes name containing @Test must end with 'Test''`() {
Konsist
.scopeFromTest()
.classes()
.withFunction { it.hasAnnotationOf(Test::class) }
.assertTrue { it.name.endsWith("Test") }
}
@Test
fun `Function which creates Presenter in test MUST be named 'createPresenterName'`() {
Konsist

2
features/call/src/test/kotlin/io/element/android/features/call/CallIntentDataParserTests.kt → features/call/src/test/kotlin/io/element/android/features/call/CallIntentDataParserTest.kt

@ -23,7 +23,7 @@ import org.robolectric.RobolectricTestRunner @@ -23,7 +23,7 @@ import org.robolectric.RobolectricTestRunner
import java.net.URLEncoder
@RunWith(RobolectricTestRunner::class)
class CallIntentDataParserTests {
class CallIntentDataParserTest {
private val callIntentDataParser = CallIntentDataParser()

2
features/login/impl/src/test/kotlin/io/element/android/features/login/impl/error/ErrorFormatterTests.kt → features/login/impl/src/test/kotlin/io/element/android/features/login/impl/error/ErrorFormatterTest.kt

@ -22,7 +22,7 @@ import io.element.android.libraries.matrix.api.auth.AuthenticationException @@ -22,7 +22,7 @@ import io.element.android.libraries.matrix.api.auth.AuthenticationException
import io.element.android.libraries.ui.strings.CommonStrings
import org.junit.Test
class ErrorFormatterTests {
class ErrorFormatterTest {
// region loginError
@Test

2
libraries/core/src/test/kotlin/io/element/android/libraries/core/extensions/ResultTests.kt → libraries/core/src/test/kotlin/io/element/android/libraries/core/extensions/ResultTest.kt

@ -19,7 +19,7 @@ package io.element.android.libraries.core.extensions @@ -19,7 +19,7 @@ package io.element.android.libraries.core.extensions
import com.google.common.truth.Truth.assertThat
import org.junit.Test
class ResultTests {
class ResultTest {
@Test
fun testFlatMap() {

2
libraries/eventformatter/impl/src/test/kotlin/io/element/android/libraries/eventformatter/impl/DefaultRoomLastMessageFormatterTests.kt → libraries/eventformatter/impl/src/test/kotlin/io/element/android/libraries/eventformatter/impl/DefaultRoomLastMessageFormatterTest.kt

@ -61,7 +61,7 @@ import org.robolectric.annotation.Config @@ -61,7 +61,7 @@ import org.robolectric.annotation.Config
@Suppress("LargeClass")
@RunWith(RobolectricTestRunner::class)
class DefaultRoomLastMessageFormatterTests {
class DefaultRoomLastMessageFormatterTest {
private lateinit var context: Context
private lateinit var fakeMatrixClient: FakeMatrixClient

2
libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/auth/AuthErrorCodeTests.kt → libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/auth/AuthErrorCodeTest.kt

@ -19,7 +19,7 @@ package io.element.android.libraries.matrix.api.auth @@ -19,7 +19,7 @@ package io.element.android.libraries.matrix.api.auth
import com.google.common.truth.Truth.assertThat
import org.junit.Test
class AuthErrorCodeTests {
class AuthErrorCodeTest {
@Test
fun `errorCode finds UNKNOWN code`() {

2
libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/MatrixToConverterTests.kt → libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/MatrixToConverterTest.kt

@ -23,7 +23,7 @@ import org.junit.runner.RunWith @@ -23,7 +23,7 @@ import org.junit.runner.RunWith
import org.robolectric.RobolectricTestRunner
@RunWith(RobolectricTestRunner::class)
class MatrixToConverterTests {
class MatrixToConverterTest {
@Test
fun `converting a matrix-to url does nothing`() {

2
libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/PermalinkBuilderTests.kt → libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/PermalinkBuilderTest.kt

@ -23,7 +23,7 @@ import io.element.android.tests.testutils.assertThrowsInDebug @@ -23,7 +23,7 @@ import io.element.android.tests.testutils.assertThrowsInDebug
import io.element.android.tests.testutils.isInDebug
import org.junit.Test
class PermalinkBuilderTests {
class PermalinkBuilderTest {
fun `building a permalink for an invalid user id throws when verifying the id`() {
assertThrowsInDebug {

2
libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/PermalinkParserTests.kt → libraries/matrix/api/src/test/kotlin/io/element/android/libraries/matrix/api/permalink/PermalinkParserTest.kt

@ -22,7 +22,7 @@ import org.junit.runner.RunWith @@ -22,7 +22,7 @@ import org.junit.runner.RunWith
import org.robolectric.RobolectricTestRunner
@RunWith(RobolectricTestRunner::class)
class PermalinkParserTests {
class PermalinkParserTest {
@Test
fun `parsing an invalid url returns a fallback link`() {

2
libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/auth/AuthenticationExceptionMappingTests.kt → libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/auth/AuthenticationExceptionMappingTest.kt

@ -22,7 +22,7 @@ import io.element.android.libraries.matrix.api.auth.AuthenticationException @@ -22,7 +22,7 @@ import io.element.android.libraries.matrix.api.auth.AuthenticationException
import org.junit.Test
import org.matrix.rustcomponents.sdk.AuthenticationException as RustAuthenticationException
class AuthenticationExceptionMappingTests {
class AuthenticationExceptionMappingTest {
@Test
fun `mapping an exception with no message returns 'Unknown error' message`() {

2
libraries/mediapickers/api/src/test/kotlin/io/element/android/libraries/mediapickers/PickerTypeTests.kt → libraries/mediapickers/api/src/test/kotlin/io/element/android/libraries/mediapickers/PickerTypeTest.kt

@ -26,7 +26,7 @@ import org.junit.runner.RunWith @@ -26,7 +26,7 @@ import org.junit.runner.RunWith
import org.robolectric.RobolectricTestRunner
@RunWith(RobolectricTestRunner::class)
class PickerTypeTests {
class PickerTypeTest {
@Test
fun `ImageAndVideo - assert types`() {
Loading…
Cancel
Save