Merge pull request #7293 from element-hq/ci/run-konsist-tests-sequentially

Run `Konsist` tests sequentially

tests/konsist/build.gradle.kts

@@ -29,6 +29,11 @@ dependencies {
 // - Add every single module as a dependency of this one.
 // - Move the Konsist tests to the `app` module, but the `app` module does not need to know about Konsist.
 tasks.withType<Test>().configureEach {
+    // Disable parallel tests: every time we run a test class in parallel a new JVM is started, and the `KoScope` contents can't be shared between them.
+    // This scope can take 1-2min to load, so having several instances of this happening is a terrible idea.
+    // It's actually faster to run the tests sequentially, at least locally.
+    maxParallelForks = 1
+
     val isNotCheckTask = gradle.startParameter.taskNames.any { it.contains("check", ignoreCase = true).not() }
     outputs.upToDateWhen { isNotCheckTask }
 }

tests/konsist/src/test/kotlin/io/element/android/tests/konsist/KonsistFieldTest.kt

@@ -1,28 +0,0 @@
-/*
- * Copyright (c) 2025 Element Creations Ltd.
- * Copyright 2023-2025 New Vector Ltd.
- *
- * SPDX-License-Identifier: AGPL-3.0-only OR LicenseRef-Element-Commercial.
- * Please see LICENSE files in the repository root for full details.
- */
-
-package io.element.android.tests.konsist
-
-import com.lemonappdev.konsist.api.Konsist
-import com.lemonappdev.konsist.api.ext.list.properties
-import com.lemonappdev.konsist.api.verify.assertFalse
-import org.junit.Test
-
-class KonsistFieldTest {
-    @Test
-    fun `no field should have 'm' prefix`() {
-        Konsist
-            .scopeFromProject()
-            .classes()
-            .properties()
-            .assertFalse {
-                val secondCharacterIsUppercase = it.name.getOrNull(1)?.isUpperCase() ?: false
-                it.name.startsWith('m') && secondCharacterIsUppercase
-            }
-    }
-}