Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 30 Aug 2024 17:24:34 GMT
From:      Christos Margiolis <christos@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 080c85127e3f - main - mixer(8) tests: Fix cleanup routine
Message-ID:  <202408301724.47UHOYKR061539@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by christos:

URL: https://cgit.FreeBSD.org/src/commit/?id=080c85127e3fba2c8cfb78cb75f7b306aee4028d

commit 080c85127e3fba2c8cfb78cb75f7b306aee4028d
Author:     Olivier Cochard-Labbé <olivier@freebsd.org>
AuthorDate: 2024-08-30 17:23:29 +0000
Commit:     Christos Margiolis <christos@FreeBSD.org>
CommitDate: 2024-08-30 17:23:29 +0000

    mixer(8) tests: Fix cleanup routine
    
    The tests correctly skip if no snd_dummy neither mixer is found, but the
    cleanup is still called with the skip condition, which fails if there is
    no mixer.
    
    MFC after:              2 days
    Reviewed by:            christos
    Differential Revision:  https://reviews.freebsd.org/D46491
---
 usr.sbin/mixer/tests/mixer_test.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/usr.sbin/mixer/tests/mixer_test.sh b/usr.sbin/mixer/tests/mixer_test.sh
index e1321ad9ee13..45003251c460 100755
--- a/usr.sbin/mixer/tests/mixer_test.sh
+++ b/usr.sbin/mixer/tests/mixer_test.sh
@@ -66,7 +66,8 @@ set_default()
 
 restore_default()
 {
-	test -r "test_mixer_deflt_unit" && mixer -d $(cat test_mixer_deflt_unit)
+	test -r "test_mixer_deflt_unit" &&
+	mixer -d $(cat test_mixer_deflt_unit) || true
 }
 
 atf_test_case o_flag cleanup



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202408301724.47UHOYKR061539>