From 35c17a1e25ec24eb1f56e0a4a24703139a176823 Mon Sep 17 00:00:00 2001
From: Adam Korynta <adam@rmanet.com>
Date: Fri, 5 Dec 2025 13:56:35 -0800
Subject: [PATCH 1/5] fix jstl references for jakarta update

---
 gradle/libs.versions.toml                                    | 1 +
 opendcs-web-client/build.gradle                              | 1 +
 .../src/main/webapp/WEB-INF/common/scripts.jspf              | 5 +++--
 .../src/main/webapp/WEB-INF/common/theme-selection.jspf      | 2 +-
 4 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
index 85290695..42c3f64f 100644
--- a/gradle/libs.versions.toml
+++ b/gradle/libs.versions.toml
@@ -51,6 +51,7 @@ jersey-core = { module = "org.glassfish.jersey.core:jersey-client", version.ref
 jersey-media-multipart = { module = "org.glassfish.jersey.media:jersey-media-multipart", version.ref = "jersey" }
 
 jstl = { module = "jakarta.servlet.jsp.jstl:jakarta.servlet.jsp.jstl-api", version = "3.0.2"}
+jstl-impl = { module = "org.glassfish.web:jakarta.servlet.jsp.jstl", version = "3.0.1"}
 swagger-jaxrs2 = { module = "io.swagger.core.v3:swagger-jaxrs2-jakarta", version.ref = "swagger" }
 jackson = { module = "com.fasterxml.jackson.core:jackson-databind", version.ref = "jackson" }
 
diff --git a/opendcs-web-client/build.gradle b/opendcs-web-client/build.gradle
index f65002d5..33c982b2 100644
--- a/opendcs-web-client/build.gradle
+++ b/opendcs-web-client/build.gradle
@@ -31,6 +31,7 @@ dependencies {
     implementation(libs.auto.service)
     annotationProcessor(libs.auto.service)
     implementation(libs.jstl)
+    runtimeOnly(libs.jstl.impl)
     runtimeOnly(libs.slf4j.jdk14)
 
     local_js(project(":api-client-typescript"))
diff --git a/opendcs-web-client/src/main/webapp/WEB-INF/common/scripts.jspf b/opendcs-web-client/src/main/webapp/WEB-INF/common/scripts.jspf
index 10c3a580..f3ff656a 100644
--- a/opendcs-web-client/src/main/webapp/WEB-INF/common/scripts.jspf
+++ b/opendcs-web-client/src/main/webapp/WEB-INF/common/scripts.jspf
@@ -1,5 +1,5 @@
 <%@ page language="java" contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%>
-<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %>
+<%@ taglib prefix="c" uri="jakarta.tags.core" %>
 
 <%-- Core JS files --%>
 <script src="/webjars/jquery/jquery.min.js"></script>
@@ -27,6 +27,7 @@
 <script src="/webjars/datatables.net-select/js/dataTables.select.min.js"></script>
 <script src="/webjars/datatables.net-select-bs5/js/select.bootstrap5.min.js"></script>
 <%--  OpenDCS Web Files --%>
+<script src="../resources/js/main.js"></script>
 <script src="../resources/js/decodes.js"></script>
 <script src="../resources/js/lib/dom_utilities.js"></script>
 <script src="../resources/js/lib/object_utilities.js"></script>
@@ -40,6 +41,7 @@
 <%@include file="/WEB-INF/common/modals/waiting.jspf" %>
 <%@include file="/WEB-INF/common/modals/notification.jspf" %>
 <%@include file="/WEB-INF/common/modals/yesno.jspf" %>
+<%@include file="/WEB-INF/common/modals/organizations.jspf" %>
 
 <!-- API base -->
 <script>
@@ -49,4 +51,3 @@
 		window.API_URL = new URL(apiBasePath, origin).href;
 	})();
 </script>
