Several elements have changed on the Spirica website:
- the "Tout afficher" button disappeared,
- the forms to get the transactions list were modified,
- the forms to get investments for each transaction changed.
This patch corrects history for each account as well as the respective
invests.

Add another xpath to find button to validate otp.

Closes: 15542@sibi
"Montant net" has been fixed in a previous commit, now it's "Montant brut"

get on transfer page because estimated value on account page can be different
need estimated value to validate origin account to do transfer

Closes: 15168@sibi 15200@sibi 15553@sibi 15554@sibi 10417@sibi 13715@sibi 15566@sibi 15231@sibi 15371@sibi

A recurring problem with bnporc is that sometimes you may not have the good transactions list.
It may be truncated a lot. For example, from 50 transactions, you had only 5.
Because of that, in the backend some transactions become deleted because of one update with the bad transactions.

I tried to fix it by copying the website, since you never had bad transactions on it.
First, I changed the 'typeReleve' from 'Comptable' to 'Previsionnel' because it is the only type the website is using.
And I added some parameters to the history request. The same ones are used to go on the transaction page and to get the json data.

I tested a lot on connections, knowing the right and the wrong history state.
Since I applied the patch, I had not once the wrong state.

Closes: 5663@zendesk
Closes: 6219@zendesk
Closes: 6225@zendesk
Closes: 6229@zendesk
Closes: 6115@zendesk

Saving account can't do transfer to external recipients
Saving account can't receive transfer when limit is reached

Closes: 13881@sibi 13123@sibi

Up to now only Cards accounts had a obj_number which may lead to
failures in matching_accounts for the backend since the value is
implicitly filled with the account.id

Closes: 6022@zendesk

In some case, you may have only one market type account. If so, the website does not show a page with the list of one account but directly the details of this account.

Most of the time, getting the number in the account label is enough to recognize it in the iter_accounts.
But sometime you have custom account label, so it is not enough. That is why the entire account name is retrieved.

Closes: 13125@sibi
Closes: 12518@sibi
Closes: 13576@sibi
Closes: 12610@sibi
Closes: 12871@sibi
Closes: 11832@sibi
Closes: 11912@sibi
Closes: 11913@sibi
Closes: 11867@sibi
Closes: 11750@sibi
Closes: 11749@sibi
Closes: 11580@sibi
Closes: 11715@sibi
Closes: 11714@sibi

The real debit date was not fetched. Instead, the rdate was put into the
date field.

They were completely skipped. There's a link to access previous month
history.
However, the debit dates all seem wrong, same for coming.

It's useless and less readable

Closes: 6189@zendesk

on axabanque page, pea history leads to an error page (in HTTP ...)
get history on netfinca bourse page for pea liquidity and pea market

Closes: 12819@sibi 12814@sibi 11620@sibi 11492@sibi 13455@sibi 11618@sibi 12821@sibi 11643@sibi
9251@sibi 11496@sibi 12813@sibi 9192@sibi 9203@sibi 9750@sibi 11123@sibi 11495@sibi 12818@sibi

In iter_investments, the xpath selecting the correct <tr> for TableElement was too
restrictive (td count >7) whereas some invests may have only 7 columns.
This is corrected with a count >= 7.

Closes: 6171@zendesk

Closes: 13778@sibi

On transfer validation page, get reason with cut 'Motif : ' and skip html comment.
Get reason to check label error with 'transfer_check_label' instead of assert without message.

remove unused parameter on handle response:
- amount
- reason
- exec_date

Closes: 14399@sibi 8735@sibi 4979@sibi

In some case, you may have a student loan with only one account. It is handled by item_loan.

Closes: 5944@zendesk

Website transfer state can be "Soumis à la banque".
Check if keywords ('Exécuté', 'Soumis') is in webiste transfer state message
instead of check that state message is in keywords

tell user than transfer was done even though it wasn't is better
than tell users that transfer state is bug even though it was done

Closes: 14390@sibi

I reverted from my previous patch using the terms 'users'.
Now, there is a real multispace implemented.

You must switch from one space to another to retrieve all the accounts.
One account belongs to one space, so when data are needed from this account, the space is changed.

Previously investments were broken, it is not anymore.
I also checked retrieving of recipients.

Handles a redirected page indicating that url is temporarily unavailable
Closes: 627771@redmine

At first, only the accounts with a label were considered subscriptions.
Now, we look for the available accounts on the last statemement page, then they are compared to the ones on the search page. This way, the needed parameters are retrieved.

The exception is catched then the subscription is retrieved without a subscriber.

Closes: 13670@sibi

Closes: 10674@sibi

Need number with only 2 number after decimal point to do transfer

Sometimes in transfer 'recap', origin account balance is the estimated balance of origin account

Closes: 9988@sibi 5128@sibi 7706@sibi

refs #5867@zendesk

Need number with only 2 number after decimal point to do transfer

I forgot the cases where get_cards was not called by get_list. So I handled the accounts_list parameter.

Closes: 2568@sibi

The main change is allowing every iframe to be visited whatever its src attribute is,
in order to detect whether or not there are CGUs.

It replaces my previous patch (09621839c8bdbca2986b24747282b10f1f770aff)
in which i was trying to skip the action needed as well, but didn't
notice the bug still occured during the first iter_accounts() runned by
the blackbox (the patch corrected the bug during the second iter_accounts,
from the state we were left at after the first iter_accounts)

Also, some refactoring:
- don't deal with the action needed with an on_load method in AccountsPage
- test the existence of an action needed only once with an extra
attribute self.action_needed_processed

and clean code a little bit

Closes: 6084@zendesk

Added a liquidity line for Compte Titres PEA and removed it from
Portefeuille Titres PEA.
Reversed code for bp and linebourse as required.
Removed balance update to fit website balances.

Closes: 5653@zendesk, 5765@zendesk, 5162@zendesk