Some small test refactors

linode · Oct 8, 2024 · 5c722f5 · 5c722f5
1 parent 601dc54
commit 5c722f5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/tests/integration/account/test_account.py b/tests/integration/account/test_account.py
@@ -232,7 +232,7 @@ def test_user_list():
 
 
 @pytest.fixture
-def get_user_id():
+def username():
     user_id = (
         exec_test_command(
             [
@@ -255,11 +255,10 @@ def get_user_id():
     yield first_id
 
 
-def test_user_view(get_user_id):
-    user_id = get_user_id
+def test_user_view(username: str):
     res = (
         exec_test_command(
-            ["linode-cli", "users", "view", user_id, "--text", "--delimiter=,"]
+            ["linode-cli", "users", "view", username, "--text", "--delimiter=,"]
         )
         .stdout.decode()
         .rstrip()

diff --git a/tests/integration/linodes/test_rebuild.py b/tests/integration/linodes/test_rebuild.py
@@ -27,15 +27,13 @@ def test_linode_id(linode_cloud_firewall):
 
 @pytest.mark.flaky(reruns=3, reruns_delay=2)
 def test_rebuild_fails_without_image(test_linode_id):
-    linode_id = test_linode_id
-
     result = exec_failing_test_command(
         BASE_CMD
         + [
             "rebuild",
             "--root_pass",
             DEFAULT_RANDOM_PASS,
-            linode_id,
+            test_linode_id,
             "--text",
             "--no-headers",
         ],