Skip to content

Commit

Permalink
STNG-165 Fix slow Lambda startup (#204)
Browse files Browse the repository at this point in the history
  • Loading branch information
jkosternl authored Oct 17, 2024
1 parent 83c310c commit e574443
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,8 @@ class SeleniumTest extends SeleniumTestBase {

@Test
void testLoginAndCreateSandboxStart() {
loginUser();
driver.get(baseUrl);
if (driver.getCurrentUrl().endsWith("/login")) {
loginUser();
}
assertEquals(baseUrl + "/environment", driver.getCurrentUrl());
WebElement createSandbox = driver.findElement(By.id("create_sandbox_button"));
createSandbox.click();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static org.junit.jupiter.api.Assertions.fail;
import static org.junit.jupiter.api.Assumptions.assumeFalse;

import java.time.Duration;
import java.util.List;
Expand Down Expand Up @@ -238,8 +237,9 @@ private void updateSandboxConfigBeforeStarting(SandboxConfig sandbox1, SandboxCo
}

protected void loginUser() {
assumeFalse(alreadyLoggedIn, "Already logged in, is fine.");
if (alreadyLoggedIn) return; // Already logged in, is fine.
driver.get(baseUrl);
waitForAsyncCalls(lambdaDelay); // First redirect might be slow
assertEquals(baseUrl + "/login", driver.getCurrentUrl());
assertEquals("DCSA Conformance", driver.getTitle().substring(0, 16));

Expand All @@ -251,6 +251,7 @@ protected void loginUser() {
log.info("Logging in user into environment: {}", baseUrl);
WebElement submitButton = driver.findElement(By.id("login_button"));
submitButton.click();
waitForAsyncCalls(lambdaDelay * 3); // First login is slow in AWS, so wait a bit longer.
waitForUIReadiness();

assertEquals("Sandboxes", driver.findElement(By.className("pageTitle")).getText());
Expand All @@ -259,12 +260,9 @@ protected void loginUser() {
}

SandboxConfig createSandBox(Standard standard, String version, String suiteName, String roleName, int sandboxType) {
loginUser();
log.info("Creating Sandbox: {}, {}, {}, {}, type: {}", standard.name(), version, suiteName, roleName, sandboxType);
driver.get(baseUrl + "/create-sandbox");
if (driver.getCurrentUrl().endsWith("/login")) {
loginUser();
driver.get(baseUrl + "/create-sandbox");
}
assertEquals(baseUrl + "/create-sandbox", driver.getCurrentUrl());
waitForUIReadiness();

Expand Down

0 comments on commit e574443

Please sign in to comment.