Merge pull request #11 from tigerbeetle/tobi/fix_bunch

Fix multiple small issues.

Author: toziegler

exercises/014_get_account_balances.sh

@@ -16,4 +16,7 @@ tb "create_transfers id=14000 debit_account_id=1400 credit_account_id=1401 amoun
                      id=14002 debit_account_id=1400 credit_account_id=1401 amount=100 ledger=140 code=10;"
 
 # Uh oh! We're trying to look up historical balances for an account that doesn't have history enabled.
-tb "get_account_balances account_id=1401;"
+output=$(tb "get_account_balances account_id=1401;")
+if [[ "$output" == *"No balances were found"* ]]; then
+    exit 1
+fi

exercises/024_two_phase_transfers.sh

@@ -19,7 +19,7 @@ tb "create_transfers id=24000 debit_account_id=2400 credit_account_id=2401 amoun
 tb "lookup_accounts id=2400, id=2401;"
 
 # Now we can execute or "post" the transfer:
-tb "create_transfers id=24000 pending_id=24000 flags=post_pending_transfer;"
+tb "create_transfers id=24000 pending_id=24000 amount=200 flags=post_pending_transfer;"
 
 # Uh oh! That doesn't work. The second transfer is a separate transfer so it needs its own unique ID.
 
@@ -30,7 +30,7 @@ tb "create_transfers id=24003 pending_id=??? flags=???;"
 # Finally, we can look up the accounts again to see their final balances:
 tb "lookup_accounts id=2400, id=2401;"
 
-# Note that we don't need to specify the debit_account_id, credit_account_id, amount, ledger, or code
+# Note that we don't need to specify the debit_account_id, credit_account_id, ledger, or code
 # for the transfers that post or void pending transfers.
 # We could if we wanted to -- but they need to be the same as in the pending transfer
 # (with the exception of the `amount`, which we'll explain in another exercise).

exercises/025_two_phase_transfers_timeout.sh

@@ -21,8 +21,8 @@ sleep 2
 
 # Now, we'll try to post both transfers.
 # Can you fix this request to post both transfers?
