Remove a couple functions from test_db_util that talk to the Galaxy database directly.

These functions were just effectively ensuring a Galaxy user was created properly - but the framework does that and the UI login double does that.
This commit is contained in:
John Chilton
2022-09-20 18:25:47 -04:00
parent 8134c58f35
commit 860ab797a8
15 changed files with 0 additions and 75 deletions
-11
View File
@@ -192,17 +192,6 @@ def ga_refresh(obj):
install_session.refresh(obj)
def get_galaxy_private_role(user):
for role in user.all_roles():
if role.name == user.email and role.description == f"Private Role for {user.email}":
return role
raise AssertionError(f"Private role not found for user '{user.email}'")
def get_galaxy_user(email):
return ga_session.query(galaxy.model.User).filter(galaxy.model.User.table.c.email == email).first()
def get_repository_by_name_and_owner(name, owner_username, return_multiple=False):
owner = get_user_by_name(owner_username)
repository = (
@@ -20,11 +20,6 @@ class BasicToolShedFeatures(ShedTwillTestCase):
assert admin_user is not None, f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_private_role(admin_user)
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
def test_0005_ensure_repositories_and_categories_exist(self):
"""Create the 0000 category and upload the filtering repository to it, if necessary."""
@@ -19,9 +19,6 @@ class ToolWithToolDependencies(ShedTwillTestCase):
def test_0000_initiate_users(self):
"""Create necessary user accounts."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert admin_user is not None, f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -28,11 +28,6 @@ class ToolWithRepositoryDependencies(ShedTwillTestCase):
assert admin_user is not None, f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_private_role(admin_user)
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
def test_0005_ensure_repositories_and_categories_exist(self):
"""Create the 0020 category and any missing repositories."""
@@ -32,11 +32,6 @@ class RepositoryWithDependencyRevisions(ShedTwillTestCase):
assert admin_user is not None, f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_private_role(admin_user)
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
def test_0005_ensure_repositories_and_categories_exist(self):
"""Create the 0030 category and add repositories to it, if necessary."""
@@ -16,11 +16,6 @@ class TestFreebayesRepository(ShedTwillTestCase):
def test_0000_create_or_login_admin_user(self):
"""Create necessary user accounts and login as an admin user."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -27,11 +27,6 @@ class TestRepositoryDependencies(ShedTwillTestCase):
def test_0000_create_or_login_admin_user(self):
"""Create necessary user accounts and login as an admin user."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -25,11 +25,6 @@ class TestRepositoryDependencies(ShedTwillTestCase):
def test_0000_create_or_login_admin_user(self):
"""Create necessary user accounts and login as an admin user."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -21,11 +21,6 @@ class TestRepositoryDependencies(ShedTwillTestCase):
def test_0000_create_or_login_admin_user(self):
"""Create necessary user accounts and login as an admin user."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -41,11 +41,6 @@ class TestSimplePriorInstallation(ShedTwillTestCase):
def test_0000_initiate_users(self):
"""Create necessary user accounts."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -54,11 +54,6 @@ class TestSimplePriorInstallation(ShedTwillTestCase):
def test_0000_initiate_users(self):
"""Create necessary user accounts."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -20,11 +20,6 @@ class UninstallingAndReinstallingRepositories(ShedTwillTestCase):
assert admin_user is not None, f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_private_role(admin_user)
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
def test_0005_ensure_repositories_and_categories_exist(self):
"""Create the 0000 category and upload the filtering repository to the tool shed, if necessary."""
@@ -12,9 +12,6 @@ class UninstallingAndReinstallingRepositories(ShedTwillTestCase):
def test_0000_initiate_users(self):
"""Create necessary user accounts."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert admin_user is not None, f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -22,9 +22,6 @@ class UninstallingAndReinstallingRepositories(ShedTwillTestCase):
def test_0000_initiate_users(self):
"""Create necessary user accounts."""
self.galaxy_login(email=common.admin_email, username=common.admin_username)
admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert admin_user is not None, f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(admin_user)
self.login(email=common.test_user_1_email, username=common.test_user_1_name)
test_user_1 = self.test_db_util.get_user(common.test_user_1_email)
assert (
@@ -43,11 +43,6 @@ class TestUpdateManager(ShedTwillTestCase):
assert admin_user is not None, f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_private_role(admin_user)
self.galaxy_login(email=common.admin_email, username=common.admin_username)
galaxy_admin_user = self.test_db_util.get_galaxy_user(common.admin_email)
assert (
galaxy_admin_user is not None
), f"Problem retrieving user with email {common.admin_email} from the database"
self.test_db_util.get_galaxy_private_role(galaxy_admin_user)
def test_0005_create_filtering_repository(self):
"""Create and populate the filtering_1410 repository.