test: Remove test_max_cpus.py

test_max_cpus.py tests vCPUs are online only on a microVM without CPU templates, which is now covered as part of test_all_vcpus_online(). Remove the test and use the maximum number of vCPUs in the new tests. Signed-off-by: Takahiro Itazuri <[email protected]>
firecracker-microvm · Dec 10, 2024 · b8bd579 · b8bd579
1 parent 8ac9989
commit b8bd579
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 23 deletions.
diff --git a/...ion_tests/functional/test_cpu_multiple.py → ...egration_tests/functional/test_cpu_all.py b/...ion_tests/functional/test_cpu_multiple.py → ...egration_tests/functional/test_cpu_all.py
@@ -9,9 +9,15 @@
 are operating identically, except for the expected differences.
 """
 
+import pytest
+
 from framework.properties import global_props
 
+# Use the maximum number of vCPUs supported by Firecracker
+MAX_VCPUS = 32
+
 
+@pytest.mark.parametrize("vcpu_count", [MAX_VCPUS])
 def test_all_vcpus_online(uvm_any):
     """Check all vCPUs are online inside guest"""
     assert (
@@ -20,6 +26,7 @@ def test_all_vcpus_online(uvm_any):
     )
 
 
+@pytest.mark.parametrize("vcpu_count", [MAX_VCPUS])
 def test_all_vcpus_have_same_features(uvm_any):
     """
     Check all vCPUs have the same features inside guest.

diff --git a/tests/integration_tests/functional/test_max_vcpus.py b/tests/integration_tests/functional/test_max_vcpus.py