upgrade exercises for TB 0.17.

Author: toziegler

exercises/010_batch_limits.sh

@@ -7,31 +7,31 @@ source ./tools/tb_function.sh
 # We said that everything in TigerBeetle is batched for efficiency and performance
 # -- so let's take advantage of that!
 
-# Let's go crazy here and create 8189 accounts in a single request!
-# Why 8189? That's the maximum number we can send in a request.
+# The maximum number of events we can send in a single request is 8189.
+# Let's create 8189 accounts across two batches:
 create_account_batch="create_accounts "
-for ((i=0; i<8189; i++)); do
+for ((i=0; i<4000; i++)); do
     create_account_batch+="id=$((100000 + i)) code=10 ledger=100, "
 done
+tb "${create_account_batch%, };"
 
-# Create the accounts (removing the trailing comma and space)
+create_account_batch="create_accounts "
+for ((i=4000; i<8189; i++)); do
+    create_account_batch+="id=$((100000 + i)) code=10 ledger=100, "
+done
 tb "${create_account_batch%, };"
 
 # Now let's do the same for transfers!
+# (We have to split them into batches because of the REPL's command-line size limit.)
 
 create_transfer_batch="create_transfers "
-for ((i=0; i<8190; i++)); do
-    create_transfer_batch+="id=$((1000 + i)) \
-        debit_account_id=$((100000 + i)) \
-        credit_account_id=$((100000 + (1 + i) % 8189)) \
-        amount=1 \
-        code=10 \
-        ledger=100, "
+for ((i=0; i<1400; i++)); do
+    create_transfer_batch+="id=$((1000 + i)) debit_account_id=$((100000 + i)) credit_account_id=$((100000 + (1 + i) % 8189)) amount=1 code=10 ledger=100, "
 done
-
 tb "${create_transfer_batch%, };"
-# Uh oh! We might have gone a _little_ overboard and created one too many transfers!
 
-# (If you're running TigerBeetle using a debug build, you'll see that this error
-# is triggered by an assertion failure in the client. TigerBeetle makes heavy use
-# of assertions to ensure that it operates as expected.)
+create_transfer_batch="create_transfers "
+for ((i=1400; i<2800; i++)); do
+    create_transfer_batch+="id=$((1000 + i)) debit_account_id=$((100000 + i)) credit_account_id=$((100000 + (1 + i) % 8189)) amount=1 code=10 ledger=100, "
+done
+tb "${create_transfer_batch%, };"

exercises/021_linked_transfers.sh

@@ -40,10 +40,9 @@ output=$(tb "create_transfers id=21000 debit_account_id=2100 credit_account_id=2
                               id=21004 debit_account_id=2199 credit_account_id=2101 amount=100 ledger=200 code=10 flags=linked,
                               id=21005 debit_account_id=2100 credit_account_id=2102 amount=100 ledger=200 code=10 flags=linked;")
 
-if [[ $output =~ *"Failed to create transfer \(0|1|5\)"* || 
-      $output != *"Failed to create transfer (2): tigerbeetle.CreateTransferResult.transfer_must_have_the_same_ledger_as_accounts."* ||
-      $output != *"Failed to create transfer (3): tigerbeetle.CreateTransferResult.linked_event_failed."* ||
-      $output != *"Failed to create transfer (4): tigerbeetle.CreateTransferResult.debit_account_not_found."* ]]; then
+if [[ $output != *"CreateTransferStatus.transfer_must_have_the_same_ledger_as_accounts"* ||
+      $output != *"CreateTransferStatus.linked_event_failed"* ||
+      ( $output != *"CreateTransferStatus.debit_account_not_found"* && $output != *"CreateTransferStatus.id_already_failed"* ) ]]; then
 
     echo "Uh oh! The transfers didn't have the expected results."
     exit 1

exercises/025_two_phase_transfers_timeout.sh

@@ -30,7 +30,7 @@ echo "$output"
 # We can also check the account balances to see that only the second transfer was posted:
 tb "lookup_accounts id=2500, id=2501;"
 
-if [[ $output != *"Failed to create transfer (0): tigerbeetle.CreateTransferResult.pending_transfer_expired."* ]]; then
+if [[ $output != *"CreateTransferStatus.pending_transfer_expired"* ]]; then
     echo "The first transfer should have timed out!"
     exit 1
 fi

exercises/026_two_phase_transfers_partial.sh

@@ -27,7 +27,7 @@ output=$(tb "create_transfers id=26003 pending_id=26000 amount=500 flags=post_pe
 # Once it posts, you can check the account balances to see the results.
 tb "lookup_accounts id=2600, id=2601;"
 
-if [[ $output == *"Failed to create transfer (2): tigerbeetle.CreateTransferResult.exceeds_pending_transfer_amount."* ]]; then
+if [[ $output == *"CreateTransferStatus.exceeds_pending_transfer_amount"* ]]; then
     echo "The third transfer should have been posted!"
     exit 1
 fi

exercises/027_balancing_transfers.sh

@@ -26,7 +26,7 @@ tb "create_transfers id=27000 debit_account_id=2700 credit_account_id=2701 amoun
 # Notice that the amount is 0. TigerBeetle will calculate the account's net balance
 # (in this case the net credit balance because we're using a balancing_debit) and transfer it to the other account.
 # Also, we are ignoring the result of this command because if we run it a second time, it will return the
-# error `tigerbeetle.CreateTransferResult.exists_with_different_amount` because the amount will be 100 rather than 0.
+# error `tigerbeetle.CreateTransferStatus.exists_with_different_amount` because the amount will be 100 rather than 0.
 
 # Can you zero out the balance of the other account?
 # (Remember, we don't want a `balancing_debit` in this case...)

patches/010_batch_limits.patch

@@ -1,13 +0,0 @@
-diff --git a/exercises/010_batch_limits.sh b/exercises/010_batch_limits.sh
-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<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) % 8189)) \

