test(authorization): Optimize authorization tests

When installing the Keycloak extension, drop the `createRealmPerTest`
flag per default. The additional isolation is no longer needed for most
tests, since no roles or other data in Keycloak are manipulated during
test execution, and an initial setup of test users used by the test cases
is sufficient. For tests that do manipulate the state in Keycloak,
support enabling this feature on demand.

Avoiding the repeated setup of the realm saves a few seconds for every
test case, which has a notable effect on the total test execution times.

Signed-off-by: Oliver Heger <[email protected]>

Author: oheger-bosch

core/src/test/kotlin/api/AbstractIntegrationTest.kt

@@ -56,10 +56,18 @@ import org.eclipse.apoapsis.ortserver.secrets.SecretsProviderFactoryForTesting
 import org.eclipse.apoapsis.ortserver.utils.logging.runBlocking
 
 @Suppress("UnnecessaryAbstractClass")
-abstract class AbstractIntegrationTest(body: AbstractIntegrationTest.() -> Unit) : WordSpec() {
+abstract class AbstractIntegrationTest(
+    body: AbstractIntegrationTest.() -> Unit,
+
+    /**
+     * A flag indicating whether a new Keycloak realm should be created for each test. This should be set to *true* for
+     * test classes that manipulate the Keycloak realm during text execution.
+     */
+    createKeycloakRealmPerTest: Boolean = false
+) : WordSpec() {
     val dbExtension = extension(DatabaseTestExtension())
 
-    val keycloak = install(KeycloakTestExtension(createRealmPerTest = true)) {
+    val keycloak = install(KeycloakTestExtension(createRealmPerTest = createKeycloakRealmPerTest)) {
         setUpUser(SUPERUSER, SUPERUSER_PASSWORD)
         setUpUser(TEST_USER, TEST_USER_PASSWORD)
         setUpClientScope(TEST_SUBJECT_CLIENT)

core/src/test/kotlin/api/AdminRouteIntegrationTest.kt

@@ -258,4 +258,4 @@ class AdminRouteIntegrationTest : AbstractIntegrationTest({
             }
         }
     }
-})
+}, createKeycloakRealmPerTest = true)

shared/ktor-utils/src/testFixtures/kotlin/AbstractAuthorizationTest.kt

@@ -83,7 +83,7 @@ private const val TEST_USER_PASSWORD = "password"
 @Suppress("UnnecessaryAbstractClass")
 abstract class AbstractAuthorizationTest(body: AbstractAuthorizationTest.() -> Unit) : WordSpec() {
     val dbExtension = DatabaseTestExtension()
-    val keycloakExtension = KeycloakTestExtension(createRealmPerTest = true)
+    val keycloakExtension = KeycloakTestExtension()
 
     // The "extension()" and "install()" functions cannot be used above because of
     // https://github.com/kotest/kotest/issues/3555.