Hi Richard, I'm assuming you meant "How about we add runtime coverage to the selftest" (it got autocorrected to "How are we all runtime coverage to the selftest").
I'll have a look at adding this to selftest to ensure that the test is executed, however for context you had suggested another solution in https://lists.openembedded.org/g/openembedded-core/topic/102891486#msg191495 , which is why I hadn't done it initially (it is not a problem at all, I can change the implementation, I'm just making sure you still have the context since the previous conversation was 6 months ago): "I was looking at your patches and thinking we might want to enable minidebuginfo in poky or perhaps OE-Core by default too." "We could assume we'll end up enabling this by default and if we do that, we could add a new runtime tests if it is configured knowing it will be by default in some of our builds." "and include running things under runqemu to test target pieces but a runtime test may be enough for this feature." On Tue, Jun 4, 2024 at 2:32 PM Richard Purdie < [email protected]> wrote: > On Tue, 2024-06-04 at 13:58 +0200, Etienne Cordonnier wrote: > > I'm talking specifically about test_systemd_coredump_minidebuginfo > > (the minidebuginfo test which I wrote > > in meta/lib/oeqa/runtime/cases/systemd.py, not the tests > > in meta/lib/oeqa/selftest/cases/minidebuginfo.py). > > Ok, sorry, I'm struggling to remember all the right context! > > I get nervous about blanket distro changes as whilst it adds test > coverage to minidebuginfo, it also removes test coverage from other > things (the non-mini systemd case). > > How are we all runtime coverage to the selftest in question, so it > boots and runs the systemd selftests? > > Specifically, we'd add config along the lines of: > > > DISTRO_FEATURES:append = " systemd" > TEST_SUITES = "ping ssh systemd" > > and then add a: > > bitbake('-c testimage core-image-minimal') > > to that testcase in selftest, which would then ensure we do run the > right tests? > > If we want to test both the systemd and non-systemd cases we could add > a second test to selftest instead which covers this? > > Cheers, > > Richard >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#200360): https://lists.openembedded.org/g/openembedded-core/message/200360 Mute This Topic: https://lists.openembedded.org/mt/106365956/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
