Small cleanup
1) Remove the todo, ING coming doesn't provide usefull information 2) yield account instead to create a list 3) remove coming in the test (not implemented)
This commit is contained in:
parent
996f1e0c22
commit
f455fbf1e6
3 changed files with 1 additions and 11 deletions
|
|
@ -74,9 +74,3 @@ class INGBackend(BaseBackend, ICapBank):
|
||||||
with self.browser:
|
with self.browser:
|
||||||
for history in self.browser.get_history(account.id):
|
for history in self.browser.get_history(account.id):
|
||||||
yield history
|
yield history
|
||||||
|
|
||||||
# TODO
|
|
||||||
#def iter_coming(self, account):
|
|
||||||
# with self.browser:
|
|
||||||
# for coming in self.browser.get_coming_operations(account.id):
|
|
||||||
# yield coming
|
|
||||||
|
|
|
||||||
|
|
@ -34,7 +34,6 @@ class AccountsList(BasePage):
|
||||||
pass
|
pass
|
||||||
|
|
||||||
def get_list(self):
|
def get_list(self):
|
||||||
l = []
|
|
||||||
ids = []
|
ids = []
|
||||||
for td in self.document.xpath('.//td[@nowrap="nowrap"]'):
|
for td in self.document.xpath('.//td[@nowrap="nowrap"]'):
|
||||||
account = Account()
|
account = Account()
|
||||||
|
|
@ -50,6 +49,4 @@ class AccountsList(BasePage):
|
||||||
account.balance = Decimal(linkbis.text.replace('.', '').\
|
account.balance = Decimal(linkbis.text.replace('.', '').\
|
||||||
replace(' ', '').replace(',', '.'))
|
replace(' ', '').replace(',', '.'))
|
||||||
account.coming = NotAvailable
|
account.coming = NotAvailable
|
||||||
l.append(account)
|
yield account
|
||||||
|
|
||||||
return l
|
|
||||||
|
|
|
||||||
|
|
@ -28,5 +28,4 @@ class INGTest(BackendTest):
|
||||||
l = list(self.backend.iter_accounts())
|
l = list(self.backend.iter_accounts())
|
||||||
if len(l) > 0:
|
if len(l) > 0:
|
||||||
a = l[0]
|
a = l[0]
|
||||||
list(self.backend.iter_coming(a))
|
|
||||||
list(self.backend.iter_history(a))
|
list(self.backend.iter_history(a))
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue