From d255f4d90d369487400c100a2bc8d39b69c6726d Mon Sep 17 00:00:00 2001 From: "Mark A. Grondona" Date: Tue, 29 Mar 2022 14:49:04 -0700 Subject: [PATCH] testsuite: t0007-ping.t: do not assume short hostnames Problem: A couple tests in `t0007-ping.t` assume short hostnames by using $(hostname -s) as an expected value. This causes failures when the domain is part of the hostname. Change uses of $(hostname -s) to just $(hostname) Fixes #4109 --- t/t0007-ping.t | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/t/t0007-ping.t b/t/t0007-ping.t index 9e6b3de4362d..77f578f56e9a 100755 --- a/t/t0007-ping.t +++ b/t/t0007-ping.t @@ -138,10 +138,10 @@ test_expect_success 'ping help output works' ' ' test_expect_success 'ping works with hostname' ' - flux ping --count=1 $(hostname -s) + flux ping --count=1 $(hostname) ' test_expect_success 'ping works with hostname!service' ' - flux ping --count=1 "$(hostname -s)!broker" + flux ping --count=1 "$(hostname)!broker" ' test_expect_success 'ping fails with unknown hostname' ' test_must_fail flux ping --count=1 "notmyhost!broker"