From ebd022ee425a231f2aba34acd3f457d1274f4c21 Mon Sep 17 00:00:00 2001 From: Lee Garrett Date: Fri, 16 May 2025 08:13:55 +0200 Subject: [PATCH] Mark integration tests as needs/root (#84487) For calling setup_test_user, which adds a user to the system (and thus requiring root): - become - become_su - become_sudo - slurp And also: - apt, which installs/removes system packages - deb822_repository, which adds/removes apt repos - hardware_facts calls losetup, which requires root - package_facts, which installs/removes system packages Co-authored-by: Lee Garrett --- test/integration/targets/apt/aliases | 1 + test/integration/targets/become/aliases | 1 + test/integration/targets/become_su/aliases | 1 + test/integration/targets/become_sudo/aliases | 1 + test/integration/targets/deb822_repository/aliases | 1 + test/integration/targets/hardware_facts/aliases | 1 + test/integration/targets/package_facts/aliases | 1 + test/integration/targets/slurp/aliases | 1 + 8 files changed, 8 insertions(+) diff --git a/test/integration/targets/apt/aliases b/test/integration/targets/apt/aliases index 20c870937ff..b2ff984db43 100644 --- a/test/integration/targets/apt/aliases +++ b/test/integration/targets/apt/aliases @@ -1,5 +1,6 @@ shippable/posix/group2 destructive +needs/root skip/freebsd skip/macos skip/rhel diff --git a/test/integration/targets/become/aliases b/test/integration/targets/become/aliases index 2d93d9aa49f..b9ad1736051 100644 --- a/test/integration/targets/become/aliases +++ b/test/integration/targets/become/aliases @@ -2,4 +2,5 @@ destructive shippable/posix/group1 context/target gather_facts/no +needs/root setup/always/setup_passlib_controller # required for setup_test_user diff --git a/test/integration/targets/become_su/aliases b/test/integration/targets/become_su/aliases index db83eef7a3c..6b92895399a 100644 --- a/test/integration/targets/become_su/aliases +++ b/test/integration/targets/become_su/aliases @@ -2,6 +2,7 @@ destructive shippable/posix/group1 context/target gather_facts/no +needs/root needs/target/setup_become_user_pair needs/target/setup_test_user setup/always/setup_passlib_controller # required for setup_test_user diff --git a/test/integration/targets/become_sudo/aliases b/test/integration/targets/become_sudo/aliases index e7a37d7ea85..15e60f1a4aa 100644 --- a/test/integration/targets/become_sudo/aliases +++ b/test/integration/targets/become_sudo/aliases @@ -2,6 +2,7 @@ destructive shippable/posix/group1 context/target gather_facts/no +needs/root needs/target/setup_become_user_pair needs/target/setup_test_user setup/always/setup_passlib_controller # required for setup_test_user diff --git a/test/integration/targets/deb822_repository/aliases b/test/integration/targets/deb822_repository/aliases index 34e2b54058f..493abef2838 100644 --- a/test/integration/targets/deb822_repository/aliases +++ b/test/integration/targets/deb822_repository/aliases @@ -1,4 +1,5 @@ destructive +needs/root shippable/posix/group1 skip/freebsd skip/osx diff --git a/test/integration/targets/hardware_facts/aliases b/test/integration/targets/hardware_facts/aliases index a08519ba3c1..e61e88c0bc5 100644 --- a/test/integration/targets/hardware_facts/aliases +++ b/test/integration/targets/hardware_facts/aliases @@ -1,4 +1,5 @@ destructive needs/privileged +needs/root shippable/posix/group4 context/controller diff --git a/test/integration/targets/package_facts/aliases b/test/integration/targets/package_facts/aliases index eedfe259b66..69072ba415f 100644 --- a/test/integration/targets/package_facts/aliases +++ b/test/integration/targets/package_facts/aliases @@ -1,3 +1,4 @@ destructive +needs/root shippable/posix/group2 skip/macos diff --git a/test/integration/targets/slurp/aliases b/test/integration/targets/slurp/aliases index 069c660c332..aa171eb4fce 100644 --- a/test/integration/targets/slurp/aliases +++ b/test/integration/targets/slurp/aliases @@ -1,3 +1,4 @@ shippable/posix/group1 destructive +needs/root setup/always/setup_passlib_controller # required for setup_test_user