ci: more e2e test fixes (#8881)

create_test_user was returning a nonsense username in some cases.
determined-ai · Feb 23, 2024 · 944732a · 944732a
1 parent ab9505c
commit 944732a
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/e2e_tests/tests/api_utils.py b/e2e_tests/tests/api_utils.py
@@ -46,11 +46,10 @@ def create_test_user(
     Returns a tuple of (Session, password).
     """
     session = admin_session()
-    username = get_random_string()
-    user = user or bindings.v1User(username=username, admin=False, active=True)
+    user = user or bindings.v1User(username=get_random_string(), admin=False, active=True)
     password = get_random_string()
     bindings.post_PostUser(session, body=bindings.v1PostUserRequest(user=user, password=password))
-    sess = make_session(username, password)
+    sess = make_session(user.username, password)
     return sess, password