Fix tests to work with embedded tomcat (#1779)

LabKey · Jan 10, 2024 · fbe9d7b · fbe9d7b
1 parent d7b97e8
commit fbe9d7b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 10 deletions.
diff --git a/src/org/labkey/test/tests/AuditLogTest.java b/src/org/labkey/test/tests/AuditLogTest.java
@@ -33,7 +33,6 @@
 import org.labkey.test.BaseWebDriverTest;
 import org.labkey.test.Locator;
 import org.labkey.test.TestFileUtils;
-import org.labkey.test.TestProperties;
 import org.labkey.test.TestTimeoutException;
 import org.labkey.test.WebTestHelper;
 import org.labkey.test.categories.Daily;
@@ -175,7 +174,7 @@ protected void turnOnAuditLogFile()
     protected ArrayList<String> getAuditLogFromFile() throws IOException
     {
         ArrayList<String> auditLog = new ArrayList<>();
-        File auditLogFile = new File(TestProperties.getTomcatHome(), "logs/labkey-audit.log");
+        File auditLogFile = new File(TestFileUtils.getServerLogDir(), "labkey-audit.log");
 
         try (FileReader fileReader = new FileReader(auditLogFile))
         {

diff --git a/src/org/labkey/test/tests/DatabaseDiagnosticsTest.java b/src/org/labkey/test/tests/DatabaseDiagnosticsTest.java
@@ -19,17 +19,17 @@
 import org.junit.experimental.categories.Category;
 import org.labkey.test.BaseWebDriverTest;
 import org.labkey.test.Locator;
-import org.labkey.test.TestProperties;
+import org.labkey.test.TestFileUtils;
 import org.labkey.test.WebTestHelper;
 import org.labkey.test.categories.BVT;
 import org.labkey.test.categories.CustomModules;
 import org.labkey.test.categories.Daily;
 import org.labkey.test.categories.Git;
 import org.labkey.test.io.Grep;
 import org.labkey.test.util.Maps;
+import org.labkey.test.util.Order;
 import org.labkey.test.util.PasswordUtil;
 import org.labkey.test.util.PipelineStatusTable;
-import org.labkey.test.util.Order;
 import org.openqa.selenium.WebElement;
 
 import java.io.File;
@@ -78,19 +78,16 @@ public void databaseCheckTest()
     @Test
     public void testTomcatLogs() throws Exception
     {
-        File tomcatHome = TestProperties.getTomcatHome();
-        assertTrue("Specified tomcat.home does not exist: " + tomcatHome +
-                "\nMake sure CATALINA_HOME is set or specify 'tomcat.home' when running tests",
-                tomcatHome != null && tomcatHome.exists());
-        File logDir = new File(tomcatHome, "logs");
+        File logDir = TestFileUtils.getServerLogDir();
+        assertTrue("Server log directory does not exist: " + logDir, logDir.isDirectory());
         File[] logs = logDir.listFiles();
         Map<File, Integer> contaminatedLogs = Grep.grep(PasswordUtil.getPassword(), logs);
         Map<String, String> failureFiles = new TreeMap<>();
         contaminatedLogs.keySet().forEach(
                 file -> failureFiles.put(file.getName(), "line " + contaminatedLogs.get(file)));
 
         assertTrue(String.format("These tomcat logs (in %s) contained unwanted text [%s]:\n%s",
-                tomcatHome.getAbsolutePath(), PasswordUtil.getPassword(), failureFiles.toString()),
+                logDir.getAbsolutePath(), PasswordUtil.getPassword(), failureFiles.toString()),
                 failureFiles.isEmpty());
     }