Skip to content

dbus-broker: update 29 -> 31#575

Closed
bluca wants to merge 1 commit intoopenembedded:masterfrom
bluca:broker_31
Closed

dbus-broker: update 29 -> 31#575
bluca wants to merge 1 commit intoopenembedded:masterfrom
bluca:broker_31

Conversation

@bluca
Copy link
Contributor

@bluca bluca commented May 23, 2022

kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request May 23, 2022
The 1.6.1 incorporates the
0001-Fix-status-in-teardown-overriding-exit-code.patch backport patch.

Changelog:
==========

Fixed:
------
prevent teardown, teardown_file, and teardown_suite from overriding bats'
exit code by setting $status (e.g. via calling run) (openembedded#581, openembedded#575)
CRITICAL: this can return exit code 0 despite failed tests, thus preventing
your CI from reporting test failures! The regression happened in version 1.6.0.

Documentation:
--------------
corrected invalid documentation of run -N (had =N instead) (openembedded#579)
CRITICAL: using the incorrect form can lead to silent errors. See issue openembedded#578
for more details and how to find out if your tests are affected.

Signed-off-by: Diego Sueiro <diego.sueiro@arm.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
(cherry picked from commit 72fbb83)
Signed-off-by: Armin Kuster <akuster808@gmail.com>
@kraj
Copy link
Contributor

kraj commented May 31, 2022

merged with 962e3a3

@kraj kraj closed this May 31, 2022
mlasch pushed a commit to husqvarnagroup/smart-garden-gateway-yocto-meta-openembedded that referenced this pull request Jun 22, 2022
The 1.6.1 incorporates the
0001-Fix-status-in-teardown-overriding-exit-code.patch backport patch.

Changelog:
==========

Fixed:
------
prevent teardown, teardown_file, and teardown_suite from overriding bats'
exit code by setting $status (e.g. via calling run) (openembedded#581, openembedded#575)
CRITICAL: this can return exit code 0 despite failed tests, thus preventing
your CI from reporting test failures! The regression happened in version 1.6.0.

Documentation:
--------------
corrected invalid documentation of run -N (had =N instead) (openembedded#579)
CRITICAL: using the incorrect form can lead to silent errors. See issue openembedded#578
for more details and how to find out if your tests are affected.

Signed-off-by: Diego Sueiro <diego.sueiro@arm.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants