diff --git a/tests/integration/conftest.py b/tests/integration/conftest.py index a0f283314..e776582c0 100644 --- a/tests/integration/conftest.py +++ b/tests/integration/conftest.py @@ -513,7 +513,7 @@ def pytest_configure(config): ) -@pytest.fixture +@pytest.fixture(scope="module") def support_test_linode_id(linode_cloud_firewall): timestamp = str(time.time_ns()) label = "cli" + timestamp diff --git a/tests/integration/placements/test_placements.py b/tests/integration/placements/test_placements.py index a05b97d97..d0f52ccee 100644 --- a/tests/integration/placements/test_placements.py +++ b/tests/integration/placements/test_placements.py @@ -11,7 +11,7 @@ BASE_CMD = ["linode-cli", "placement"] -@pytest.fixture +@pytest.fixture(scope="module") def create_placement_group(): new_label = str(time.time_ns()) + "label" placement_group_id = ( @@ -68,50 +68,40 @@ def test_placement_group_view(create_placement_group): assert_headers_in_lines(headers, lines) -@pytest.mark.skip(reason="BUG TPT-3109") def test_assign_placement_group(support_test_linode_id, create_placement_group): linode_id = support_test_linode_id placement_group_id = create_placement_group - res = ( - exec_test_command( - BASE_CMD - + [ - "assign-linode", - placement_group_id, - "--linodes", - linode_id, - "--text", - "--delimiter=,", - ] - ) - .stdout.decode() - .rstrip() + process = exec_test_command( + BASE_CMD + + [ + "assign-linode", + placement_group_id, + "--linode", + linode_id, + "--text", + "--delimiter=,", + ] ) - assert placement_group_id in res + assert process.returncode == 0 -@pytest.mark.skip(reason="BUG TPT-3109") def test_unassign_placement_group( support_test_linode_id, create_placement_group ): linode_id = support_test_linode_id placement_group_id = create_placement_group - res = ( - exec_test_command( - BASE_CMD - + [ - "unassign-linode", - placement_group_id, - "--linode", - linode_id, - "--text", - "--delimiter=,", - ] - ) - .stdout.decode() - .rstrip() + process = exec_test_command( + BASE_CMD + + [ + "unassign-linode", + placement_group_id, + "--linode", + linode_id, + "--text", + "--delimiter=,", + ] ) - assert placement_group_id not in res + assert process.returncode == 0 def test_update_placement_group(create_placement_group):