Fix newlines between white/blacklist test names in buildkite annotations (#1003)

1 files changed, 4 insertions, 4 deletions

diff --git a/show-expected-fail-tests.sh b/show-expected-fail-tests.sh
index 0a4c7be8..320d4ebd 100755
--- a/show-expected-fail-tests.sh
+++ b/show-expected-fail-tests.sh
@@ -80,8 +80,8 @@ done <<< "${passed_but_expected_fail}"
 # TODO: Check that the same test doesn't appear twice in the whitelist|blacklist
 
 # Trim test output strings
-tests_to_add=$(echo -e $tests_to_add | xargs -d '\n')
-already_in_whitelist=$(echo -e $already_in_whitelist | xargs -d '\n')
+tests_to_add=$(IFS=$'\n' echo "${tests_to_add[*]%%'\n'}")
+already_in_whitelist=$(IFS=$'\n' echo "${already_in_whitelist[*]%%'\n'}")
 
 # Format output with markdown for buildkite annotation rendering purposes
 if [ -n "${tests_to_add}" ] && [ -n "${already_in_whitelist}" ]; then
@@ -91,14 +91,14 @@ fi
 if [ -n "${tests_to_add}" ]; then
 	echo "**ERROR**: The following tests passed but are not present in \`$2\`. Please append them to the file:"
     echo "\`\`\`"
-	echo -e "${tests_to_add}"
+    echo -e "${tests_to_add}"
     echo "\`\`\`"
 fi
 
 if [ -n "${already_in_whitelist}" ]; then
 	echo "**WARN**: Tests in the whitelist still marked as **expected fail**:"
     echo "\`\`\`"
-	echo -e "${already_in_whitelist}"
+    echo -e "${already_in_whitelist}"
     echo "\`\`\`"
 fi