Skip to content

Commit 480ba7a

Browse files
committed
Add limit-tests to jobs.py and debian_uefi_vm as a new marker
Signed-off-by: Andrii Sultanov <andriy.sultanov@vates.tech>
1 parent 85336d3 commit 480ba7a

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

jobs.py

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -446,6 +446,19 @@
446446
"nb_pools": 2,
447447
"params": {},
448448
"paths": ["tests/misc/test_pool.py"],
449+
},
450+
"limit-tests": {
451+
"description": "Tests verifying we can hit our supported limits",
452+
"requirements": [
453+
"1 XCP-ng host >= 8.2"
454+
],
455+
"nb_pools": 1,
456+
"params": {
457+
# The test does not work on Alpine because of how it handles
458+
# multiple interfaces on the same network, so use Debian instead
459+
"--vm": "single/debian_uefi_vm",
460+
},
461+
"paths": ["tests/limits/test_vif_limit.py"],
449462
}
450463
}
451464

@@ -637,7 +650,7 @@ def extract_tests(cmd):
637650

638651
print("*** Checking that all tests that use VMs have VM target markers (small_vm, etc.)... ", end="")
639652
tests_missing_vm_markers = extract_tests(
640-
["pytest", "--collect-only", "-q", "-m", "not no_vm and not (small_vm or multi_vm or big_vm)"]
653+
["pytest", "--collect-only", "-q", "-m", "not no_vm and not (small_vm or multi_vm or big_vm or debian_uefi_vm)"]
641654
)
642655
if tests_missing_vm_markers:
643656
error = True

pytest.ini

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ markers =
3030
small_vm: tests that it is enough to run just once, using the smallest possible VM.
3131
big_vm: tests that it would be good to run with a big VM.
3232
multi_vms: tests that it would be good to run on a variety of VMs (includes `small_vm` but excludes `big_vm`).
33+
debian_uefi_vm: tests that require a Debian UEFI VM
3334

3435
# * Other markers
3536
reboot: tests that reboot one or more hosts.

0 commit comments

Comments
 (0)