Attention is currently required from: Edward O'Callaghan, Angel Pons, Anastasia Klimchuk. Hello build bot (Jenkins), Nico Huber, Edward O'Callaghan, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/55295
to look at the new patch set (#4).
Change subject: tests: Do not run a test if its driver is not built ......................................................................
tests: Do not run a test if its driver is not built
For all tests that exist as of today, drivers are built by default, however config options can be disabled and in that case test should not be run.
Technically, this is done by skipping the test scenario, so the test itself is running empty.
BUG=b:181803212 TEST=1) Tested by adding into tests/meson.build -DCONFIG_xxx=0 4 times (for every driver with test), and then running ninja test Result: corresponding test is running empty, all other tests are running fine
2) Running ninja test with default config settings (everything is enabled, no overriding in test meson). Result: all tests are running.
3) Replacing one of config options in the patch with CONFIG_JLINK_SPI which is disabled by default. Result: corresponding test is running empty.
Change-Id: Ic1c48e41f658045a608f46636071f478ba646f77 Signed-off-by: Anastasia Klimchuk aklm@chromium.org --- M tests/init_shutdown.c 1 file changed, 8 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/95/55295/4