[GNUnet-SVN] [taler-merchant] branch master updated: Simplifying payment generator.

[gnunet]

This is an automated email from the git hooks/post-receive script.

marcello pushed a commit to branch master
in repository merchant.

The following commit(s) were added to refs/heads/master by this push:
     new a53d346  Simplifying payment generator.
a53d346 is described below

commit a53d346f325260e7293ed1f8534f6b15e604ad8e
Author: Marcello Stanisci <address@hidden>
AuthorDate: Thu Jun 14 12:46:38 2018 +0200

    Simplifying payment generator.
    
    It grew too complicated as corner cases were
    inserted there too.  Those corner cases included
    non-aggregated transactions and 2-coins payments,
    aimed to test the backoffice site.
    
    However, that way the original purpose of the
    generator was subverted: it is a tool to measure
    performance under normal circumstances.
    
    A new tool where all the other corner cases
    are generated should instead be created.
---
 .../taler-merchant-generate-payments.c             | 86 ----------------------
 1 file changed, 86 deletions(-)

diff --git a/src/merchant-tools/taler-merchant-generate-payments.c 
b/src/merchant-tools/taler-merchant-generate-payments.c
index 1878ab7..29cddd8 100644
--- a/src/merchant-tools/taler-merchant-generate-payments.c
+++ b/src/merchant-tools/taler-merchant-generate-payments.c
@@ -305,32 +305,6 @@ run (void *cls,
        CURRENCY_5,
        MHD_HTTP_OK),
 
-    /* This coin will be spent but never aggregated,
-     * in order to get 202 responses from tracks.  */
-    TALER_TESTING_cmd_withdraw_amount
-      ("withdraw-coin-3",
-       is->exchange,
-       "create-reserve-1",
-       CURRENCY_5,
-       MHD_HTTP_OK),
-
-    /* coin 4 & 5 will be deposited for the same
-     * contract; needed in case some testing utility
-     * wants to trigger a "failed dependency" error. */
-    TALER_TESTING_cmd_withdraw_amount
-      ("withdraw-coin-4",
-       is->exchange,
-       "create-reserve-1",
-       CURRENCY_5,
-       MHD_HTTP_OK),
-
-    TALER_TESTING_cmd_withdraw_amount
-      ("withdraw-coin-5",
-       is->exchange,
-       "create-reserve-1",
-       CURRENCY_5,
-       MHD_HTTP_OK),
-
     TALER_TESTING_cmd_proposal
       ("create-proposal-1",
        merchant_url,
@@ -401,66 +375,6 @@ run (void *cls,
        "track-transaction-1",
        "deposit-simple-2"),
 
-    /* Doing the 2-coins payment; needed to generate the
-     * "failed dependency" response error, at /track/transaction.
-     * NOTE: not used here, but done just in case a testing
-     * program would need it.  And this MUST happen here, as
-     * no tracking operation happens next and so the merchant
-     * won't be able to use a cached version in its database
-     * when serving /track/..; therefore it will relate to the
-     * exchange that can be twisted by the testing logic.  */
-    TALER_TESTING_cmd_proposal
-      ("create-proposal-4&5",
-       merchant_url,
-       is->ctx,
-       MHD_HTTP_OK,
-       order_worth_10_2coins,
-       NULL),
-
-    TALER_TESTING_cmd_pay ("deposit-4&5",
-                           merchant_url,
-                           is->ctx,
-                           MHD_HTTP_OK,
-                           "create-proposal-4&5",
-                           "withdraw-coin-4;" \
-                           "withdraw-coin-5",
-                           CURRENCY_10,
-                           CURRENCY_9_98, // no sense now
-                           CURRENCY_0_02), // no sense now
-
-    TALER_TESTING_cmd_exec_aggregator
-      ("aggregate-2",
-       cfg_filename),
-
-    /* Must be _after_ any aggregation takes place.  */
-    TALER_TESTING_cmd_proposal
-      ("create-proposal-3",
-       merchant_url,
-       is->ctx,
-       MHD_HTTP_OK,
-       order_worth_5_unaggregated,
-       NULL),
-
-    TALER_TESTING_cmd_pay
-      ("deposit-simple-3",
-       merchant_url,
-       is->ctx,
-       MHD_HTTP_OK,
-       "create-proposal-3",
-       "withdraw-coin-3",
-       CURRENCY_5,
-       CURRENCY_4_99,
-       CURRENCY_0_01),
-
-    TALER_TESTING_cmd_merchant_track_transaction
-      ("track-transaction-2",
-       merchant_url,
-       is->ctx,
-       MHD_HTTP_ACCEPTED,
-       "dummy", // "check bank" CMD, never used, to be deleted.
-       "deposit-simple-3",
-       CURRENCY_0_01),
-
     TALER_TESTING_cmd_rewind_ip
       ("rewind-tracks",
        TRACKS_INSTRUCTION,

-- 
To stop receiving notification emails like this one, please contact
address@hidden