test refactor

thehyve · Jan 10, 2024 · 4b8bf4f · 4b8bf4f
1 parent 78c62ac
commit 4b8bf4f
Show file tree

Hide file tree

Showing 11 changed files with 27 additions and 357 deletions.
diff --git a/src/test/java/org/cbioportal/test/integration/MockServerContainer.java b/src/test/java/org/cbioportal/test/integration/MockServerContainer.java
diff --git a/src/test/java/org/cbioportal/test/integration/OAuth2KeycloakInitializer.java b/src/test/java/org/cbioportal/test/integration/OAuth2KeycloakInitializer.java
@@ -23,12 +23,6 @@ public void initializeImpl(ConfigurableApplicationContext configurableApplicatio
             TestPropertyValues values = TestPropertyValues.of(
 
                 // These urls are from the perspective of cBioPortal application (port on host system).
-                String.format(
-                    "spring.security.oauth2.client.provider.keycloak.issuer=%s/realms/cbio",
-                    keycloakUrlForBrowser),
-                String.format(
-                    "spring.security.oauth2.client.provider.keycloak.issuer-uri=%s/realms/cbio",
-                    keycloakUrlForBrowser),
                 String.format(
                     "spring.security.oauth2.client.provider.keycloak.token-uri=%s/realms/cbio/protocol/openid-connect/token",
                     keycloakUrlForCBioportal),
@@ -45,6 +39,12 @@ public void initializeImpl(ConfigurableApplicationContext configurableApplicatio
                     keycloakUrlForCBioportal),
 
                 // This url is from the perspective of the browser.
+                String.format(
+                    "spring.security.oauth2.client.provider.keycloak.issuer=%s/realms/cbio",
+                    keycloakUrlForBrowser),
+                String.format(
+                    "spring.security.oauth2.client.provider.keycloak.issuer-uri=%s/realms/cbio",
+                    keycloakUrlForBrowser),
                 String.format(
                     "spring.security.oauth2.client.provider.keycloak.authorization-uri=%s/realms/cbio/protocol/openid-connect/auth",
                     keycloakUrlForBrowser),

diff --git a/src/test/java/org/cbioportal/test/integration/SharedChromeContainer.java b/src/test/java/org/cbioportal/test/integration/SharedChromeContainer.java
diff --git a/src/test/java/org/cbioportal/test/integration/SharedKeycloakContainer.java b/src/test/java/org/cbioportal/test/integration/SharedKeycloakContainer.java
diff --git a/src/test/java/org/cbioportal/test/integration/SharedMongoContainer.java b/src/test/java/org/cbioportal/test/integration/SharedMongoContainer.java
diff --git a/src/test/java/org/cbioportal/test/integration/SharedMysqlContainer.java b/src/test/java/org/cbioportal/test/integration/SharedMysqlContainer.java
diff --git a/src/test/java/org/cbioportal/test/integration/SharedSessionServiceContainer.java b/src/test/java/org/cbioportal/test/integration/SharedSessionServiceContainer.java
diff --git a/src/test/java/org/cbioportal/test/integration/security/AbstractContainerTest.java b/src/test/java/org/cbioportal/test/integration/security/AbstractContainerTest.java
@@ -5,6 +5,7 @@
 import org.cbioportal.test.integration.OAuth2KeycloakInitializer;
 import org.cbioportal.test.integration.SamlKeycloakInitializer;
 import org.openqa.selenium.chrome.ChromeOptions;
+import org.springframework.boot.test.util.TestPropertyValues;
 import org.springframework.boot.web.context.WebServerInitializedEvent;
 import org.springframework.context.ApplicationContextInitializer;
 import org.springframework.context.ApplicationListener;
@@ -68,17 +69,15 @@ public class AbstractContainerTest {
         sessionServiceContainer.setPortBindings(ImmutableList.of(String.format("%s:5000", SESSION_SERVICE_PORT)));
 
         mongoContainer = new GenericContainer(DockerImageName.parse(MONGO_IMAGE_VERSION))
-            .withEnv("MONGO_INITDB_DATABASE", "session_service")
-            .withReuse(true);
+            .withEnv("MONGO_INITDB_DATABASE", "session_service");
         mongoContainer.setPortBindings(ImmutableList.of(String.format("%s:27017", MONGO_PORT, MONGO_PORT)));
 
         keycloakContainer = new KeycloakContainer(KEYCLOAK_IMAGE_VERSION)
             .withRealmImportFile("security/keycloak-configuration-generated.json")
             .withAdminUsername("admin")
             .withAdminPassword("admin")
             .withEnv("KC_HOSTNAME", "host.testcontainers.internal")
-            .withEnv("KC_HOSTNAME_ADMIN", "localhost")
-            .withReuse(true);
+            .withEnv("KC_HOSTNAME_ADMIN", "localhost");
         keycloakContainer.setPortBindings(ImmutableList.of(String.format("%s:8080", KEYCLOAK_PORT)));
 
         mockServerContainer = new GenericContainer(MOCKSERVER_IMAGE_VERSION)
@@ -88,8 +87,7 @@ public class AbstractContainerTest {
         mysqlContainer = (MySQLContainer) new MySQLContainer(MYSQL_IMAGE_VERSION)
             .withClasspathResourceMapping("cgds.sql", "/docker-entrypoint-initdb.d/a_schema.sql", BindMode.READ_ONLY)
             .withClasspathResourceMapping("seed_mini.sql", "/docker-entrypoint-initdb.d/b_seed.sql", BindMode.READ_ONLY)
-            .withStartupTimeout(Duration.ofMinutes(10))
-            .withReuse(true);
+            .withStartupTimeout(Duration.ofMinutes(10));
         mysqlContainer.setPortBindings(ImmutableList.of(String.format("%s:3306", MYSQL_PORT)));
 
         ChromeOptions options = new ChromeOptions();
@@ -104,8 +102,7 @@ public class AbstractContainerTest {
             // activate this to record movies of the tests (greate for debugging)
             .withRecordingMode(RECORD_ALL, new File("/home/pnp300"))
             .withAccessToHost(true)
-            .withCapabilities(options)
-            .withReuse(true);
+            .withCapabilities(options);
 
         sessionServiceContainer.start();
         mongoContainer.start();
@@ -149,9 +146,13 @@ public static class PortInitializer implements
         ApplicationContextInitializer<ConfigurableApplicationContext> {
         @Override
         public void initialize(ConfigurableApplicationContext applicationContext) {
+            TestPropertyValues values = TestPropertyValues.of(
+                "server.port=" + CBIO_PORT
+            );
+            values.applyTo(applicationContext);
             applicationContext.addApplicationListener(
                 (ApplicationListener<WebServerInitializedEvent>) event -> {
-                    Testcontainers.exposeHostPorts(CBIO_PORT, KEYCLOAK_PORT, SESSION_SERVICE_PORT);
+                    Testcontainers.exposeHostPorts(CBIO_PORT, KEYCLOAK_PORT, MONGO_PORT);
                 });
         }
     }

diff --git a/src/test/java/org/cbioportal/test/integration/security/OAuth2AuthIntegrationTest.java b/src/test/java/org/cbioportal/test/integration/security/OAuth2AuthIntegrationTest.java
@@ -1,25 +1,18 @@
 package org.cbioportal.test.integration.security;
 
 import org.cbioportal.PortalApplication;
-import org.cbioportal.test.integration.MysqlInitializer;
-import org.cbioportal.test.integration.OAuth2KeycloakInitializer;
 import org.cbioportal.test.integration.security.util.Util;
 import org.junit.FixMethodOrder;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.junit.runners.MethodSorters;
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.web.context.WebServerInitializedEvent;
-import org.springframework.context.ApplicationContextInitializer;
-import org.springframework.context.ApplicationListener;
-import org.springframework.context.ConfigurableApplicationContext;
 import org.springframework.test.annotation.DirtiesContext;
 import org.springframework.test.context.ContextConfiguration;
 import org.springframework.test.context.TestPropertySource;
 import org.springframework.test.context.junit4.SpringRunner;
-import org.testcontainers.Testcontainers;
 
-import static org.cbioportal.test.integration.security.OAuth2AuthIntegrationTest.MyKeycloakInitializer;
+import static org.cbioportal.test.integration.security.AbstractContainerTest.*;
 
 @RunWith(SpringRunner.class)
 @SpringBootTest(
@@ -28,9 +21,6 @@
 )
 @TestPropertySource(
     properties = {
-        "app.name=cbioportal",
-        "server.port=8080",
-        "filter_groups_by_appname=false",
         "authenticate=oauth2",
         "dat.method=oauth2",
         // DB settings (also see MysqlInitializer)
@@ -56,51 +46,16 @@
     }
 )
 @ContextConfiguration(initializers = {
-    OAuth2AuthIntegrationTest.MyMysqlInitializer.class,
-    MyKeycloakInitializer.class,
-    OAuth2AuthIntegrationTest.PortInitializer.class
+    MyMysqlInitializer.class,
+    MyOAuth2KeycloakInitializer.class,
+    PortInitializer.class
 })
 @FixMethodOrder(MethodSorters.NAME_ASCENDING)
 @DirtiesContext // needed to reuse port 8080 for multiple tests
 public class OAuth2AuthIntegrationTest extends AbstractContainerTest {
 
-    private final static int CBIO_PORT = 8080;
-    private final static int KC_PORT = 8084;
-    private final static int SESSION_PORT = 27017;
     public final static String CBIO_URL_FROM_BROWSER =
         String.format("http://host.testcontainers.internal:%d", CBIO_PORT);
-
-    // Update application properties with connection info on Keycloak container
-    public static class MyKeycloakInitializer extends OAuth2KeycloakInitializer {
-        @Override
-        public void initialize(
-            ConfigurableApplicationContext configurableApplicationContext) {
-            super.initializeImpl(configurableApplicationContext, keycloakContainer);
-        }
-    }
-
-    // Update application properties with connection info on Mysql container
-    public static class MyMysqlInitializer extends MysqlInitializer {
-        @Override
-        public void initialize(
-            ConfigurableApplicationContext configurableApplicationContext) {
-            super.initializeImpl(configurableApplicationContext, mysqlContainer);
-        }
-    }
-
-    // Expose the ports for the cBioPortal Spring application and keycloak inside 
-    // the Chrome container. Each address is available on http://host.testcontainers.internal:<port>
-    // in the browser container.
-    public static class PortInitializer implements
-        ApplicationContextInitializer<ConfigurableApplicationContext> {
-        @Override
-        public void initialize(ConfigurableApplicationContext applicationContext) {
-            applicationContext.addApplicationListener(
-                (ApplicationListener<WebServerInitializedEvent>) event -> {
-                    Testcontainers.exposeHostPorts(CBIO_PORT, KC_PORT, SESSION_PORT);
-                });
-        }
-    }
 
     @Test
     public void a_loginSuccess() {