set tests to run in bash strict mode (#320)

apphost-framework-lookup/test.sh

@@ -8,6 +8,7 @@ fi
 # because that usually contains lots of "errors".
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 runtime_id="$(../runtime-id --portable)"

aspnet-same-runtime-version-2x/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 # There's always a version of Microsoft.AspNetCore.App for each

aspnet-same-runtime-version/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 
 # Check Microsoft.AspNetCore.App's version matches that of Microsoft.NETCore.App
 

aspnetpatch-21/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 # Ensure we have an up-to-date value for latest_aspnet_package.

bash-completion/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 
 DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )"
 

bundled-libunwind/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 framework_dir=$(../dotnet-directory --framework "$1")

cgroup-limit/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 runtime_id="$(../runtime-id)"

commit-ids-in-binaries/test.sh

@@ -12,7 +12,6 @@
 
 set -euo pipefail
 IFS=$'\n\t'
-
 set -x
 
 dotnet_home="$(dirname "$(readlink -f "$(command -v dotnet)")")"

createdump-aspnet/test.sh

@@ -2,7 +2,7 @@
 
 # Enable "unofficial bash strict mode"
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 IFS='.-' read -ra VERSION_SPLIT <<< "$1"

distribution-packages/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 sdk_version="$(dotnet --version)"

dotnet-info-commit-ids/test.sh

@@ -2,7 +2,7 @@
 
 # unofficial bash strict mode
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 dotnet --info

dotnet-monitor-works/test.sh

@@ -1,8 +1,8 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-set -x
 IFS=$'\n\t'
+set -x
 
 IFS='.-' read -ra VERSION_SPLIT <<< "$1" 
 dotnet tool update -g dotnet-monitor --version "${VERSION_SPLIT[0]}.*-*"

dotnet-trace/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 PROJNAME=dotnettrace

extract-bundle-dir/test.sh

@@ -7,6 +7,7 @@ if [ -f /etc/profile ]; then
 fi
 
 set -euo pipefail
+IFS=$'\n\t'
 
 # Verify DOTNET_BUNDLE_EXTRACT_BASE_DIR is set.
 if [[ "${DOTNET_BUNDLE_EXTRACT_BASE_DIR:-}" != "$HOME/.cache/dotnet_bundle_extract" ]]; then

fdd-no-nuget/test.sh

@@ -1,6 +1,8 @@
 #!/usr/bin/env bash
 
-set +x
+set -euo pipefail
+IFS=$'\n\t'
+set -x
 
 rm -rf ~/.nuget
 

helloworld/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 
 PROJNAME=helloworld
 

host-probes-rid-assets-legacy/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euox pipefail
+IFS=$'\n\t'
 
 # This test is testing the same functionality as host-probes-rid-assets
 # but when opting in to use the legacy graph (by setting System.Runtime.Loader.UseRidGraph=true).

host-probes-rid-assets/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euox pipefail
+IFS=$'\n\t'
 
 # The lib project packages a native library in a rid-specific folder.
 # Only one rid is considered per nuget package.

liblttng-ust_sys-sdt.h/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 RUNTIME_ID=$(../runtime-id)

libuv-kestrel-sample-app-2x/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 dotnet restore

lttng/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 
 SESSION_NAME=my-session
 TEST_FOLDER=/tmp/$SESSION_NAME

man-pages/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 
 helpPages=$(dotnet --help | grep -A 999 'SDK commands' | grep -E -B 999 'Common options|Additional commands' | awk 'NR>1 {print $1}' | head -n-2)
 

nativeaot/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 rm -rf bin obj

omnisharp/test.sh

@@ -2,7 +2,6 @@
 
 set -euo pipefail
 IFS=$'\n\t'
-
 set -x
 
 rm -rf workdir

openssl-alpn/test.sh

@@ -3,6 +3,7 @@
 # Make sure .NET Core has linked to SSL_*_alpn_* functions from OpenSSL
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 dotnet_dir="$(../dotnet-directory)"

pgo-dynamic/test.sh

@@ -3,7 +3,7 @@
 # Check if the .NET Runtime can use dynamic PGO at runtime
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 IFS='.-' read -ra VERSION <<< "$1"

publish-aspnet-selfcontained/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 # Clear nuget sources to use the bundled runtime.

publish-dotnet-selfcontained/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 # Clear nuget sources to use the bundled runtime.

publish-ready-to-run-linux/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 dotnet new console --no-restore

publish-ready-to-run/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 dotnet new console --no-restore

restore-with-rid/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 dotnet new console --no-restore

system-data-odbc/test.sh

@@ -1,7 +1,7 @@
 #!/bin/bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 # This test *must* be run as non-root; postgresql will refuse to start as root.

system-libunwind/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 framework_dir=$(../dotnet-directory --framework "$1")

system-openssl/test.sh

@@ -4,6 +4,7 @@
 # using OpenSSL via dlopen (which is more likely to fail at runtime).
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 dotnet_dir="$(../dotnet-directory)"

targeting-packs-bad-files/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 dotnet_dir=$(dirname "$(readlink -f "$(command -v dotnet)")")

telemetry-is-off-by-default/test-telemetry-tcpdump.sh

@@ -4,6 +4,7 @@
 
 set -euo pipefail
 set -x
+IFS=$'\n\t'
 
 # found via experimentation
 telemetry_host=dc.services.visualstudio.com

telemetry-is-off-by-default/test.sh

@@ -9,6 +9,7 @@
 # access.
 
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 no_server=("/nodeReuse:false" "/p:UseSharedCompilation=false" "/p:UseRazorBuildServer=false")

template-test/test.sh

@@ -4,7 +4,8 @@
 # dotnet new <template>
 
 set -euo pipefail
-#set -x
+IFS=$'\n\t'
+set -x
 
 # The list of templates in each version of .NET that we want to test.
 # If additional templates are found via `dotnet new --list`, this test

tool-dev-certs/test.sh

@@ -7,6 +7,7 @@ fi
 # Enable "unofficial strict mode" only after loading /etc/profile
 # because that usually contains lots of "errors".
 set -euo pipefail
+IFS=$'\n\t'
 set -x
 
 IFS='.-' read -ra VERSION_SPLIT <<< "$1"

use-current-runtime/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 PORTABLE_RID="$(../runtime-id --portable)"

workload/test.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-
+IFS=$'\n\t'
 set -x
 
 # In some environments, such as Github Actions, /tmp/ is strange and

xunit-smoketest/test.sh

@@ -1,6 +1,7 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
+IFS=$'\n\t'
 
 for TEST_RESULT in pass fail ;
 do