Skip to content
This repository was archived by the owner on Dec 20, 2023. It is now read-only.

Commit ab024a5

Browse files
committed
Address duplicative unit and functional test invocation against the 'coverage' target.
1 parent b0b8783 commit ab024a5

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

src/test-apps/Makefile.am

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -359,9 +359,9 @@ endif
359359

360360
check_PROGRAMS += \
361361
TestTDM \
362-
TestPathStore \
363-
TestWdmUpdateEncoder \
364-
TestWdmUpdateResponse \
362+
TestPathStore \
363+
TestWdmUpdateEncoder \
364+
TestWdmUpdateResponse \
365365
$(NULL)
366366

367367
if WEAVE_BUILD_WARM
@@ -402,7 +402,6 @@ check_SCRIPTS += \
402402
$(NULL)
403403
endif # WEAVE_WITH_JAVA
404404

405-
406405
# Test applications that should be built but not installed that
407406
# require no network or complicated setup and should always be
408407
# built to ensure overall "build sanity".
@@ -1596,7 +1595,7 @@ $(WEAVE_LIBS_COVERAGE_SOURCE):
15961595
$(WEAVE_LIBS_COVERAGE_BUNDLE):
15971596
$(call create-directory)
15981597

1599-
$(WEAVE_COVERAGE_INFO): check | $(WEAVE_COVERAGE_BUNDLE)
1598+
$(WEAVE_COVERAGE_INFO): check-local | $(WEAVE_COVERAGE_BUNDLE)
16001599
$(call generate-coverage-report,${abs_top_builddir})
16011600

16021601
# collect all the *.gcno & *gcda files of libWeave.a for Weave Library coverage analysis, rename the *h file paths to make it a clean report
@@ -1618,7 +1617,7 @@ $(WEAVE_LIBS_COVERAGE_INFO): $(WEAVE_COVERAGE_INFO) | $(WEAVE_LIBS_COVERAGE_BUND
16181617
$(NL_V_GENHTML)$(GENHTML) $(NL_V_GENHTML_FLAGS) --config-file="$(abs_top_nlbuild_autotools_dir)/etc/lcov.config" $(WEAVE_LIBS_COVERAGE_INFO).clean --output-directory $(WEAVE_LIBS_COVERAGE_BUNDLE)
16191618
$(AM_V_at)rm -rf $(WEAVE_LIBS_COVERAGE_SOURCE)
16201619

1621-
coverage: $(WEAVE_LIBS_COVERAGE_INFO)
1620+
coverage-local: $(WEAVE_LIBS_COVERAGE_INFO)
16221621

16231622
clean-local: clean-local-coverage
16241623

0 commit comments

Comments
 (0)