From f99deb1e9438b02e22ce000ba8c0f19bfc189b2d Mon Sep 17 00:00:00 2001
From: Yu Watanabe <watanabe.yu+github@gmail.com>
Date: Thu, 16 Mar 2023 05:04:35 +0900
Subject: test: reindent testsuite-17.11.sh

---
 test/units/testsuite-17.11.sh | 71 ++++++++++++++++++++++---------------------
 1 file changed, 36 insertions(+), 35 deletions(-)

(limited to 'test')

diff --git a/test/units/testsuite-17.11.sh b/test/units/testsuite-17.11.sh
index 8bcf517ea1..3775496010 100755
--- a/test/units/testsuite-17.11.sh
+++ b/test/units/testsuite-17.11.sh
@@ -9,9 +9,9 @@ set -o pipefail
 . "$(dirname "$0")"/assert.sh
 
 cleanup() {
-        cd /
-        rm -rf "${workdir}"
-        workdir=
+    cd /
+    rm -rf "${workdir}"
+    workdir=
 }
 
 workdir="$(mktemp -d)"
@@ -23,35 +23,35 @@ rules=
 exp=
 err=
 next_test_number() {
-        : $((++test_number))
+    : $((++test_number))
 
-        local num_str
-        num_str=$(printf %05d "${test_number}")
+    local num_str
+    num_str=$(printf %05d "${test_number}")
 
-        rules="sample-${num_str}.rules"
-        exp="sample-${num_str}.exp"
-        err="sample-${num_str}.err"
+    rules="sample-${num_str}.rules"
+    exp="sample-${num_str}.exp"
+    err="sample-${num_str}.err"
 }
 
 assert_0() {
-        assert_rc 0 udevadm verify "$@"
-        next_test_number
+    assert_rc 0 udevadm verify "$@"
+    next_test_number
 }
 
 assert_1() {
-        if [ -f "${exp}" ]; then
-                set +e
-                udevadm verify "$@" 2>"${err}"
-                assert_eq "$?" 1
-                set -e
-                diff "${exp}" "${err}"
-        else
-                set +e
-                udevadm verify "$@"
-                assert_eq "$?" 1
-                set -e
-        fi
-        next_test_number
+    if [ -f "${exp}" ]; then
+        set +e
+        udevadm verify "$@" 2>"${err}"
+        assert_eq "$?" 1
+        set -e
+        diff "${exp}" "${err}"
+    else
+        set +e
+        udevadm verify "$@"
+        assert_eq "$?" 1
+        set -e
+    fi
+    next_test_number
 }
 
 assert_0 -h
@@ -100,9 +100,9 @@ echo "Failed to parse rules file ${rules}: No buffer space available" >"${exp}"
 assert_1 "${rules}"
 
 {
-  printf 'RUN+="/bin/true"%8175s\\\n' ' '
-  printf 'RUN+="/bin/false"%8174s\\\n' ' '
-  echo
+    printf 'RUN+="/bin/true"%8175s\\\n' ' '
+    printf 'RUN+="/bin/false"%8174s\\\n' ' '
+    echo
 } >"${rules}"
 assert_0 "${rules}"
 
@@ -122,16 +122,17 @@ EOF
 assert_1 "${rules}"
 
 test_syntax_error() {
-        local rule msg
-        rule="$1"; shift
-        msg="$1"; shift
+    local rule msg
 
-        printf '%s\n' "${rule}" >"${rules}"
-        cat >"${exp}" <<EOF
+    rule="$1"; shift
+    msg="$1"; shift
+
+    printf '%s\n' "${rule}" >"${rules}"
+    cat >"${exp}" <<EOF
 ${rules}:1 ${msg}
 ${rules}: udev rules check failed
 EOF
-        assert_1 "${rules}"
+    assert_1 "${rules}"
 }
 
 test_syntax_error '=' 'Invalid key/value pair, ignoring.'
@@ -243,11 +244,11 @@ test_syntax_error 'KERNEL=="", KERNEL=="?*", NAME="a"' 'conflicting match expres
 test_syntax_error 'KERNEL=="abc", KERNEL!="abc", NAME="b"' 'conflicting match expressions, the line takes no effect'
 # shellcheck disable=SC2016
 test_syntax_error 'ENV{DISKSEQ}=="?*", ENV{DEVTYPE}!="partition", ENV{DISKSEQ}!="?*" ENV{ID_IGNORE_DISKSEQ}!="1", SYMLINK+="disk/by-diskseq/$env{DISKSEQ}"' \
-        'conflicting match expressions, the line takes no effect'
+                  'conflicting match expressions, the line takes no effect'
 test_syntax_error 'KERNEL!="", KERNEL=="?*", NAME="a"' 'duplicate expressions'
 # shellcheck disable=SC2016
 test_syntax_error 'ENV{DISKSEQ}=="?*", ENV{DEVTYPE}!="partition", ENV{DISKSEQ}=="?*" ENV{ID_IGNORE_DISKSEQ}!="1", SYMLINK+="disk/by-diskseq/$env{DISKSEQ}"' \
-        'duplicate expressions'
+                  'duplicate expressions'
 
 echo 'GOTO="a"' >"${rules}"
 cat >"${exp}" <<EOF
-- 
cgit v1.2.1


From 7b4cae904b28449604b1387c84cd9f01907cad15 Mon Sep 17 00:00:00 2001
From: Yu Watanabe <watanabe.yu+github@gmail.com>
Date: Thu, 16 Mar 2023 05:25:39 +0900
Subject: test: add tests for summary output of udevadm verify

