diff options
| author | kre <kre@NetBSD.org> | 2019-04-10 06:13:21 +0000 |
|---|---|---|
| committer | kre <kre@NetBSD.org> | 2019-04-10 06:13:21 +0000 |
| commit | c1630ced41319edf05e6dcc4cc7247fa85132d0c (patch) | |
| tree | 79cfd4e9e7723cc7f3da15f8262582016be19c44 /tests/dev | |
| parent | fe77336559b63365c7df8ce0bbf9128736b5aed7 (diff) | |
Fix quoting (quotes really do not nest...) and remove a bunch of it
that is harmless, but also pointless (in sh, quotes do not make strings,
everything is a string, rather they hide characters which would have
some other meaning unquoted (like spaces) - quotes are not needed around
strings like "descr" so remove them...
Be more consistent with line wrap style, try to avoid wrapping in the
middle of a (sh) word where possible. Avoid \ use when it is not needed.
Un-KNF (C style) - sh has no declarations, there is no need to leave
blank lines at the head of a function to mark the end of the declarations.
This should be a NFC - but the quoting really was broken before, just
was probably harmless breakage.
Diffstat (limited to 'tests/dev')
| -rw-r--r-- | tests/dev/cgd/t_cgd.sh | 64 |
1 files changed, 29 insertions, 35 deletions
diff --git a/tests/dev/cgd/t_cgd.sh b/tests/dev/cgd/t_cgd.sh index e3cc5b43528..b1ff53d316e 100644 --- a/tests/dev/cgd/t_cgd.sh +++ b/tests/dev/cgd/t_cgd.sh @@ -1,4 +1,4 @@ -# $NetBSD: t_cgd.sh,v 1.12 2019/04/10 06:09:39 kre Exp $ +# $NetBSD: t_cgd.sh,v 1.13 2019/04/10 06:13:21 kre Exp $ # # Copyright (c) 2010 The NetBSD Foundation, Inc. # All rights reserved. @@ -34,13 +34,12 @@ atf_test_case basic cleanup basic_head() { - atf_set "descr" "Tests that encrypt/decrypt works" - atf_set "require.progs" "rump_server" + atf_set descr "Tests that encrypt/decrypt works" + atf_set require.progs rump_server } basic_body() { - d=$(atf_get_srcdir) atf_check -s exit:0 \ ${cgdserver} -d key=/dev/dk,hostpath=dk.img,size=1m unix://csock @@ -57,7 +56,6 @@ basic_body() basic_cleanup() { - env RUMP_SERVER=unix://csock rump.halt || true } @@ -65,28 +63,27 @@ atf_test_case wrongpass cleanup wrongpass_head() { - atf_set "descr" "Tests that wrong password does not give original " \ - "plaintext" - atf_set "require.progs" "rump_server" + atf_set descr \ + "Tests that wrong password does not give original plaintext" + atf_set require.progs rump_server } wrongpass_body() { - d=$(atf_get_srcdir) atf_check -s exit:0 \ ${cgdserver} -d key=/dev/dk,hostpath=dk.img,size=1m unix://csock export RUMP_SERVER=unix://csock - atf_check -s exit:0 -x "echo 12345 | \ - rump.cgdconfig -p cgd0 /dev/dk ${d}/paramsfile" + atf_check -s exit:0 -x \ + "echo 12345 | rump.cgdconfig -p cgd0 /dev/dk ${d}/paramsfile" atf_check -s exit:0 -e ignore -x \ "dd if=${d}/t_cgd | rump.dd of=${rawcgd} count=2" # unconfig and reconfig cgd atf_check -s exit:0 rump.cgdconfig -u cgd0 - atf_check -s exit:0 -x "echo 54321 | \ - rump.cgdconfig -p cgd0 /dev/dk ${d}/paramsfile" + atf_check -s exit:0 -x \ + "echo 54321 | rump.cgdconfig -p cgd0 /dev/dk ${d}/paramsfile" atf_check -s exit:0 -e ignore dd if=${d}/t_cgd of=testfile count=2 atf_check -s exit:0 -e ignore -o not-file:testfile \ @@ -95,7 +92,6 @@ wrongpass_body() wrongpass_cleanup() { - env RUMP_SERVER=unix://csock rump.halt || true } @@ -103,9 +99,8 @@ wrongpass_cleanup() atf_test_case unaligned_write cleanup unaligned_write_head() { - - atf_set "descr" "Attempt unaligned writes to a raw cgd device" - atf_set "require.progs" "rump_server" + atf_set descr "Attempt unaligned writes to a raw cgd device" + atf_set require.progs rump_server } unaligned_write_body() @@ -115,8 +110,8 @@ unaligned_write_body() ${cgdserver} -d key=/dev/dk,hostpath=dk.img,size=1m unix://csock export RUMP_SERVER=unix://csock - atf_check -s exit:0 -x "echo 12345 | \ - rump.cgdconfig -p cgd0 /dev/dk ${d}/paramsfile" + atf_check -s exit:0 -x \ + "echo 12345 | rump.cgdconfig -p cgd0 /dev/dk ${d}/paramsfile" # Check that cgd rejects writes of totally bogus lengths. atf_check -s not-exit:0 -e ignore -x \ @@ -133,8 +128,8 @@ unaligned_write_body() # packetizing the input on bogus boundaries and using the # bizarre behaviour of `bs=N' in dd. atf_check -s not-exit:0 -e ignore -x \ - "(echo -n x && sleep 1 && head -c 511 </dev/zero) \ - | rump.dd of=${rawcgd} bs=512" + "(echo -n x && sleep 1 && head -c 511 </dev/zero) | + rump.dd of=${rawcgd} bs=512" # Check that cgd rejects sector-length writes if they are not # on sector boundaries. Doesn't work because dd can't be @@ -152,35 +147,35 @@ unaligned_write_cleanup() vmeth_failure_body() { - local vmeth="$1" local d=$(atf_get_srcdir) atf_check -s exit:0 \ ${cgdserver} -d key=/dev/dk,hostpath=dk.img,size=1m unix://csock export RUMP_SERVER=unix://csock - atf_check -s not-exit:0 -e ignore -x "echo 12345 | \ - rump.cgdconfig -V "${vmeth}" -p cgd0 /dev/dk ${d}/paramsfile" - atf_check -s exit:0 -o not-match:"(^| )cgd0( |$)" rump.sysctl -n hw.disknames + atf_check -s not-exit:0 -e ignore -x \ + "echo 12345 | + rump.cgdconfig -V '${vmeth}' -p cgd0 /dev/dk ${d}/paramsfile" + atf_check -s exit:0 -o not-match:'(^| )cgd0( |$)' \ + rump.sysctl -n hw.disknames } test_case_vmeth_failure() { - local vmeth="${1}" local name="vmeth_failure_${vmeth}" atf_test_case "${name}" cleanup - eval "${name}_head() { \ - atf_set "descr" "Tests verification method \"${vmeth}\" failure" ; \ - atf_set "require.progs" "rump_server" ; \ + eval "${name}_head() { + atf_set descr 'Tests verification method \"${vmeth}\" failure' + atf_set require.progs rump_server }" - eval "${name}_body() { \ - vmeth_failure_body "${vmeth}" ; \ + eval "${name}_body() { + vmeth_failure_body '${vmeth}' }" - eval "${name}_cleanup() { \ - rump.cgdconfig -u cgd0 2>/dev/null ; \ - env RUMP_SERVER=unix://csock rump.halt || true ; \ + eval "${name}_cleanup() { + rump.cgdconfig -u cgd0 2>/dev/null + env RUMP_SERVER=unix://csock rump.halt || true }" } @@ -191,7 +186,6 @@ test_case_vmeth_failure mbr atf_init_test_cases() { - atf_add_test_case basic atf_add_test_case wrongpass atf_add_test_case unaligned_write |
