Skip to content

Commit a1a9126

Browse files
duda-patrykcfriedt
authored andcommitted
testsuite: Do not specify dump method if hardware coverage is not set
After #94079, CONFIG_COVERAGE_DUMP is enabled even if CONFIG_COVERAGE is not set. As a result CONFIG_ZTEST_NO_YIELD is disabled which has some implications on other tests. Coverage dump method is necessary only if coverage is collected on hardware, which is enabled by CONFIG_COVERAGE_GCOV. Let's specify default dump method, only if CONFIG_COVERAGE_GCOV is set. Fixes: #94079 Signed-off-by: Patryk Duda <[email protected]>
1 parent 302e4d2 commit a1a9126

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

subsys/testsuite/Kconfig.coverage

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,8 @@ config COVERAGE_GCOV_HEAP_SIZE
6161
data to be dumped over serial. If the value is 0, no buffer will be used,
6262
data will be dumped directly over serial.
6363

64+
if COVERAGE_GCOV
65+
6466
choice COVERAGE_DUMP_METHOD
6567
prompt "Method to dump coverage data"
6668
default COVERAGE_DUMP
@@ -78,6 +80,8 @@ config COVERAGE_SEMIHOST
7880

7981
endchoice
8082

83+
endif # COVERAGE_GCOV
84+
8185
config FORCE_COVERAGE
8286
bool "Force coverage"
8387
select HAS_COVERAGE_SUPPORT

0 commit comments

Comments
 (0)