-<script src="../resources/js/main.js"></script>
diff --git a/opendcs-web-client/src/main/webapp/WEB-INF/common/theme-selection.jspf b/opendcs-web-client/src/main/webapp/WEB-INF/common/theme-selection.jspf
index 9160c0ac..cd798760 100644
--- a/opendcs-web-client/src/main/webapp/WEB-INF/common/theme-selection.jspf
+++ b/opendcs-web-client/src/main/webapp/WEB-INF/common/theme-selection.jspf
@@ -1,4 +1,4 @@
-<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %>
+<%@ taglib prefix="c" uri="jakarta.tags.core" %>
 <li class="nav-item dropdown dropstart ">
     <button class="btn btn-link nav-link py-2 px-0 px-lg-2 dropdown-toggle d-flex align-items-center dropdown-item"
                 id="bd-theme-select"

From 7aabc1b1184d357a525dc9c4b42a01ff9e1f7a5c Mon Sep 17 00:00:00 2001
From: Adam Korynta <adam@rmanet.com>
Date: Mon, 8 Dec 2025 09:12:34 -0800
Subject: [PATCH 2/5] adding integration test

---
 gradle/libs.versions.toml                     |  5 +-
 opendcs-integration-test/build.gradle         |  6 ++
 .../odcsapi/gui/login/LoginPageTest.java      | 75 +++++++++++++++++++
 .../main/webapp/WEB-INF/common/scripts.jspf   |  1 -
 4 files changed, 84 insertions(+), 3 deletions(-)
 create mode 100644 opendcs-integration-test/src/test/java/org/opendcs/odcsapi/gui/login/LoginPageTest.java

diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
index 42c3f64f..51b12049 100644
--- a/gradle/libs.versions.toml
+++ b/gradle/libs.versions.toml
@@ -5,13 +5,13 @@ servlet-api = "6.1.0" #Updating this further will require a change to the jakart
 slf4j = { strictly = "2.0.17" }
 jersey = "4.0.0"
 
-swagger = "2.2.28"
+swagger = "2.2.41"
 swagger-ui = "5.18.3"
 swagger-parser = "2.1.25"
 rs-api = "4.0.0"
 jackson = "2.20.1"
 jackson-annotations = "2.20"
-selenium = "3.141.59"
+selenium = "4.38.0"
 nimbus = "9.41.1"
 jwt = "4.4.0"
 auto-service = "1.1.1"
