git: afdce0837014 - main - renice: Clean up the tests a bit.
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 20 Dec 2023 16:59:50 UTC
The branch main has been updated by des:
URL: https://cgit.FreeBSD.org/src/commit/?id=afdce0837014fb7a41fced959c9457f89e878052
commit afdce0837014fb7a41fced959c9457f89e878052
Author: Dag-Erling Smørgrav <des@FreeBSD.org>
AuthorDate: 2023-12-20 16:59:37 +0000
Commit: Dag-Erling Smørgrav <des@FreeBSD.org>
CommitDate: 2023-12-20 16:59:37 +0000
renice: Clean up the tests a bit.
MFC after: 1 week
Sponsored by: Klara, Inc.
Reviewed by: ngie, asomers
Differential Revision: https://reviews.freebsd.org/D43087
---
usr.bin/renice/tests/renice_test.sh | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/usr.bin/renice/tests/renice_test.sh b/usr.bin/renice/tests/renice_test.sh
index 8a4df7879ad4..7983eb594716 100755
--- a/usr.bin/renice/tests/renice_test.sh
+++ b/usr.bin/renice/tests/renice_test.sh
@@ -14,8 +14,9 @@ _renice() {
atf_check_nice_value() {
local pid=$1
- local nice=$2
- atf_check test "$(ps -o nice= -p $pid)" -eq "$nice"
+ local expected=$2
+ local actual="$(ps -o nice= -p $pid)"
+ atf_check test "$actual" -eq "$expected"
}
atf_test_case renice_abs_pid
@@ -57,7 +58,7 @@ renice_abs_pgid_head() {
renice_abs_pgid_body() {
local pid pgid nice incr
# make sure target runs in a different pgrp than ours
- pid=$(sh -mc "sleep 60 >/dev/null & echo \$!")
+ pid="$(sh -mc "sleep 60 >/dev/null & echo \$!")"
pgid="$(ps -o pgid= -p $pid)"
nice="$(ps -o nice= -p $pid)"
incr=3
@@ -73,7 +74,7 @@ renice_rel_pgid_head() {
renice_rel_pgid_body() {
local pid pgid nice incr
# make sure target runs in a different pgrp than ours
- pid=$(sh -mc "sleep 60 >/dev/null & echo \$!")
+ pid="$(sh -mc "sleep 60 >/dev/null & echo \$!")"
pgid="$(ps -o pgid= -p $pid)"
nice="$(ps -o nice= -p $pid)"
incr=3
@@ -90,7 +91,7 @@ renice_abs_user_head() {
}
renice_abs_user_body() {
local user pid nice incr
- pid=$(su -m $TEST_USER -c "/bin/sh -c 'sleep 60 >/dev/null & echo \$!'")
+ pid="$(su -m $TEST_USER -c "/bin/sh -c 'sleep 60 >/dev/null & echo \$!'")"
nice="$(ps -o nice= -p $pid)"
incr=3
_renice $((nice+incr)) -u $TEST_USER
@@ -105,7 +106,7 @@ renice_rel_user_head() {
}
renice_rel_user_body() {
local user pid nice incr
- pid=$(su -m $TEST_USER -c "/bin/sh -c 'sleep 60 >/dev/null & echo \$!'")
+ pid="$(su -m $TEST_USER -c "/bin/sh -c 'sleep 60 >/dev/null & echo \$!'")"
nice="$(ps -o nice= -p $pid)"
incr=3
_renice -u -n $incr $TEST_USER