---
 test/units/testsuite-17.11.sh | 61 +++++++++++++++++++++++++++++++++++--------
 1 file changed, 50 insertions(+), 11 deletions(-)

(limited to 'test')

diff --git a/test/units/testsuite-17.11.sh b/test/units/testsuite-17.11.sh
index 3775496010..50e1820957 100755
--- a/test/units/testsuite-17.11.sh
+++ b/test/units/testsuite-17.11.sh
@@ -18,10 +18,24 @@ workdir="$(mktemp -d)"
 trap cleanup EXIT
 cd "${workdir}"
 
+cat >"${workdir}/default_output_1_success" <<EOF
+
+1 udev rules files have been checked.
+  Success: 1
+  Fail:    0
+EOF
+cat >"${workdir}/default_output_1_fail" <<EOF
+
+1 udev rules files have been checked.
+  Success: 0
+  Fail:    1
+EOF
+
 test_number=0
 rules=
 exp=
 err=
+out=
 next_test_number() {
     : $((++test_number))
 
@@ -31,29 +45,39 @@ next_test_number() {
     rules="sample-${num_str}.rules"
     exp="sample-${num_str}.exp"
     err="sample-${num_str}.err"
+    exo="sample-${num_str}.exo"
+    out="sample-${num_str}.out"
 }
 
 assert_0() {
-    assert_rc 0 udevadm verify "$@"
+    udevadm verify "$@" >"${out}"
+    if [ -f "${rules}" ]; then
+       diff -u "${workdir}/default_output_1_success" "${out}"
+    fi
+
     next_test_number
 }
 
 assert_1() {
+    set +e
+    udevadm verify "$@" >"${out}" 2>"${err}"
+    assert_eq "$?" 1
+    set -e
+
     if [ -f "${exp}" ]; then
-        set +e
-        udevadm verify "$@" 2>"${err}"
-        assert_eq "$?" 1
-        set -e
-        diff "${exp}" "${err}"
-    else
-        set +e
-        udevadm verify "$@"
-        assert_eq "$?" 1
-        set -e
+        diff -u "${exp}" "${err}"
+    fi
+
+    if [ -f "${exo}" ]; then
+        diff -u "${exo}" "${out}"
     fi
+
     next_test_number
 }
 
+# initialize variables
+next_test_number
+
 assert_0 -h
 assert_0 --help
 assert_0 -V
@@ -71,8 +95,15 @@ assert_1 --resolve-names
 # --resolve-names= takes "early" or "never"
 assert_1 --resolve-names=now
 # Failed to parse rules file .: Is a directory
+cp "${workdir}/default_output_1_fail" "${exo}"
 assert_1 .
 # Failed to parse rules file .: Is a directory
+cat >"${exo}" <<EOF
+
+3 udev rules files have been checked.
+  Success: 2
+  Fail:    1
+EOF
 assert_1 /dev/null . /dev/null
 
 rules_dir='etc/udev/rules.d'
@@ -97,6 +128,7 @@ assert_0 "${rules}"
 # Failed to parse rules file ${rules}: No buffer space available
 printf '%16384s\n' ' ' >"${rules}"
 echo "Failed to parse rules file ${rules}: No buffer space available" >"${exp}"
+cp "${workdir}/default_output_1_fail" "${exo}"
 assert_1 "${rules}"
 
 {
@@ -112,6 +144,7 @@ cat >"${exp}" <<EOF
 ${rules}:5 Line is too long, ignored
 ${rules}: udev rules check failed
 EOF
+cp "${workdir}/default_output_1_fail" "${exo}"
 assert_1 "${rules}"
 
 printf '\\\n' >"${rules}"
@@ -119,6 +152,7 @@ cat >"${exp}" <<EOF
 ${rules}:1 Unexpected EOF after line continuation, line ignored
 ${rules}: udev rules check failed
 EOF
+cp "${workdir}/default_output_1_fail" "${exo}"
 assert_1 "${rules}"
 
 test_syntax_error() {
@@ -132,6 +166,7 @@ test_syntax_error() {
 ${rules}:1 ${msg}
 ${rules}: udev rules check failed
 EOF
+    cp "${workdir}/default_output_1_fail" "${exo}"
     assert_1 "${rules}"
 }
 
@@ -256,6 +291,7 @@ ${rules}:1 GOTO="a" has no matching label, ignoring
 ${rules}:1 The line takes no effect any more, dropping
 ${rules}: udev rules check failed
 EOF
+cp "${workdir}/default_output_1_fail" "${exo}"
 assert_1 "${rules}"
 
 cat >"${rules}" <<'EOF'
@@ -273,6 +309,7 @@ cat >"${exp}" <<EOF
 ${rules}:3 LABEL="b" is unused.
 ${rules}: udev rules check failed
 EOF
+cp "${workdir}/default_output_1_fail" "${exo}"
 assert_1 "${rules}"
 
 cat >"${rules}" <<'EOF'
@@ -286,6 +323,7 @@ ${rules}:1 The line takes no effect any more, dropping
 ${rules}:2 LABEL="b" is unused.
 ${rules}: udev rules check failed
 EOF
+cp "${workdir}/default_output_1_fail" "${exo}"
 assert_1 "${rules}"
 
 cat >"${rules}" <<'EOF'
@@ -296,6 +334,7 @@ ${rules}:1 duplicate expressions
 ${rules}:1 conflicting match expressions, the line takes no effect
 ${rules}: udev rules check failed
 EOF
+cp "${workdir}/default_output_1_fail" "${exo}"
 assert_1 "${rules}"
 
 # udevadm verify --root
-- 
cgit v1.2.1