@@ -67,6 +67,7 @@ jakarta-xml-bind-api = { module = "jakarta.xml.bind:jakarta.xml.bind-api", versi
 org-glassfish-jaxb-runtime = {module = "org.glassfish.jaxb:jaxb-runtime", version.ref = "jaxb" }
 
 selenium = { module = "org.seleniumhq.selenium:selenium-java", version.ref = "selenium" }
+web-driver-manager = { module = "io.github.bonigarcia:webdrivermanager", version = "6.3.3" }
 nimbus = { module = "com.nimbusds:nimbus-jose-jwt", version.ref = "nimbus" }
 jwt = { module = "com.auth0:java-jwt", version.ref = "jwt" }
 auto-service = { module = "com.google.auto.service:auto-service", version.ref = "auto-service" }
diff --git a/opendcs-integration-test/build.gradle b/opendcs-integration-test/build.gradle
index 73e552a5..9f538d69 100644
--- a/opendcs-integration-test/build.gradle
+++ b/opendcs-integration-test/build.gradle
@@ -40,6 +40,8 @@ dependencies {
     testImplementation(libs.nimbus)
     testImplementation(libs.jwt)
     testImplementation(libs.swagger.parser)
+    testImplementation(libs.selenium)
+    testImplementation(libs.web.driver.manager)
     testRuntimeOnly(libs.byte.buddy)
     testImplementation enforcedPlatform(libs.junit.bom)
     testImplementation(libs.bundles.junit)
@@ -61,6 +63,10 @@ configurations.all {
     exclude group: "javax.media"
     exclude group: "com.sun.media"
     exclude group: "org.eclipse.jetty"
+    exclude group: "javax.xml.bind", module: "jaxb-api"
+    exclude group: "javax.transaction", module: "jta"
+    exclude group: "javax.xml.stream", module: "stax-api"
+
 }
 
 
diff --git a/opendcs-integration-test/src/test/java/org/opendcs/odcsapi/gui/login/LoginPageTest.java b/opendcs-integration-test/src/test/java/org/opendcs/odcsapi/gui/login/LoginPageTest.java
new file mode 100644
index 00000000..a9b014a8
--- /dev/null
+++ b/opendcs-integration-test/src/test/java/org/opendcs/odcsapi/gui/login/LoginPageTest.java
@@ -0,0 +1,75 @@
+/*
+ *  Copyright 2025 OpenDCS Consortium and its Contributors
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License")
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ */
+
+package org.opendcs.odcsapi.gui.login;
+
+import java.time.Duration;
+
+import io.github.bonigarcia.wdm.WebDriverManager;
+import io.restassured.RestAssured;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Tag;
+import org.junit.jupiter.api.TestTemplate;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.opendcs.odcsapi.fixtures.DatabaseContextProvider;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.chrome.ChromeDriver;
+import org.openqa.selenium.chrome.ChromeOptions;
+import org.openqa.selenium.support.ui.WebDriverWait;
+
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+@Tag("integration")
+@ExtendWith(DatabaseContextProvider.class)
+final class LoginPageTest
+{
+
+	private WebDriver driver;
+	private WebDriverWait wait;
+
+	@BeforeEach
+	public void setUp()
+	{
+		WebDriverManager.chromedriver().setup();
+		ChromeOptions options = new ChromeOptions();
+		options.addArguments("--headless=new");
+		options.addArguments("--disable-gpu");
+		options.addArguments("--no-sandbox");
+		options.addArguments("--disable-dev-shm-usage");
+		options.addArguments("--window-size=1920,1080");
+		driver = new ChromeDriver(options);
+		wait = new WebDriverWait(driver, Duration.ofSeconds(10));
+	}
+
+	@AfterEach
+	public void tearDown()
+	{
+		if(driver != null)
+		{
+			driver.quit();
+		}
+	}
+
+	@TestTemplate
+	public void testInvalidLoginShowsError()
+	{
+		driver.get(RestAssured.baseURI + ":" + RestAssured.port + "/" + "login");
+		String pageTitle = driver.getTitle();
+		String pageSource = driver.getPageSource();
+		assertTrue(pageTitle.contains("Login"), "The valid login page content was not found");
+		assertTrue(pageSource.contains("OpenDCS Web Client"), "The valid login page content was not found");
+	}
+}
\ No newline at end of file
diff --git a/opendcs-web-client/src/main/webapp/WEB-INF/common/scripts.jspf b/opendcs-web-client/src/main/webapp/WEB-INF/common/scripts.jspf
index f3ff656a..ce553c35 100644
--- a/opendcs-web-client/src/main/webapp/WEB-INF/common/scripts.jspf
+++ b/opendcs-web-client/src/main/webapp/WEB-INF/common/scripts.jspf
@@ -41,7 +41,6 @@
 <%@include file="/WEB-INF/common/modals/waiting.jspf" %>
 <%@include file="/WEB-INF/common/modals/notification.jspf" %>
 <%@include file="/WEB-INF/common/modals/yesno.jspf" %>
-<%@include file="/WEB-INF/common/modals/organizations.jspf" %>
 
 <!-- API base -->
 <script>

From b1b9829b10c8c55130b9827f6e008cbaeafc53bf Mon Sep 17 00:00:00 2001
From: Adam Korynta <adam@rmanet.com>
Date: Mon, 8 Dec 2025 10:36:52 -0800
Subject: [PATCH 3/5] remove classpath scanning for web gui. don't want to pick
 up rest api

---
 .../opendcs/odcsapi/fixtures/TomcatServer.java   | 16 ++++++----------
 .../opendcs/odcsapi/gui/login/LoginPageTest.java |  5 ++---
 2 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/opendcs-integration-test/src/main/java/org/opendcs/odcsapi/fixtures/TomcatServer.java b/opendcs-integration-test/src/main/java/org/opendcs/odcsapi/fixtures/TomcatServer.java
index 21725aa7..56bdd8c9 100644
--- a/opendcs-integration-test/src/main/java/org/opendcs/odcsapi/fixtures/TomcatServer.java
+++ b/opendcs-integration-test/src/main/java/org/opendcs/odcsapi/fixtures/TomcatServer.java
@@ -28,6 +28,7 @@
 import java.util.ServiceLoader;
 import java.util.function.Supplier;
 
+import decodes.util.DecodesSettings;
 import org.apache.catalina.Host;
 import org.apache.catalina.LifecycleException;
 import org.apache.catalina.Manager;
@@ -35,16 +36,14 @@
 import org.apache.catalina.core.StandardContext;
 import org.apache.catalina.startup.Tomcat;
 import org.apache.tomcat.jdbc.pool.DataSourceFactory;
+import org.apache.tomcat.util.scan.StandardJarScanner;
 import org.opendcs.database.api.DataTransaction;
 import org.opendcs.database.api.OpenDcsDataException;
 import org.opendcs.fixtures.configurations.cwms.CwmsOracleConfiguration;
 import org.opendcs.fixtures.spi.Configuration;
 import org.opendcs.fixtures.spi.ConfigurationProvider;
-import org.slf4j.Logger;
-
-import decodes.util.DecodesSettings;
-
 import org.opendcs.utils.logging.OpenDcsLoggerFactory;
+import org.slf4j.Logger;
 import uk.org.webcompere.systemstubs.environment.EnvironmentVariables;
 import uk.org.webcompere.systemstubs.properties.SystemProperties;
 import uk.org.webcompere.systemstubs.security.SystemExit;
@@ -96,16 +95,13 @@ public TomcatServer(String baseDir, int port, String restWar, String guiWar) thr
 		restApiContext.setParentClassLoader(TomcatServer.class.getClassLoader());
 		restApiContext.setReloadable(true);
 		restApiContext.setPrivileged(true);
+		((StandardJarScanner) restApiContext.getJarScanner()).setScanClassPath(false);
 		sessionManager = restApiContext::getManager;
-		if(System.getProperty(DB_OFFICE) != null)
-		{
-			restApiContext.removeParameter("opendcs.rest.api.cwms.office");
-			restApiContext.addParameter("opendcs.rest.api.cwms.office", System.getProperty(DB_OFFICE));
-		}
 
 		StandardContext guiContext = (StandardContext) tomcatInstance.addWebapp("", guiWar);
+		((StandardJarScanner) guiContext.getJarScanner()).setScanClassPath(false);
 		guiContext.setDelegate(true);
-		guiContext.setParentClassLoader(TomcatServer.class.getClassLoader());
+		guiContext.setParentClassLoader(null);
 		guiContext.setReloadable(true);
 		guiContext.setPrivileged(true);
 	}
diff --git a/opendcs-integration-test/src/test/java/org/opendcs/odcsapi/gui/login/LoginPageTest.java b/opendcs-integration-test/src/test/java/org/opendcs/odcsapi/gui/login/LoginPageTest.java
index a9b014a8..e117b7ec 100644
--- a/opendcs-integration-test/src/test/java/org/opendcs/odcsapi/gui/login/LoginPageTest.java
+++ b/opendcs-integration-test/src/test/java/org/opendcs/odcsapi/gui/login/LoginPageTest.java
@@ -67,9 +67,8 @@ public void tearDown()
 	public void testInvalidLoginShowsError()
 	{
 		driver.get(RestAssured.baseURI + ":" + RestAssured.port + "/" + "login");
-		String pageTitle = driver.getTitle();
 		String pageSource = driver.getPageSource();
-		assertTrue(pageTitle.contains("Login"), "The valid login page content was not found");
-		assertTrue(pageSource.contains("OpenDCS Web Client"), "The valid login page content was not found");
+		assertTrue(pageSource.contains("Login"), "The valid login page content was not found");
+		assertTrue(pageSource.contains("Don't have an account?"), "The valid login page content was not found");
 	}
 }
