[GNUnet-SVN] [taler-bank] branch master updated: adjust test case to last change

[gnunet]

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

marcello pushed a commit to branch master
in repository bank.

The following commit(s) were added to refs/heads/master by this push:
     new 97bea4a  adjust test case to last change
97bea4a is described below

commit 97bea4a0c881c549226153891fa6da10697a4c6a
Author: Marcello Stanisci <address@hidden>
AuthorDate: Mon Dec 18 19:17:56 2017 +0100

    adjust test case to last change
---
 talerbank/app/tests.py | 16 ++++++++++------
 talerbank/app/views.py |  5 +++--
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/talerbank/app/tests.py b/talerbank/app/tests.py
index 1cba338..fb2d437 100644
--- a/talerbank/app/tests.py
+++ b/talerbank/app/tests.py
@@ -119,9 +119,10 @@ class InternalWireTransferTestCase(TestCase):
         BankAccount(user=User.objects.create_user(
             username='give_money',
             password="gm")).save()
-        BankAccount(user=User.objects.create_user(
-            username='take_money'),
-            account_no=88).save()
+        self.take_money = BankAccount(
+            user=User.objects.create_user(
+                username='take_money'), account_no=4)
+        self.take_money.save()
 
     def tearDown(self):
         clear_db()
@@ -131,10 +132,13 @@ class InternalWireTransferTestCase(TestCase):
         client.login(username="give_money", password="gm")
         response = client.post(reverse("profile", urlconf=urls),
                                {"amount": 3.0,
-                                "counterpart": 88,
+                                "receiver": self.take_money.account_no,
                                 "subject": "charity"})
-        self.assertEqual(0, Amount.cmp(Amount(settings.TALER_CURRENCY, 3),
-                                       
BankAccount.objects.get(account_no=88).amount))
+        take_money = BankAccount.objects.get(account_no=4)
+        self.assertEqual(0, Amount.cmp(
+            Amount(settings.TALER_CURRENCY, 3),
+            take_money.amount),
+                         msg=self.take_money.amount.stringify(2))
         self.assertEqual(200, response.status_code)
 
 
diff --git a/talerbank/app/views.py b/talerbank/app/views.py
index edc48d7..c8c6ccb 100644
--- a/talerbank/app/views.py
+++ b/talerbank/app/views.py
@@ -129,7 +129,7 @@ def profile_page(request):
                         user=request.user),
                     BankAccount.objects.get(
                         account_no=wtf.cleaned_data.get(
-                            "counterpart")),
+                            "receiver")),
                         wtf.cleaned_data.get("subject"))
                 request.session["just_wire_transferred"] = True
             except BankAccount.DoesNotExist:
@@ -143,6 +143,8 @@ def profile_page(request):
                     info_bar = "Operation not possible:" \
                                " debit and credit account" \
                                " are the same!"
+        else:
+            LOGGER.warning("invalid wire transfer POSTed", wtf.errors)
     wtf = WTForm()
 
     just_withdrawn = get_session_flag(request, "just_withdrawn")
@@ -637,7 +639,6 @@ def wire_transfer(amount,
                   credit_account,
                   subject,
                   **kwargs):
-
     def err_cb(exc, resp):
         LOGGER.error(str(exc))
         raise WireTransferException(exc, resp)

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