* [PATCH] selftests: hid: fix typo and exit code
@ 2024-11-26 13:58 Maximilian Heyne
2024-11-28 15:40 ` Benjamin Tissoires
0 siblings, 1 reply; 2+ messages in thread
From: Maximilian Heyne @ 2024-11-26 13:58 UTC (permalink / raw)
Cc: Maximilian Heyne, Jiri Kosina, Benjamin Tissoires, Shuah Khan,
Peter Hutterer, linux-input, linux-kselftest, linux-kernel
The correct exit code to mark a test as skipped is 4.
Fixes: ffb85d5c9e80 ("selftests: hid: import hid-tools hid-core tests")
Signed-off-by: Maximilian Heyne <mheyne@amazon.de>
---
.../testing/selftests/hid/run-hid-tools-tests.sh | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/tools/testing/selftests/hid/run-hid-tools-tests.sh b/tools/testing/selftests/hid/run-hid-tools-tests.sh
index bdae8464da865..af1682a53c27e 100755
--- a/tools/testing/selftests/hid/run-hid-tools-tests.sh
+++ b/tools/testing/selftests/hid/run-hid-tools-tests.sh
@@ -2,24 +2,26 @@
# SPDX-License-Identifier: GPL-2.0
# Runs tests for the HID subsystem
+KSELFTEST_SKIP_TEST=4
+
if ! command -v python3 > /dev/null 2>&1; then
echo "hid-tools: [SKIP] python3 not installed"
- exit 77
+ exit $KSELFTEST_SKIP_TEST
fi
if ! python3 -c "import pytest" > /dev/null 2>&1; then
- echo "hid: [SKIP/ pytest module not installed"
- exit 77
+ echo "hid: [SKIP] pytest module not installed"
+ exit $KSELFTEST_SKIP_TEST
fi
if ! python3 -c "import pytest_tap" > /dev/null 2>&1; then
- echo "hid: [SKIP/ pytest_tap module not installed"
- exit 77
+ echo "hid: [SKIP] pytest_tap module not installed"
+ exit $KSELFTEST_SKIP_TEST
fi
if ! python3 -c "import hidtools" > /dev/null 2>&1; then
- echo "hid: [SKIP/ hid-tools module not installed"
- exit 77
+ echo "hid: [SKIP] hid-tools module not installed"
+ exit $KSELFTEST_SKIP_TEST
fi
TARGET=${TARGET:=.}
--
2.40.1
Amazon Web Services Development Center Germany GmbH
Krausenstr. 38
10117 Berlin
Geschaeftsfuehrung: Christian Schlaeger, Jonathan Weiss
Eingetragen am Amtsgericht Charlottenburg unter HRB 257764 B
Sitz: Berlin
Ust-ID: DE 365 538 597
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] selftests: hid: fix typo and exit code
2024-11-26 13:58 [PATCH] selftests: hid: fix typo and exit code Maximilian Heyne
@ 2024-11-28 15:40 ` Benjamin Tissoires
0 siblings, 0 replies; 2+ messages in thread
From: Benjamin Tissoires @ 2024-11-28 15:40 UTC (permalink / raw)
To: Maximilian Heyne
Cc: Jiri Kosina, Shuah Khan, Peter Hutterer, linux-input,
linux-kselftest, linux-kernel
On Tue, 26 Nov 2024 13:58:50 +0000, Maximilian Heyne wrote:
> The correct exit code to mark a test as skipped is 4.
>
>
Applied to hid/hid.git (for-6.13/upstream-fixes), thanks!
[1/1] selftests: hid: fix typo and exit code
https://git.kernel.org/hid/hid/c/e8f34747bdde
Cheers,
--
Benjamin Tissoires <bentiss@kernel.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-11-28 15:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-26 13:58 [PATCH] selftests: hid: fix typo and exit code Maximilian Heyne
2024-11-28 15:40 ` Benjamin Tissoires
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox