From 1f56331435101c9bb71dedf4a466b7562fb315d8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Vin=C3=ADcius=20Ferr=C3=A3o?=
 <2031761+viniciusferrao@users.noreply.github.com>
Date: Mon, 27 Jul 2026 15:22:12 -0300
Subject: [PATCH] test(xnba): cover conditional pxelinux warnings
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Vinícius Ferrão <2031761+viniciusferrao@users.noreply.github.com>
---
 xCAT-test/unit/xnba_semicolon.t | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/xCAT-test/unit/xnba_semicolon.t b/xCAT-test/unit/xnba_semicolon.t
index fdf998526..abcc8411e 100644
--- a/xCAT-test/unit/xnba_semicolon.t
+++ b/xCAT-test/unit/xnba_semicolon.t
@@ -34,4 +34,18 @@ like($src, qr/sub _use_efistub_for_uefi\b/, 'UEFI EFI-stub selection helper exis
 like($src, qr/sles\?11/, 'SLES 11 UEFI compatibility rule matches sle11 and sles11 images');
 like($src, qr/if \(_use_efistub_for_uefi\(\$kern\)\)/, 'UEFI boot path uses compatibility helper before direct EFI-stub boot');
 
+# pxelinux is only needed for multiboot, COMBOOT, and memdisk configurations.
+# Missing syslinux must not warn for ordinary direct-kernel nodeset requests.
+like($src, qr/sub _requires_pxelinux\b/, 'pxelinux requirement helper exists');
+like(
+    $src,
+    qr/\$::XNBA_pxelinux_required = 1 if \(_requires_pxelinux\(\$kern\)\)/,
+    'generated boot configuration records when pxelinux is required'
+);
+like(
+    $src,
+    qr/if \(\$::XNBA_pxelinux_required\) \{.*?Unable to find pxelinux\.0/s,
+    'missing pxelinux warning is limited to requests that generate a pxelinux chain'
+);
+
 done_testing();