fix use of index in testing_api_cmd_refresh.c

src/include/taler_testing_lib.h

@@ -1384,7 +1384,9 @@ TALER_TESTING_cmd_refund (const char *label,
  * @param label the command label
  * @param expected_response_code expected HTTP status code
  * @param coin_reference reference to any command which
- *        offers a coin & reserve private key.
+ *        offers a coin and reserve private key.  May specify
+ *        the index of the coin using "$LABEL#$INDEX" syntax.
+ *        Here, $INDEX must be a non-negative number.
  * @param amount denomination to pay back.
  *
  * @return the command.

src/lib/.gitignore

@@ -1 +1,2 @@
 test_auditor_api_version
+test_exchange_api_revocation

src/lib/test_exchange_api_revocation.c

@@ -161,7 +161,7 @@ run (void *cls,
     /* Refund coin to original coin */
     TALER_TESTING_cmd_payback ("payback-1",
                                MHD_HTTP_OK,
-                               "refresh-melt-1",
+                               "refresh-reveal-1",
                                "EUR:5"),
     /**
      * Melt original coin AGAIN
@@ -178,7 +178,7 @@ run (void *cls,
     /* Make refreshed coin invalid */
     TALER_TESTING_cmd_revoke ("revoke-2",
                               MHD_HTTP_OK,
-                              "refresh-melt-2",
+                              "refresh-reveal-2",
                               CONFIG_FILE),
     /* Make also original coin invalid */
     TALER_TESTING_cmd_revoke ("revoke-3",

src/lib/testing_api_cmd_payback.c

@@ -124,7 +124,6 @@ payback_cb (void *cls,
             const struct TALER_ReservePublicKeyP *reserve_pub,
             const json_t *full_response)
 {
-
   struct PaybackState *ps = cls;
   struct TALER_TESTING_Interpreter *is = ps->is;
   struct TALER_TESTING_Command *cmd = &is->commands[is->ip];
@@ -132,6 +131,9 @@ payback_cb (void *cls,
   const struct TALER_ReservePrivateKeyP *reserve_priv;
   struct TALER_ReservePublicKeyP rp;
   struct TALER_Amount expected_amount;
+  char *cref;
+  const char *index;
+  unsigned int idx;
 
   ps->ph = NULL;
   if (ps->expected_response_code != http_status)
@@ -148,8 +150,37 @@ payback_cb (void *cls,
     return;
   }
 
+  /* We allow command referneces of the form "$LABEL#$INDEX" or
+     just "$LABEL", which implies the index is 0. Figure out
+     which one it is. */
+  index = strchr (ps->coin_reference, '#');
+  if (NULL == index)
+  {
+    idx = 0;
+    cref = GNUNET_strdup (ps->coin_reference);
+  }
+  else
+  {
+    cref = GNUNET_strndup (ps->coin_reference,
+                           index - ps->coin_reference);
+    if (1 != sscanf (index,
+                     "%u",
+                     &idx))
+    {
+      GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                  "Numeric index (not `%s') required after `#' in command reference of command %s in %s:%u\n",
+                  index,
+                  cmd->label,
+                  __FILE__,
+                  __LINE__);
+      TALER_TESTING_interpreter_fail (is);
+      GNUNET_free (cref);
+      return;
+    }
+  }
   reserve_cmd = TALER_TESTING_interpreter_lookup_command
-    (is, ps->coin_reference);
+    (is, cref);
+  GNUNET_free (cref);
 
   if (NULL == reserve_cmd)
   {
@@ -159,7 +190,7 @@ payback_cb (void *cls,
   }
 
   if (GNUNET_OK != TALER_TESTING_get_trait_reserve_priv
-    (reserve_cmd, 0, &reserve_priv))
+    (reserve_cmd, idx, &reserve_priv))
   {
     GNUNET_break (0);
     TALER_TESTING_interpreter_fail (is);

src/lib/testing_api_cmd_refresh.c

@@ -83,7 +83,7 @@ struct RefreshMeltState
 
   /**
    * Array of the denomination public keys
-   * corresponding to the @e fresh_amounts.
+   * corresponding to the @e num_fresh_coins;
    */
   struct TALER_EXCHANGE_DenomPublicKey *fresh_pks;
 
@@ -102,6 +102,11 @@ struct RefreshMeltState
    */
   size_t refresh_data_length;
 
+  /**
+   * Number of fresh coins generated by the melt.
+   */
+  unsigned int num_fresh_coins;
+
   /**
    * Expected HTTP response code.
    */
@@ -340,24 +345,19 @@ reveal_cb (void *cls,
   case MHD_HTTP_OK:
     rrs->fresh_coins = GNUNET_new_array
       (num_coins, struct FreshCoin);
-
+    for (unsigned int i=0; i<num_coins; i++)
-    const struct TALER_EXCHANGE_DenomPublicKey *fresh_pks;
+    {
+      struct FreshCoin *fc = &rrs->fresh_coins[i];
 
       if (GNUNET_OK !=
           TALER_TESTING_get_trait_denom_pub (melt_cmd,
-                                           0,
+                                             i,
-                                           &fresh_pks))
+                                             &fc->pk))
       {
         GNUNET_break (0);
         TALER_TESTING_interpreter_fail (rrs->is);
         return;
       }
-
-    for (unsigned int i=0; i<num_coins; i++)
-    {
-      struct FreshCoin *fc = &rrs->fresh_coins[i];
-
-      fc->pk = &fresh_pks[i];
       fc->coin_priv = coin_privs[i];
       fc->sig.rsa_signature = GNUNET_CRYPTO_rsa_signature_dup
         (sigs[i].rsa_signature);
@@ -883,7 +883,7 @@ refresh_melt_run (void *cls,
   for (num_fresh_coins=0;
        NULL != melt_fresh_amounts[num_fresh_coins];
        num_fresh_coins++) ;
-
+  rms->num_fresh_coins = num_fresh_coins;
   rms->fresh_pks = GNUNET_new_array
     (num_fresh_coins,
      struct TALER_EXCHANGE_DenomPublicKey);
@@ -941,7 +941,6 @@ refresh_melt_run (void *cls,
       TALER_TESTING_interpreter_fail (rms->is);
       return;
     }
-
     for (unsigned int i=0;i<num_fresh_coins;i++)
     {
       const struct TALER_EXCHANGE_DenomPublicKey *fresh_pk;
@@ -1035,7 +1034,6 @@ refresh_melt_cleanup (void *cls,
  * @param ret[out] result (could be anything).
  * @param trait name of the trait.
  * @param index index number of the object to offer.
- *
  * @return #GNUNET_OK on success.
  */
 static int
@@ -1046,8 +1044,14 @@ refresh_melt_traits (void *cls,
 {
   struct RefreshMeltState *rms = cls;
 
+  if (index >= rms->num_fresh_coins)
+  {
+    GNUNET_break (0);
+    return GNUNET_SYSERR;
+  }
+  {
     struct TALER_TESTING_Trait traits[] = {
-    TALER_TESTING_make_trait_denom_pub (0, rms->fresh_pks),
+      TALER_TESTING_make_trait_denom_pub (index, &rms->fresh_pks[index]),
       TALER_TESTING_trait_end ()
     };
 
@@ -1055,6 +1059,7 @@ refresh_melt_traits (void *cls,
                                     ret,
                                     trait,
                                     index);
+  }
 }
 
 
@@ -1081,11 +1086,10 @@ TALER_TESTING_cmd_refresh_melt
 
   md.coin_reference = coin_reference;
   md.amount = amount;
-
   rms = GNUNET_new (struct RefreshMeltState);
   rms->melted_coin = md;
   rms->expected_response_code = expected_response_code;
-
+  {
     struct TALER_TESTING_Command cmd = {
       .label = label,
       .cls = rms,
@@ -1095,6 +1099,7 @@ TALER_TESTING_cmd_refresh_melt
     };
 
     return cmd;
+  }
 }
 
 
@@ -1109,7 +1114,6 @@ TALER_TESTING_cmd_refresh_melt
  * @param coin_reference reference to a command that will provide
  *        a coin to refresh
  * @param expected_response_code expected HTTP code
- *
  * @return the command.
  */
 struct TALER_TESTING_Command
@@ -1124,12 +1128,11 @@ TALER_TESTING_cmd_refresh_melt_double
 
   md.coin_reference = coin_reference;
   md.amount = amount;
-
   rms = GNUNET_new (struct RefreshMeltState);
   rms->melted_coin = md;
   rms->expected_response_code = expected_response_code;
   rms->double_melt = GNUNET_YES;
-
+  {
     struct TALER_TESTING_Command cmd = {
       .label = label,
       .cls = rms,
@@ -1139,6 +1142,7 @@ TALER_TESTING_cmd_refresh_melt_double
     };
 
     return cmd;
+  }
 }
 
 
@@ -1237,7 +1241,7 @@ TALER_TESTING_cmd_refresh_reveal
   rrs = GNUNET_new (struct RefreshRevealState);
   rrs->melt_reference = melt_reference;
   rrs->expected_response_code = expected_response_code;
-
+  {
     struct TALER_TESTING_Command cmd = {
       .cls = rrs,
       .label = label,
@@ -1247,6 +1251,7 @@ TALER_TESTING_cmd_refresh_reveal
     };
 
     return cmd;
+  }
 }
 
 

src/lib/testing_api_trait_denom_pub.c

@@ -59,7 +59,6 @@ TALER_TESTING_get_trait_denom_pub
  *
  * @param index index number to associate to the offered denom pub.
  * @param denom_pub denom pub to offer with this trait.
- *
  * @return the trait.
  */
 struct TALER_TESTING_Trait