From bc7138eba77d53db6d1efae8ae4d7a22bf9f89ff Mon Sep 17 00:00:00 2001
From: labkey-jeckels <jeckels@labkey.com>
Date: Tue, 23 Sep 2025 16:38:43 -0700
Subject: [PATCH] Constants for many commonly used role names in tests

---
 .../test/tests/external/labModules/LabModulesTest.java       | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/LDK/test/src/org/labkey/test/tests/external/labModules/LabModulesTest.java b/LDK/test/src/org/labkey/test/tests/external/labModules/LabModulesTest.java
index 59b5d372..e51aa912 100644
--- a/LDK/test/src/org/labkey/test/tests/external/labModules/LabModulesTest.java
+++ b/LDK/test/src/org/labkey/test/tests/external/labModules/LabModulesTest.java
@@ -75,6 +75,7 @@
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
+import static org.labkey.test.util.PermissionsHelper.READER_ROLE;
 
 /**
  * Contains a series of tests designed to test the UI in the laboratory module.
@@ -634,7 +635,7 @@ private void overviewUITest()
         goToProjectHome();
 
         //verify import not visible to reader
-        impersonateRole("Reader");
+        impersonateRole(READER_ROLE);
         _helper.goToLabHome();
 
         waitForElement(LabModuleHelper.getNavPanelRow("Sequence:"), WAIT_FOR_PAGE);
@@ -1216,7 +1217,7 @@ private void labToolsWebpartTest()
         //verify settings hidden for readers
         Locator settings = _helper.toolIcon("Settings");
         assertElementPresent(settings);
-        impersonateRole("Reader");
+        impersonateRole(READER_ROLE);
         _helper.goToLabHome();
         assertElementNotPresent(settings);
         stopImpersonatingRole();