diff options
| author | Xavier Morel <xmo@odoo.com> | 2020-08-17 09:25:08 +0000 |
|---|---|---|
| committer | Xavier Morel <xmo@odoo.com> | 2020-08-17 09:30:53 +0000 |
| commit | 17ae856cc10634c5d1c88fdc01a61b0e1b876fc4 (patch) | |
| tree | 346a8c0a0fcd2b21eb3df739ce15293e165f526c /addons/auth_totp/static | |
| parent | bfa00919cd893694c55d5fa9680c3dbbd1f66d3e (diff) | |
[FIX] auto_totp: tour & check RPC w/ token
* Not sure how the tour passed during merge as it would not be looking
for the button in the right tab, it would fail locally, fix this
issue by properly switching to the Account Security tab
* add the missing test of RPC (which should not work on an account
with totp enabled)
* also reorder ops & fix comments: turns out `totp_login_enabled`
checks that totp is enabled *then disables it*
closes odoo/odoo#55979
Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
Diffstat (limited to 'addons/auth_totp/static')
| -rw-r--r-- | addons/auth_totp/static/tests/totp_flow.js | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/addons/auth_totp/static/tests/totp_flow.js b/addons/auth_totp/static/tests/totp_flow.js index 3b3f1b67cc1..1eed5c3f88e 100644 --- a/addons/auth_totp/static/tests/totp_flow.js +++ b/addons/auth_totp/static/tests/totp_flow.js @@ -57,6 +57,10 @@ tour.register('totp_tour_setup', { trigger: '[data-menu=settings]', run: 'click', }, { + content: "Switch to security tab", + trigger: 'a[role=tab]:contains("Account Security")', + run: 'click', +}, { content: "Check that the button has changed", trigger: 'button:contains(Disable two-factor authentication)', run: () => {} @@ -134,6 +138,10 @@ tour.register('totp_login_enabled', { trigger: '[data-menu=settings]', run: 'click', }, { + content: "Switch to security tab", + trigger: 'a[role=tab]:contains("Account Security")', + run: 'click', +}, { content: "Check that the button has changed", trigger: 'button:contains(Enable two-factor authentication)', run: () => {} |
