remove redundant exchange_account_url

This commit is contained in:
Florian Dold 2020-01-18 04:10:02 +01:00
parent add346eb6c
commit 22bd615fa9
No known key found for this signature in database
GPG Key ID: D2E4F00F29D02A4B
8 changed files with 12 additions and 25 deletions

View File

@ -268,13 +268,6 @@ TALER_TESTING_url_port_free (const char *url);
*/ */
struct TALER_TESTING_BankConfiguration struct TALER_TESTING_BankConfiguration
{ {
/**
* Base URL of the exchange's bank account. Basically
* @e bank_url plus the exchange account.
*
* FIXME: remove? This duplicates exchange_auth.wire_gateway_url!
*/
char *exchange_account_url;
/** /**
* Authentication data for the exchange user at the bank. * Authentication data for the exchange user at the bank.

View File

@ -82,7 +82,7 @@ static struct TALER_TESTING_BankConfiguration bc;
*/ */
#define CMD_TRANSFER_TO_EXCHANGE(label,amount) \ #define CMD_TRANSFER_TO_EXCHANGE(label,amount) \
TALER_TESTING_cmd_admin_add_incoming (label, amount, \ TALER_TESTING_cmd_admin_add_incoming (label, amount, \
bc.exchange_account_url, \ bc.exchange_auth.wire_gateway_url, \
&bc.exchange_auth, \ &bc.exchange_auth, \
bc.user42_payto) bc.user42_payto)

View File

@ -69,40 +69,40 @@ run (void *cls,
{ {
struct TALER_TESTING_Command commands[] = { struct TALER_TESTING_Command commands[] = {
TALER_TESTING_cmd_bank_credits ("history-0", TALER_TESTING_cmd_bank_credits ("history-0",
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
NULL, NULL,
1), 1),
TALER_TESTING_cmd_admin_add_incoming ("credit-1", TALER_TESTING_cmd_admin_add_incoming ("credit-1",
"KUDOS:5.01", "KUDOS:5.01",
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
bc.user42_payto), bc.user42_payto),
TALER_TESTING_cmd_bank_credits ("history-1c", TALER_TESTING_cmd_bank_credits ("history-1c",
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
NULL, NULL,
5), 5),
TALER_TESTING_cmd_bank_debits ("history-1d", TALER_TESTING_cmd_bank_debits ("history-1d",
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
NULL, NULL,
5), 5),
TALER_TESTING_cmd_admin_add_incoming ("credit-2", TALER_TESTING_cmd_admin_add_incoming ("credit-2",
"KUDOS:3.21", "KUDOS:3.21",
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
bc.user42_payto), bc.user42_payto),
TALER_TESTING_cmd_transfer ("debit-1", TALER_TESTING_cmd_transfer ("debit-1",
"KUDOS:3.22", "KUDOS:3.22",
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
bc.exchange_payto, bc.exchange_payto,
bc.user42_payto, bc.user42_payto,
&wtid, &wtid,
"http://exchange.example.com/"), "http://exchange.example.com/"),
TALER_TESTING_cmd_bank_debits ("history-2b", TALER_TESTING_cmd_bank_debits ("history-2b",
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
NULL, NULL,
5), 5),

View File

@ -82,7 +82,7 @@ static struct TALER_TESTING_BankConfiguration bc;
*/ */
#define CMD_TRANSFER_TO_EXCHANGE(label,amount) \ #define CMD_TRANSFER_TO_EXCHANGE(label,amount) \
TALER_TESTING_cmd_admin_add_incoming (label, amount, \ TALER_TESTING_cmd_admin_add_incoming (label, amount, \
bc.exchange_account_url, \ bc.exchange_auth.wire_gateway_url, \
&bc.exchange_auth, \ &bc.exchange_auth, \
bc.user42_payto) bc.user42_payto)

View File

@ -68,7 +68,7 @@ run (void *cls,
*/ */
TALER_TESTING_cmd_admin_add_incoming ("create-reserve-1", TALER_TESTING_cmd_admin_add_incoming ("create-reserve-1",
"EUR:5.01", "EUR:5.01",
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
bc.user42_payto), bc.user42_payto),
TALER_TESTING_cmd_check_bank_admin_transfer ("check-create-reserve-1", TALER_TESTING_cmd_check_bank_admin_transfer ("check-create-reserve-1",

View File

@ -91,7 +91,7 @@ static struct GNUNET_OS_Process *twisterd;
*/ */
#define CMD_TRANSFER_TO_EXCHANGE(label,amount) \ #define CMD_TRANSFER_TO_EXCHANGE(label,amount) \
TALER_TESTING_cmd_admin_add_incoming (label, amount, \ TALER_TESTING_cmd_admin_add_incoming (label, amount, \
bc.exchange_account_url, \ bc.exchange_auth.wire_gateway_url, \
&bc.exchange_auth, \ &bc.exchange_auth, \
bc.user42_payto) bc.user42_payto)

View File

@ -54,7 +54,7 @@ transfer_to_exchange (const char *label,
{ {
return TALER_TESTING_cmd_admin_add_incoming (label, return TALER_TESTING_cmd_admin_add_incoming (label,
amount, amount,
bc.exchange_account_url, bc.exchange_auth.wire_gateway_url,
&bc.exchange_auth, &bc.exchange_auth,
bc.user42_payto); bc.user42_payto);
} }

View File

@ -361,7 +361,6 @@ TALER_TESTING_prepare_bank (const char *config_filename,
return GNUNET_SYSERR; return GNUNET_SYSERR;
} }
GNUNET_CONFIGURATION_destroy (cfg); GNUNET_CONFIGURATION_destroy (cfg);
bc->exchange_account_url = GNUNET_strdup (bc->exchange_auth.wire_gateway_url);
bc->exchange_payto = exchange_payto_uri; bc->exchange_payto = exchange_payto_uri;
bc->user42_payto = "payto://x-taler-bank/localhost/42"; bc->user42_payto = "payto://x-taler-bank/localhost/42";
bc->user43_payto = "payto://x-taler-bank/localhost/43"; bc->user43_payto = "payto://x-taler-bank/localhost/43";
@ -457,11 +456,6 @@ TALER_TESTING_prepare_fakebank (const char *config_filename,
bc->exchange_auth.wire_gateway_url = NULL; bc->exchange_auth.wire_gateway_url = NULL;
return GNUNET_SYSERR; return GNUNET_SYSERR;
} }
/* FIXME: this duplicates bc->exchange_auth.wire_gateway_url */
bc->exchange_account_url = GNUNET_strdup (bc->exchange_auth.wire_gateway_url);
GNUNET_assert (NULL != bc->exchange_account_url);
GNUNET_log (GNUNET_ERROR_TYPE_INFO, "fakebank account URL: %s\n",
bc->exchange_account_url);
/* Now we know it's the fake bank, for purpose of authentication, we /* Now we know it's the fake bank, for purpose of authentication, we
* don't have any auth. */ * don't have any auth. */
bc->exchange_auth.method = TALER_BANK_AUTH_NONE; bc->exchange_auth.method = TALER_BANK_AUTH_NONE;