patches/021_linked_transfers.patch

@@ -1,8 +1,6 @@
-diff --git a/exercises/021_linked_transfers.sh b/exercises/021_linked_transfers.sh
-index 81589f7..4186bdf 100755
 --- a/exercises/021_linked_transfers.sh
 +++ b/exercises/021_linked_transfers.sh
-@@ -34,11 +34,11 @@ tb "create_accounts id=2100 code=10 ledger=200 flags=linked,
+@@ -34,11 +34,11 @@
  # - The 6th transfer succeeds on its own
 
  output=$(tb "create_transfers id=21000 debit_account_id=2100 credit_account_id=2102 amount=100 ledger=200 code=10 flags=linked,
@@ -16,5 +14,5 @@ index 81589f7..4186bdf 100755
 +                              id=21004 debit_account_id=2199 credit_account_id=2101 amount=100 ledger=200 code=10,
 +                              id=21005 debit_account_id=2100 credit_account_id=2102 amount=100 ledger=200 code=10;")
 
- if [[ $output =~ *"Failed to create transfer \(0|1|5\)"* || 
-       $output != *"Failed to create transfer (2): tigerbeetle.CreateTransferResult.transfer_must_have_the_same_ledger_as_accounts."* ||
+ if [[ $output != *"CreateTransferStatus.transfer_must_have_the_same_ledger_as_accounts"* ||
+       $output != *"CreateTransferStatus.linked_event_failed"* ||

patches/025_two_phase_transfers_timeout.patch

@@ -1,14 +1,13 @@
-diff --git a/exercises/025_two_phase_transfers_timeout.sh b/exercises/025_two_phase_transfers_timeout.sh
 --- a/exercises/025_two_phase_transfers_timeout.sh
 +++ b/exercises/025_two_phase_transfers_timeout.sh
 @@ -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=??? 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,4 +1,3 @@
-diff --git a/exercises/026_two_phase_transfers_partial.sh b/exercises/026_two_phase_transfers_partial.sh
 --- a/exercises/026_two_phase_transfers_partial.sh
 +++ b/exercises/026_two_phase_transfers_partial.sh
 @@ -16,7 +16,7 @@
@@ -7,6 +6,6 @@ diff --git a/exercises/026_two_phase_transfers_partial.sh b/exercises/026_two_ph
                                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 the same amount as the pending transfer posts the full amount.

patches/027_balancing_transfers.patch

@@ -1,8 +1,6 @@
-diff --git a/exercises/027_balancing_transfers.sh b/exercises/027_balancing_transfers.sh
-index f144f7e..3f1ebb8 100755
 --- a/exercises/027_balancing_transfers.sh
 +++ b/exercises/027_balancing_transfers.sh
-@@ -30,7 +30,7 @@ tb "create_transfers id=27000 debit_account_id=2700 credit_account_id=2701 amoun
+@@ -30,7 +30,7 @@
 
  # Can you zero out the balance of the other account?
  # (Remember, we don't want a `balancing_debit` in this case...)