-output=$(tb "create_transfers id=25002 pending_id=??? flags=post_pending_transfer,
-                              id=25003 pending_id=25001 flags=???;")
+output=$(tb "create_transfers id=25002 pending_id=??? amount=200 flags=post_pending_transfer,
+                              id=25003 pending_id=25001 amount=100 flags=???;")
 echo "$output"
 
 # Notice that the first transfer returns an error because the pending transfer already timed out.

exercises/026_two_phase_transfers_partial.sh

@@ -14,12 +14,12 @@ tb "create_transfers id=26000 debit_account_id=2600 credit_account_id=2601 amoun
                      id=26002 debit_account_id=2600 credit_account_id=2601 amount=100 ledger=260 code=10 flags=pending;"
 
 # Now, we'll post them -- but we're going to specify different amounts:
-output=$(tb "create_transfers id=26003 pending_id=26000 amount=0    flags=post_pending_transfer,
-                              id=26004 pending_id=26001 amount=200  flags=post_pending_transfer,
-                              id=26005 pending_id=26002 amount=200  flags=post_pending_transfer;")
+output=$(tb "create_transfers id=26003 pending_id=26000 amount=500 flags=post_pending_transfer,
+                              id=26004 pending_id=26001 amount=200 flags=post_pending_transfer,
+                              id=26005 pending_id=26002 amount=200 flags=post_pending_transfer;")
 
 # Uh oh! That didn't work!
-# The first transfer is actually okay -- specifying an amount of 0 means the full amount is posted.
+# The first transfer is actually okay -- specifying the same amount as the pending transfer posts the full amount.
 # The second is also okay, because the amount is less than the pending transfer.
 # But that third transfer...
 # Can you fix it?

patches/010_batch_limits.patch

@@ -1,13 +1,13 @@
 diff --git a/exercises/010_batch_limits.sh b/exercises/010_batch_limits.sh
-index 4d6372b..bc28aea 100755
+index fa34cff..7f6a21c 100755
 --- a/exercises/010_batch_limits.sh
 +++ b/exercises/010_batch_limits.sh
 @@ -20,7 +20,7 @@ tb "${create_account_batch%, };"
  # Now let's do the same for transfers!
- 
+
  create_transfer_batch="create_transfers "
--for ((i=0; i<8191; i++)); do
+-for ((i=0; i<8190; i++)); do
 +for ((i=0; i<8189; i++)); do
      create_transfer_batch+="id=$((1000 + i)) \
          debit_account_id=$((100000 + i)) \
-         credit_account_id=$((100000 + (1 + i) % 8190)) \
+         credit_account_id=$((100000 + (1 + i) % 8189)) \

patches/014_get_account_balances.patch

@@ -1,10 +1,13 @@
 diff --git a/exercises/014_get_account_balances.sh b/exercises/014_get_account_balances.sh
-index a4267e0..2392d08 100755
+index fe56825..43e3b41 100755
 --- a/exercises/014_get_account_balances.sh
 +++ b/exercises/014_get_account_balances.sh
-@@ -16,4 +16,4 @@ tb "create_transfers id=14000 debit_account_id=1400 credit_account_id=1401 amoun
+@@ -16,7 +16,7 @@ tb "create_transfers id=14000 debit_account_id=1400 credit_account_id=1401 amoun
                       id=14002 debit_account_id=1400 credit_account_id=1401 amount=100 ledger=140 code=10;"
- 
+
  # Uh oh! We're trying to look up historical balances for an account that doesn't have history enabled.
--tb "get_account_balances account_id=1401;"
-+tb "get_account_balances account_id=1400;"
+-output=$(tb "get_account_balances account_id=1401;")
++output=$(tb "get_account_balances account_id=1400;")
+ if [[ "$output" == *"No balances were found"* ]]; then
+     exit 1
+ fi

patches/024_two_phase_transfers.patch

@@ -1,20 +1,19 @@
 diff --git a/exercises/024_two_phase_transfers.sh b/exercises/024_two_phase_transfers.sh
-index a340abd..8171019 100755
 --- a/exercises/024_two_phase_transfers.sh
 +++ b/exercises/024_two_phase_transfers.sh
-@@ -19,13 +19,13 @@ tb "create_transfers id=24000 debit_account_id=2400 credit_account_id=2401 amoun
+@@ -19,13 +19,13 @@
  tb "lookup_accounts id=2400, id=2401;"
- 
+
  # Now we can execute or "post" the transfer:
--tb "create_transfers id=24000 pending_id=24000 flags=post_pending_transfer;"
-+tb "create_transfers id=24002 pending_id=24000 flags=post_pending_transfer;"
- 
+-tb "create_transfers id=24000 pending_id=24000 amount=200 flags=post_pending_transfer;"
++tb "create_transfers id=24002 pending_id=24000 amount=200 flags=post_pending_transfer;"
+
  # Uh oh! That doesn't work. The second transfer is a separate transfer so it needs its own unique ID.
- 
+
  # Can you fix this so that the second transfer is voided?
  # Hint: we need to use the `void_pending_transfer` flag.
 -tb "create_transfers id=24003 pending_id=??? flags=???;"
 +tb "create_transfers id=24003 pending_id=24001 flags=void_pending_transfer;"
- 
+
  # Finally, we can look up the accounts again to see their final balances:
  tb "lookup_accounts id=2400, id=2401;"

patches/025_two_phase_transfers_timeout.patch

@@ -1,15 +1,14 @@
 diff --git a/exercises/025_two_phase_transfers_timeout.sh b/exercises/025_two_phase_transfers_timeout.sh
-index 9c66a93..8fa425e 100755
 --- a/exercises/025_two_phase_transfers_timeout.sh
 +++ b/exercises/025_two_phase_transfers_timeout.sh
-@@ -21,8 +21,8 @@ sleep 2
- 
+@@ -21,8 +21,8 @@
+
  # Now, we'll try to post both transfers.
  # Can you fix this request to post both transfers?
--output=$(tb "create_transfers id=25002 pending_id=??? flags=post_pending_transfer,
--                              id=25003 pending_id=25001 flags=???;")
-+output=$(tb "create_transfers id=25002 pending_id=25000 flags=post_pending_transfer,
-+                              id=25003 pending_id=25001 flags=post_pending_transfer;")
+-output=$(tb "create_transfers id=25002 pending_id=??? amount=200 flags=post_pending_transfer,
+-                              id=25003 pending_id=25001 amount=100 flags=???;")
++output=$(tb "create_transfers id=25002 pending_id=25000 amount=200 flags=post_pending_transfer,
++                              id=25003 pending_id=25001 amount=100 flags=post_pending_transfer;")
  echo "$output"
- 
+
  # Notice that the first transfer returns an error because the pending transfer already timed out.

patches/026_two_phase_transfers_partial.patch

@@ -1,13 +1,12 @@
 diff --git a/exercises/026_two_phase_transfers_partial.sh b/exercises/026_two_phase_transfers_partial.sh
-index f3dcc0a..555d557 100755
 --- a/exercises/026_two_phase_transfers_partial.sh
 +++ b/exercises/026_two_phase_transfers_partial.sh
-@@ -16,7 +16,7 @@ tb "create_transfers id=26000 debit_account_id=2600 credit_account_id=2601 amoun
+@@ -16,7 +16,7 @@
  # Now, we'll post them -- but we're going to specify different amounts:
- output=$(tb "create_transfers id=26003 pending_id=26000 amount=0    flags=post_pending_transfer,
-                               id=26004 pending_id=26001 amount=200  flags=post_pending_transfer,
--                              id=26005 pending_id=26002 amount=200  flags=post_pending_transfer;")
-+                              id=26005 pending_id=26002 amount=100  flags=post_pending_transfer;")
- 
+ output=$(tb "create_transfers id=26003 pending_id=26000 amount=500 flags=post_pending_transfer,
+                               id=26004 pending_id=26001 amount=200 flags=post_pending_transfer,
+-                              id=26005 pending_id=26002 amount=200 flags=post_pending_transfer;")
++                              id=26005 pending_id=26002 amount=100 flags=post_pending_transfer;")
+
  # Uh oh! That didn't work!
- # The first transfer is actually okay -- specifying an amount of 0 means the full amount is posted.
+ # The first transfer is actually okay -- specifying the same amount as the pending transfer posts the full amount.

run.sh

@@ -6,14 +6,32 @@ source ./tools/terminal_format.sh
 # Trap the script exit signal and send SIGINT to the server process
 trap 'processes_descendents_kill $$' EXIT SIGHUP SIGINT SIGQUIT SIGTERM SIGKILL
 
+# Optional: skip ahead to a specific exercise number (e.g., ./run.sh 15)
+# Setup exercises (000-002) always run regardless.
+start_from=$((10#${1:-0}))
+
+# Remove old data file to ensure a fresh start
+rm -f 0_0.tigerbeetle
+
 echo "${bold}Welcome to the Tigerlings exercises!${normal}"
 echo ""
 echo "This script will run each exercise and stop when it gets to a broken one."
 echo "Once you've fixed that exercise, run this script again to continue."
+if [ "$start_from" -gt 0 ]; then
+    echo "Skipping ahead to exercise $start_from."
+fi
 echo ""
 
 # Run each exercise
 for file in $(ls exercises/[0-9][0-9][0-9]*.sh | sort -n); do
+    # Extract the exercise number from the filename
+    exercise_num=$((10#$(echo "$file" | grep -o '[0-9]\{3\}' | head -1)))
+
+    # Always run setup exercises (000-002), skip others before start_from
+    if [ "$exercise_num" -gt 2 ] && [ "$exercise_num" -lt "$start_from" ]; then
+        continue
+    fi
+
     echo "${bold}Running exercise: ./$file${normal}"
 
     if [ "$file" = "exercises/000_download.sh" ]; then