From 9ac0c8f505e4095a1943b544395ed0805d949315 Mon Sep 17 00:00:00 2001
From: Glenn Jackman <jackman@pythian.com>
Date: Fri, 16 Aug 2019 08:00:00 -0400
Subject: [PATCH] force students to handle positional parameters safely

---
 exercises/acronym/.meta/version         |  2 +-
 exercises/acronym/acronym_test.sh       |  7 +++++++
 exercises/acronym/example.sh            |  5 ++---
 exercises/pig-latin/.meta/version       |  2 +-
 exercises/pig-latin/example.sh          |  2 +-
 exercises/pig-latin/pig_latin_test.sh   |  8 ++++++++
 exercises/proverb/.meta/version         |  2 +-
 exercises/proverb/proverb_test.sh       | 24 ++++++++++++++++++++++++
 exercises/word-count/.meta/version      |  2 +-
 exercises/word-count/example.sh         |  2 +-
 exercises/word-count/word_count_test.sh |  9 +++++++++
 11 files changed, 56 insertions(+), 9 deletions(-)

diff --git a/exercises/acronym/.meta/version b/exercises/acronym/.meta/version
index bd8bf882..943f9cbc 100644
--- a/exercises/acronym/.meta/version
+++ b/exercises/acronym/.meta/version
@@ -1 +1 @@
-1.7.0
+1.7.1
diff --git a/exercises/acronym/acronym_test.sh b/exercises/acronym/acronym_test.sh
index 388d39f6..d2dd2370 100644
--- a/exercises/acronym/acronym_test.sh
+++ b/exercises/acronym/acronym_test.sh
@@ -62,3 +62,10 @@
   [[ "$status" -eq 0 ]]
   [[ "$output" == "TRNT" ]]
 }
+
+@test "contains shell globbing character" {
+  [[ $BATS_RUN_SKIPPED == true  ]] || skip
+  run bash acronym.sh "Two * Words"
+  [[ "$status" -eq 0 ]]
+  [[ "$output" == "TW" ]]
+}
diff --git a/exercises/acronym/example.sh b/exercises/acronym/example.sh
index f8013b30..2ff87558 100644
--- a/exercises/acronym/example.sh
+++ b/exercises/acronym/example.sh
@@ -5,9 +5,8 @@ if [ "$#" -ne 1 ]; then
   exit 1
 fi
 
-phrase=$(echo "$1" | tr '-' ' ')
-set -f -- junk $phrase
-shift
+phrase=$(echo "$1" | tr -s '*-' ' ')
+set -f -- $phrase
 
 shopt -s extglob
 for word; do
diff --git a/exercises/pig-latin/.meta/version b/exercises/pig-latin/.meta/version
index 26aaba0e..6085e946 100644
--- a/exercises/pig-latin/.meta/version
+++ b/exercises/pig-latin/.meta/version
@@ -1 +1 @@
-1.2.0
+1.2.1
diff --git a/exercises/pig-latin/example.sh b/exercises/pig-latin/example.sh
index 6ed28e15..04e2ff18 100644
--- a/exercises/pig-latin/example.sh
+++ b/exercises/pig-latin/example.sh
@@ -19,4 +19,4 @@ translate() {
     fi
 }
 
-main "$@"
\ No newline at end of file
+main "$@"
diff --git a/exercises/pig-latin/pig_latin_test.sh b/exercises/pig-latin/pig_latin_test.sh
index e64bf4b0..4e5d9640 100644
--- a/exercises/pig-latin/pig_latin_test.sh
+++ b/exercises/pig-latin/pig_latin_test.sh
@@ -157,3 +157,11 @@
     [[ $status -eq 0 ]]
     [[ $output == "ickquay astfay unray" ]]
 }
+
+# danger this way lies
+@test "shell globbing" {
+    [[ $BATS_RUN_SKIPPED == true  ]] || skip
+    run bash pig_latin.sh "pig*"
+    [[ $status -eq 0 ]]
+    [[ $output == "ig*pay" ]]
+}
diff --git a/exercises/proverb/.meta/version b/exercises/proverb/.meta/version
index 9084fa2f..524cb552 100644
--- a/exercises/proverb/.meta/version
+++ b/exercises/proverb/.meta/version
@@ -1 +1 @@
-1.1.0
+1.1.1
diff --git a/exercises/proverb/proverb_test.sh b/exercises/proverb/proverb_test.sh
index df12f997..1a47807a 100644
--- a/exercises/proverb/proverb_test.sh
+++ b/exercises/proverb/proverb_test.sh
@@ -75,3 +75,27 @@ END
     [[ $status -eq 0 ]]
     [[ $output == "$expected" ]]
 }
+
+@test "items with whitespace" {
+    [[ $BATS_RUN_SKIPPED == true  ]] || skip
+    expected=$(cat <<END
+For want of a rusty nail the horse shoe was lost.
+And all for the want of a rusty nail.
+END
+)
+    run bash proverb.sh "rusty nail" "horse shoe"
+    [[ $status -eq 0 ]]
+    [[ $output == "$expected" ]]
+}
+
+@test "shell globbing character" {
+    [[ $BATS_RUN_SKIPPED == true  ]] || skip
+    expected=$(cat <<END
+For want of a quotes the * was lost.
+And all for the want of a quotes.
+END
+)
+    run bash proverb.sh quotes "*"
+    [[ $status -eq 0 ]]
+    [[ $output == "$expected" ]]
+}
diff --git a/exercises/word-count/.meta/version b/exercises/word-count/.meta/version
index 88c5fb89..347f5833 100644
--- a/exercises/word-count/.meta/version
+++ b/exercises/word-count/.meta/version
@@ -1 +1 @@
-1.4.0
+1.4.1
diff --git a/exercises/word-count/example.sh b/exercises/word-count/example.sh
index 941d1b64..f7e19d26 100755
--- a/exercises/word-count/example.sh
+++ b/exercises/word-count/example.sh
@@ -1,6 +1,6 @@
 #!/bin/bash
 
-init_var=$(echo -e "$1" | tr -d '!@$%^&:.')
+init_var=$(echo -e "$1" | tr -d '*!@$%^&:.')
 preprocessed_var=$(echo "${init_var//[$'\n,']/ }" | tr '[:upper:]' '[:lower:]')
 
 read -r -a words <<< "$preprocessed_var"
diff --git a/exercises/word-count/word_count_test.sh b/exercises/word-count/word_count_test.sh
index 5c6c9a1a..0ec0a049 100644
--- a/exercises/word-count/word_count_test.sh
+++ b/exercises/word-count/word_count_test.sh
@@ -139,3 +139,12 @@
   [[ $(wc -l <<< "$output") -eq 3 ]]
 }
 
+@test "contains shell globbing character" {
+  [[ $BATS_RUN_SKIPPED == true  ]] || skip
+  run bash word_count.sh "two * words"
+  [[ $status -eq 0 ]]
+  echo "$output" | grep -qFx "two: 1"
+  echo "$output" | grep -qFx "words: 1"
+  [[ $(wc -l <<< "$output") -eq 2 ]]
+}
+