diff options
author | Christian Grothoff <christian@grothoff.org> | 2020-01-16 20:26:16 +0100 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2020-01-16 20:26:16 +0100 |
commit | 621f33a3d8e5ed9fcc827712c87c7e0296b2883b (patch) | |
tree | 300361d942a7d64896e08c542733090bb0a1ced9 | |
parent | 80b5433b24772c1050692461f168b535e69eb4dc (diff) | |
download | exchange-621f33a3d8e5ed9fcc827712c87c7e0296b2883b.tar.xz |
address FIXMEs
-rw-r--r-- | src/lib/test_exchange_api.c | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/src/lib/test_exchange_api.c b/src/lib/test_exchange_api.c index dc07e6b56..e8bb1efb5 100644 --- a/src/lib/test_exchange_api.c +++ b/src/lib/test_exchange_api.c @@ -206,18 +206,14 @@ run (void *cls, }; struct TALER_TESTING_Command refresh[] = { - /* Fill reserve with EUR:5, 1ct is for fees. NOTE: the old - * test-suite gave a account number of _424_ to the user at - * this step; to type less, here the _42_ number is reused. - * Does this change the tests semantics? */ + /* Fill reserve with EUR:5, 1ct is for fees. */ CMD_TRANSFER_TO_EXCHANGE ("refresh-create-reserve-1", "EUR:5.01"), - TALER_TESTING_cmd_check_bank_admin_transfer ( - "check-refresh-create-reserve-1", - "EUR:5.01", - bc.user42_payto, - bc.exchange_payto, - "refresh-create-reserve-1"), + TALER_TESTING_cmd_check_bank_admin_transfer ("ck-refresh-create-reserve-1", + "EUR:5.01", + bc.user42_payto, + bc.exchange_payto, + "refresh-create-reserve-1"), /** * Make previous command effective. */ @@ -294,8 +290,13 @@ run (void *cls, "refresh-withdraw-coin-1", MHD_HTTP_CONFLICT, NULL), - /* FIXME: also test with coin that was already melted - * (signature differs from coin that was deposited...) */ + /* Test running a failing melt operation (on a coin that + was itself revealed and subsequently deposited) */ + TALER_TESTING_cmd_refresh_melt ("refresh-melt-failing-2", + "refresh-reveal-1", + MHD_HTTP_CONFLICT, + NULL), + TALER_TESTING_cmd_end () }; |