diff options
| author | Odoo's Mergebot <xmo+roboto@odoo.com> | 2020-01-24 09:53:34 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-01-24 09:53:34 +0100 |
| commit | d065a38acbe16e82238b91be56fd9c77eca66316 (patch) | |
| tree | 8a0f8f228b2dbab8d5dc44e2b235b44e0975a980 /addons/web/static/src/js/fields/basic_fields.js | |
| parent | 4a19e48e4ae865f952f9acc4d6697b069ddb0e77 (diff) | |
| parent | d2c7efa9be66beb5d96fdffc3f849ddf136e20c9 (diff) | |
[FW][IMP] core: reporting of browser logs / errors during setup
Improve handling of JS logging in headless runner in order to avoid losing logs
and errors e.g. the issue fixed by odoo/odoo#41231 passed because it occurred
during module loading, which happens during initial page loading (browser_js >
navigate_to > _websocket_wait_event), which ignored logs (and exceptions though
here it's a console.error log), and as a result reported no failure (and would
simply miss that specific test as well as every test following it).
This requires additional modifications as we have a fair amount of silent
failures (exceptions or logging.error calls) at the moment:
- downgrade one error to a warning (which becomes an info at the python level)
- cleanup some synthetic / mock errors to better match what comes over RPC (and avoid transient or setup failures)
- tours which end in an action, leading to JS code executing during browser cleanup (and blowing up)
- if the last step triggers a default (implicit) run, replace by a no-op
- fix tours for which that does not work by either modifying the last step or adding an additional check step
closes odoo/odoo#43588
Forward-port-of: odoo/odoo#43567
Forward-port-of: odoo/odoo#41334
Related: odoo/enterprise#7813
Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
Diffstat (limited to 'addons/web/static/src/js/fields/basic_fields.js')
0 files changed, 0 insertions, 0 deletions