\ No newline at end of file

From f5ec60d6f5d99ab8fedb4cb47f53049fb877fa95 Mon Sep 17 00:00:00 2001
From: Adam Korynta <adam@rmanet.com>
Date: Mon, 8 Dec 2025 10:48:19 -0800
Subject: [PATCH 4/5] fix transitive non-jakarta swagger deps

---
 gradle/libs.versions.toml             | 1 +
 opendcs-integration-test/build.gradle | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
index 51b12049..37f1d1f5 100644
--- a/gradle/libs.versions.toml
+++ b/gradle/libs.versions.toml
@@ -53,6 +53,7 @@ jersey-media-multipart = { module = "org.glassfish.jersey.media:jersey-media-mul
 jstl = { module = "jakarta.servlet.jsp.jstl:jakarta.servlet.jsp.jstl-api", version = "3.0.2"}
 jstl-impl = { module = "org.glassfish.web:jakarta.servlet.jsp.jstl", version = "3.0.1"}
 swagger-jaxrs2 = { module = "io.swagger.core.v3:swagger-jaxrs2-jakarta", version.ref = "swagger" }
+swagger-models = { module = "io.swagger.core.v3:swagger-models-jakarta", version.ref = "swagger" }
 jackson = { module = "com.fasterxml.jackson.core:jackson-databind", version.ref = "jackson" }
 
 swagger-codegen = { module = "io.swagger.codegen.v3:swagger-codegen-cli", version.ref = "swagger-codegen" }
diff --git a/opendcs-integration-test/build.gradle b/opendcs-integration-test/build.gradle
index 9f538d69..d9f401e2 100644
--- a/opendcs-integration-test/build.gradle
+++ b/opendcs-integration-test/build.gradle
@@ -40,6 +40,7 @@ dependencies {
     testImplementation(libs.nimbus)
     testImplementation(libs.jwt)
     testImplementation(libs.swagger.parser)
+    testImplementation(libs.swagger.models)
     testImplementation(libs.selenium)
     testImplementation(libs.web.driver.manager)
     testRuntimeOnly(libs.byte.buddy)
@@ -66,6 +67,8 @@ configurations.all {
     exclude group: "javax.xml.bind", module: "jaxb-api"
     exclude group: "javax.transaction", module: "jta"
     exclude group: "javax.xml.stream", module: "stax-api"
+    exclude group: "io.swagger.core.v3", module: "swagger-core"
+    exclude group: "io.swagger.core.v3", module: "swagger-models"
 
 }
 

From 11b64fda4b6e5c3fb0ad8285fd2c1e08085ed5b7 Mon Sep 17 00:00:00 2001
From: Adam Korynta <adam@rmanet.com>
Date: Mon, 8 Dec 2025 11:55:42 -0800
Subject: [PATCH 5/5] fix compiler error

---
 opendcs-web-client/src/test/java/selenium/LoginTest.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/opendcs-web-client/src/test/java/selenium/LoginTest.java b/opendcs-web-client/src/test/java/selenium/LoginTest.java
index 5fec1305..f71cd3ad 100644
--- a/opendcs-web-client/src/test/java/selenium/LoginTest.java
+++ b/opendcs-web-client/src/test/java/selenium/LoginTest.java
@@ -9,6 +9,7 @@
 import org.openqa.selenium.chrome.ChromeDriver;
 import org.openqa.selenium.firefox.FirefoxDriver;
 
+import java.time.Duration;
 import java.util.Arrays;
 import java.util.List;
 import java.util.concurrent.TimeUnit;
@@ -44,7 +45,7 @@ else if (browser.equalsIgnoreCase("chrome"))
             }
 
             // Set implicit wait to handle any delays in finding elements
-            driver.manage().timeouts().implicitlyWait(5, TimeUnit.SECONDS);
+            driver.manage().timeouts().implicitlyWait(Duration.ofSeconds(5));
 
             // Open the login page
             String baseUrl = System.getProperty("opendcs.webclient.url") + "/portal/login"; // Change to your actual URL