From 1df0225b6b19133ba6ec1605a6471e105ce8f0e3 Mon Sep 17 00:00:00 2001
From: Cora Iberkleid <ciberkleid@gmail.com>
Date: Mon, 20 May 2024 13:51:17 -0400
Subject: [PATCH] Confirm expected and actual script results match

---
 server/order_and_pay.sh | 53 +++++++++++++++++++++++++++++++----------
 1 file changed, 40 insertions(+), 13 deletions(-)

diff --git a/server/order_and_pay.sh b/server/order_and_pay.sh
index ea8977a..8772c97 100755
--- a/server/order_and_pay.sh
+++ b/server/order_and_pay.sh
@@ -6,6 +6,8 @@
 # Define locations
 locations=("To go" "In store")
 
+error_flag=false
+
 # Get drink options
 drinks_http_response=$(http :8080/drinks/by-name)
 declare -a drinks
@@ -24,14 +26,15 @@ done
 num_locations=${#locations[@]}
 num_drinks=${#drinks[@]}
 expected_num_orders=$((num_drinks * (num_drinks + 1) / 2 * num_locations))
-echo "Generating $expected_num_orders orders. Options: $num_locations locations, $num_drinks drinks"
+echo "Generating orders based on $num_locations locations and $num_drinks drinks"
 
 # Place orders
 declare -a payment_links
 
 # Outer loop: locations
-orders_index=0
+num_orders=0
 location_index=0
+echo "Order counters: [order:location:drinks]"
 for location in "${locations[@]}"; do
   ((location_index++))
   drinks_index=0
@@ -40,9 +43,9 @@ for location in "${locations[@]}"; do
   for ((i = 0; i < ${#drinks[@]}; i++)); do
     # Order all single drink combinations
     ((drinks_index++))
-    ((orders_index++))
+    ((num_orders++))
     drink_order="${drinks[i]}"
-    echo "Loop $orders_index:$location_index:$drinks_index - $location ${drink_names[i]}"
+    echo "Order $num_orders:$location_index:$drinks_index - $location ${drink_names[i]}"
     order=$(echo "{ \"drinks\": [${drink_order}], \"location\": \"${location}\" }" | http http://localhost:8080/orders)
     payment_links+=($(echo ${order} | jq -r '._links["restbucks:payment"].href'))
   done
@@ -51,9 +54,9 @@ for location in "${locations[@]}"; do
   for ((i = 0; i < ${#drinks[@]} - 1; i++)); do
     for ((j = i + 1; j < ${#drinks[@]}; j++)); do
       ((drinks_index++))
-      ((orders_index++))
+      ((num_orders++))
       drink_order="${drinks[i]},${drinks[j]}"
-      echo "Loop $orders_index:$location_index:$drinks_index - $location ${drink_names[i]}, ${drink_names[j]}"
+      echo "Order $num_orders:$location_index:$drinks_index - $location ${drink_names[i]}, ${drink_names[j]}"
       order=$(echo "{ \"drinks\": [${drink_order}], \"location\": \"${location}\" }" | http http://localhost:8080/orders)
       payment_links+=($(echo ${order} | jq -r '._links["restbucks:payment"].href'))
     done
@@ -61,27 +64,51 @@ for location in "${locations[@]}"; do
 
 done
 
-if [ "$orders_index" -ne "$expected_num_orders" ]; then
-  echo "WARN: Generated $orders_index orders. Expected to generate $expected_num_orders."
+if [ "$num_orders" -ne "$expected_num_orders" ]; then
+  error_flag=true
+  echo "ERROR: Expected to generate $expected_num_orders, but generated $num_orders"
 else
-  echo "Generated $orders_index orders."
+  echo "Generated $num_orders orders"
 fi
 
 ##### MAKE PAYMENTS
 
-echo "Processing ${#payment_links[@]} orders"
+echo "Submitting payments"
 creditCardNumber=""
+expectedResult=""
+num_payments_submitted=0
+num_payments_accepted=0
 for payment_link in "${payment_links[@]}"; do
   # Conditional logic to set the credit card number based on the random number
-  random_number=$(( RANDOM % 10 ))       # Generate a random number between 0 and 99
+  random_number=$(( RANDOM % 10 ))        # Generate a random number between 0 and 99
   if [ "$random_number" -lt 8 ]; then
-      creditCardNumber="1234123412341234"     # 80% of the time, card is valid
+      creditCardNumber="1234123412341234" # 80% of the time, card is valid
+      expectedResult="201"
   elif [ "$random_number" -lt 9 ]; then
       creditCardNumber="1111222233334444" # 10% of the time, card not found in db
+      expectedResult="500"
   else
       creditCardNumber="abcdefghijklmnop" # 10% of the time, invalid format
+      expectedResult="400"
   fi
 
   status_code=$(echo "{ \"number\": \"${creditCardNumber}\" }" | http PUT ${payment_link} --print=h | grep HTTP | awk '{print $2}')
-  echo "Got HTTP status $status_code using card $creditCardNumber at $payment_link"
+  if [ $status_code == $expectedResult ]; then
+    ((num_payments_submitted++))
+    echo "OK: HTTP status $status_code for card $creditCardNumber at $payment_link"
+    if [ $status_code == "201" ]; then
+      ((num_payments_accepted++))
+    fi
+  else
+    error_flag=true
+    echo "ERROR: Expected HTTP status $expectedResult, but got $status_code for card $creditCardNumber at $payment_link"
+  fi
 done
+echo "$num_payments_accepted of $num_payments_submitted payments passed credit card validation"
+
+echo
+if [[ $error_flag == "true" ]]; then
+  echo "ERROR!! Expected to process $num_orders payments, but processed $num_payments_submitted"
+else
+  echo "SUCCESS: Expected and actual results match"
+fi