Daily updates from Odoo
Saturday, June 8, 2019
2 changes
Resolved issues and error corrections
This change updates an internal translation test so it no longer conflicts with similar sample selection values from another test module. It helps prevent intermittent test failures, improving confidence in translation-related development without affecting end users.
Original PR description
export.selection.withdefault and export.selection both have a selection field with Foo and Bar as possible selections. When doing `self.env["base.update.translations"].create({'lang':…
export.selection.withdefault and export.selection both have a selection field
with Foo and Bar as possible selections.
When doing
`self.env["base.update.translations"].create({'lang': 'fr_FR'}).act_update()`
the base.update.translations wizard exports the translations of ALL modules,
including the ones of the test_impex module.
The generate po entry is something like:
```
#. modules: test_impex, test_translation_import
#: selection:export.selection,value:0
#: selection:export.selection.withdefault,value:0
#: selection:test.translation.import,import_type:0
msgid "Bar"
msgstr "Bar in french"
```
but may also be
```
#. modules: test_translation_import, test_impex
#: selection:export.selection,value:0
#: selection:export.selection.withdefault,value:0
#: selection:test.translation.import,import_type:0
msgid "Bar"
msgstr "Bar in french"
```
(not sure where the indeterminism comes from)
which creates 3 translation entries that are sometimes linked to the module
test_impex (ignored when exporting the translations of test_translation_import)
but are sometimes linked to the module test_translation_import instead
In the second case, this makes 3 more translations that is was supposed to
exist.
This commit hides the symptom by making sure there is no longer a clash
when exporting the translations of the module test_translation_import.
A real fix to avoid creating unnecessary translations during an execution of
the base.update.translations wizard must also be made in the future.
It is assumed the unnecessary translations problem was already present before
a7621137cb0d75ec545152d5e1d413c9d3548e45 but polib introduced the indeterminism
making the test fails from time to time.JavaScript test runs will no longer mark slow startup checks as warnings that can fail automated status checks. This helps teams avoid blocked merges when there is no actual product issue to resolve.
Original PR description
In JS tests, when the ready code takes too much time, a warning is logged. As a consequence, the runbot updates the github status as a failure, causing real pain to people who simply wants to merge their features. With this commit the warning is changed into an info and the runbot will be happy.