Skip to content

Syncing from upstream odoo/runbot (19.0-coverage-resurect-xdo)#1077

Open
bt-admin wants to merge 4 commits intobrain-tec:19.0-coverage-resurect-xdofrom
odoo:19.0-coverage-resurect-xdo
Open

Syncing from upstream odoo/runbot (19.0-coverage-resurect-xdo)#1077
bt-admin wants to merge 4 commits intobrain-tec:19.0-coverage-resurect-xdofrom
odoo:19.0-coverage-resurect-xdo

Conversation

@bt-admin
Copy link
Copy Markdown

@bt-admin bt-admin commented May 6, 2026

bt_gitbot

Sylsee and others added 3 commits May 6, 2026 08:49
When reusing a similar build as a quick result, first look for a build
with global_result == 'ok'. Fall back to the existing logic (any done
build not skipped/killed) only if no successful build is found.

Without this, a previously failed build could be used to shortcut
subsequent similar builds, propagating the failure without letting
them run.
When a breaking PR is assigned to multiple build errors, a stat button
now appears showing the count of duplicates. Clicking it opens a list
of errors sharing the same breaking PR, allowing the user to review
and potentially merge them.
The coverage was unusable because the test time was to long, and nothing
was made to make it run in parllel.

This new version allow to configure the coverage via config data to be
able to enable it on a parallel testing or single app test as an example
, and a custom step will be needed to recover all result, combine them
and generate a report.

Moreover, the --branch options adds and overhead of 20% but is needed to
but may not be 100% necessary (better have a coverage with no branch
than no coverage at all), so it is now optional.
It is also possible to enable the test context to know what test was
reached what branch, experimental but could be useful if the overhead is
not to high.
@Xavier-Do Xavier-Do force-pushed the 19.0-coverage-resurect-xdo branch from 7c84b9a to 2c6ccc3 Compare May 7, 2026 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants