review of new/changed translatable program strings

[apt.git] / test / integration / test-acquire-same-file-multiple-times

diff --git a/test/integration/test-acquire-same-file-multiple-times b/test/integration/test-acquire-same-file-multiple-times
index 5267655215c92dd7f0b3f77976a4b2bed3296a12..354c683f2395bcef1d82bee13c2faec470837b7e 100755 (executable)
--- a/test/integration/test-acquire-same-file-multiple-times
+++ b/test/integration/test-acquire-same-file-multiple-times
@@ -2,41 +2,41 @@
 set -e
 
 TESTDIR=$(readlink -f $(dirname $0))
-. $TESTDIR/framework
+. "$TESTDIR/framework"
 setupenvironment
 configarchitecture 'amd64'
 
 TESTFILE="$TESTDIR/framework"
-cp $TESTFILE aptarchive/foo
+cp "$TESTFILE" aptarchive/foo
 APTARCHIVE="$(readlink -f ./aptarchive)"
 
 filedown() {
        msgtest 'Downloading the same URI twice over file' "$1"
-       testsuccess --nomsg apthelper download-file file:///$APTARCHIVE/foo ./downloaded/foo1 '' file:///$APTARCHIVE/foo ./downloaded/foo2 '' -o Debug::pkgAcquire::Worker=1
+       testsuccess --nomsg apthelper download-file "file:///$APTARCHIVE/foo" './downloaded/foo1' '' \
+               "file:///$APTARCHIVE/foo" './downloaded/foo2' '' -o Debug::pkgAcquire::Worker=1
        cp rootdir/tmp/testsuccess.output download.log
-       #cat download.log
-       testsuccess cmp $TESTFILE ./downloaded/foo1
+       testsuccess cmp "$TESTFILE" ./downloaded/foo1
        testsuccess cmp ./downloaded/foo1 ./downloaded/foo2
-       #testequal '1' grep -c '200%20URI%20Start' ./download.log
+       testequal '1' grep -c '200%20URI%20Start' ./download.log
        testequal '1' grep -c '201%20URI%20Done' ./download.log
        rm -f ./downloaded/foo1 ./downloaded/foo2
 }
 
 testrun() {
        $1 'no partial'
-       cp $TESTFILE ./downloaded/foo1
+       cp "$TESTFILE" ./downloaded/foo1
        $1 'complete partial 1'
-       cp $TESTFILE ./downloaded/foo2
+       cp "$TESTFILE" ./downloaded/foo2
        $1 'complete partial 2'
-       cp $TESTFILE ./downloaded/foo1
-       cp $TESTFILE ./downloaded/foo2
+       cp "$TESTFILE" ./downloaded/foo1
+       cp "$TESTFILE" ./downloaded/foo2
        $1 'complete partial 1+2'
-       dd if=$TESTFILE of=./downloaded/foo1 bs=500 count=1 2>/dev/null
+       dd if="$TESTFILE" of=./downloaded/foo1 bs=500 count=1 2>/dev/null
        $1 'partial partial 1'
-       dd if=$TESTFILE of=./downloaded/foo2 bs=500 count=1 2>/dev/null
+       dd if="$TESTFILE" of=./downloaded/foo2 bs=500 count=1 2>/dev/null
        $1 'partial partial 2'
-       dd if=$TESTFILE of=./downloaded/foo1 bs=500 count=1 2>/dev/null
-       dd if=$TESTFILE of=./downloaded/foo2 bs=500 count=1 2>/dev/null
+       dd if="$TESTFILE" of=./downloaded/foo1 bs=500 count=1 2>/dev/null
+       dd if="$TESTFILE" of=./downloaded/foo2 bs=500 count=1 2>/dev/null
        $1 'partial partial 1+2'
 }
 testrun 'filedown'
@@ -45,9 +45,10 @@ changetowebserver -o aptwebserver::redirect::replace::/foo2=/foo
 
 httpdown() {
        msgtest 'Downloading the same URI to different files' 'twice over http'
-       testsuccess --nomsg apthelper download-file http://localhost:8080/foo ./downloaded/foo1 '' http://localhost:8080/foo ./downloaded/foo2 '' -o Debug::pkgAcquire::Worker=1 -o Debug::Acquire::http=1
+       testsuccess --nomsg apthelper download-file "http://localhost:${APTHTTPPORT}/foo" "./downloaded/foo1" '' \
+               "http://localhost:${APTHTTPPORT}/foo" './downloaded/foo2' '' -o Debug::pkgAcquire::Worker=1 -o Debug::Acquire::http=1
        cp rootdir/tmp/testsuccess.output download.log
-       testsuccess cmp $TESTDIR/framework ./downloaded/foo1
+       testsuccess cmp "$TESTDIR/framework" ./downloaded/foo1
        testsuccess cmp ./downloaded/foo1 ./downloaded/foo2
        testequal '1' grep -c '200%20URI%20Start' ./download.log
        testequal '1' grep -c '201%20URI%20Done' ./download.log
@@ -57,9 +58,10 @@ testrun 'httpdown'
 
 httpredirectdown() {
        msgtest 'Redirect leads' 'first URI to the second URI'
-       testsuccess --nomsg apthelper download-file http://localhost:8080/foo2 ./downloaded/foo1 '' http://localhost:8080/foo ./downloaded/foo2 '' -o Debug::pkgAcquire::Worker=1 -o Debug::Acquire::http=1
+       testsuccess --nomsg apthelper download-file "http://localhost:${APTHTTPPORT}/foo2" "./downloaded/foo1" '' \
+               "http://localhost:${APTHTTPPORT}/foo" './downloaded/foo2' '' -o Debug::pkgAcquire::Worker=1 -o Debug::Acquire::http=1
        cp rootdir/tmp/testsuccess.output download.log
-       testsuccess cmp $TESTDIR/framework ./downloaded/foo1
+       testsuccess cmp "$TESTDIR/framework" ./downloaded/foo1
        testsuccess cmp ./downloaded/foo1 ./downloaded/foo2
        testequal '1' grep -c '200%20URI%20Start' ./download.log
        testequal '1' grep -c '103%20Redirect' ./download.log
@@ -70,9 +72,10 @@ testrun 'httpredirectdown'
 
 httpsamedown() {
        msgtest 'Downloading two files via the same URI to' 'the same file'
-       testsuccess --nomsg apthelper download-file http://localhost:8080/foo ./downloaded/foo1 '' http://localhost:8080/foo ./downloaded/foo1 '' -o Debug::pkgAcquire::Worker=1
+       testsuccess --nomsg apthelper download-file "http://localhost:${APTHTTPPORT}/foo" "./downloaded/foo1" '' \
+               "http://localhost:${APTHTTPPORT}/foo" './downloaded/foo1' '' -o Debug::pkgAcquire::Worker=1
        cp rootdir/tmp/testsuccess.output download.log
-       testsuccess cmp $TESTDIR/framework ./downloaded/foo1
+       testsuccess cmp "$TESTDIR/framework" ./downloaded/foo1
        testequal '1' grep -c '200%20URI%20Start' ./download.log
        testequal '1' grep -c '201%20URI%20Done' ./download.log
        rm -f ./downloaded/foo1