tests docs: fix lcov code coverage report generation
[vpp.git] / test / Makefile
index 2421086..27e7501 100644 (file)
@@ -13,23 +13,11 @@ ifndef TEST_DIR
        $(error TEST_DIR is not set)
 endif
 
-export TEST_BR = $(BR)/build-test
-export TEST_DOC_BR = $(TEST_BR)/doc
-export BUILD_TEST_SRC = $(TEST_BR)/src
+export TEST_BR = $(TEST_DIR)
 FAILED_DIR=/tmp/vpp-failed-unittests/
-PLUGIN_TEST_DIRS=$(shell find $(PLUGIN_SRC_DIR) -type d -name test -exec echo -n " -d {}" \;)
-CORE_TEST_DIRS=$(shell find $(WS_ROOT)/src -not \( -path $(INTERN_PLUGIN_SRC_DIR) -prune \) -type d -name test -exec echo -n " -d {}" \;)
-VPP_TEST_DIRS=$(shell ls -d $(TEST_DIR)$(PLUGIN_TEST_DIRS)$(CORE_TEST_DIRS) $(EXTERN_TESTS))
-VPP_TEST_SRC=$(shell for dir in $(VPP_TEST_DIRS) ; do ls $$dir/*.py 2>/dev/null; done)
-
-.PHONY: verify-no-running-vpp
-
-ifdef VPP_ZOMBIE_NOCHECK
-VPP_PIDS=
-else
-VPP_PIDS=$(shell pgrep -d, -x vpp_main)
-endif
+VPP_TEST_DIRS=$(shell ls -d $(TEST_DIR) $(EXTERN_TESTS))
 
+FORCE_NO_WIPE=0
 ifeq ($(DEBUG),gdb)
 FORCE_FOREGROUND=1
 else ifeq ($(DEBUG),gdbserver)
@@ -40,6 +28,9 @@ else ifeq ($(DEBUG),gdbserver-all)
 FORCE_FOREGROUND=1
 else ifeq ($(DEBUG),core)
 FORCE_FOREGROUND=1
+else ifeq ($(DEBUG),attach)
+FORCE_FOREGROUND=1
+FORCE_NO_WIPE=1
 else ifeq ($(STEP),yes)
 FORCE_FOREGROUND=1
 else ifeq ($(STEP),y)
@@ -59,32 +50,12 @@ PROFILE_SORT_BY=cumtime
 endif
 
 ifeq ($(PROFILE),1)
-PYTHON_PROFILE_OPTS=-m cProfile $(PROFILE_OUTPUT_OPTS) -s $(PROFILE_SORT_BY)
+PYTHON_OPTS="-m cProfile $(PROFILE_OUTPUT_OPTS) -s $(PROFILE_SORT_BY)"
 FORCE_FOREGROUND=1
 endif
 
-verify-no-running-vpp:
-       @if [ "$(VPP_PIDS)" != "" ]; then \
-               echo; \
-               echo "*** Existing vpp processes detected (PID(s): $(VPP_PIDS)). Running tests under these conditions is not supported. ***"; \
-               echo; \
-               ps -fp $(VPP_PIDS);\
-               echo; \
-               false; \
-       fi
-
-UNITTEST_EXTRA_OPTS=
-UNITTEST_FAILFAST_OPTS=
-
-ifeq ($(FAILFAST),1)
-UNITTEST_EXTRA_OPTS=-f
-endif
-
-ifneq ($(EXTERN_TESTS),)
-UNITTEST_EXTRA_OPTS=$(UNITTEST_FAILFAST_OPTS) -d $(EXTERN_TESTS)
-endif
-
-VENV_PATH=$(TEST_BR)/venv
+VENV_BR_DIR=$(BR)/test
+VENV_PATH=$(VENV_BR_DIR)/venv
 
 ifeq ($(TEST_DEBUG),1)
 VENV_RUN_DIR:=$(VENV_PATH)/run-debug
@@ -98,18 +69,18 @@ else
 PYTHON_INTERP=$(PYTHON)
 endif
 
-empty:=
-space:= $(empty) $(empty)
-export PYTHONPATH=$(subst $(space),:,$(VPP_TEST_DIRS))
-export PYTHONPYCACHEPREFIX=$(TEST_BR)/pycache
+ifeq ($(V),)
+V=0
+endif
 
 PYTHON_VERSION=$(shell $(PYTHON_INTERP) -c 'import sys; print(sys.version_info.major)')
-PIP_VERSION=20.1.1
+PIP_VERSION=22.0.4
 # Keep in sync with requirements.txt
-PIP_TOOLS_VERSION=5.1.2
+PIP_TOOLS_VERSION=6.6.0
+PIP_SETUPTOOLS_VERSION=62.1.0
 PYTHON_DEPENDS=requirements-$(PYTHON_VERSION).txt
 SCAPY_SOURCE=$(shell find $(VENV_PATH)/lib/python* -name site-packages)
-BUILD_COV_DIR=$(TEST_BR)/coverage
+BUILD_COV_DIR=$(BR)/test-coverage
 
 PIP_TOOLS_INSTALL_DONE=$(VENV_RUN_DIR)/pip-tools-install-$(PYTHON_VERSION)-$(PIP_TOOLS_VERSION).done
 PIP_INSTALL_DONE=$(VENV_RUN_DIR)/pip-install-$(PYTHON_VERSION)-$(PIP_VERSION).done
@@ -124,22 +95,24 @@ PAPI_WIPE_DIST=$(WS_ROOT)/src/vpp-api/vapi/__pycache__ \
 $(PIP_TOOLS_INSTALL_DONE):
        @rm -rf $(VENV_PATH)
        @mkdir -p $(VENV_RUN_DIR)
-       @virtualenv $(VENV_PATH) -p $(PYTHON_INTERP)
+       @$(PYTHON_INTERP) -m venv $(VENV_PATH)
        # pip version pinning
        @bash -c "source $(VENV_PATH)/bin/activate && \
-                 $(PYTHON_INTERP) -m pip install pip===$(PIP_VERSION)"
+                 python3 -m pip install pip===$(PIP_VERSION)"
+       @bash -c "source $(VENV_PATH)/bin/activate && \
+                 python3 -m pip install pip-tools===$(PIP_TOOLS_VERSION)"
        @bash -c "source $(VENV_PATH)/bin/activate && \
-                 $(PYTHON_INTERP) -m pip install pip-tools===$(PIP_TOOLS_VERSION)"
+                 python3 -m pip install setuptools===$(PIP_SETUPTOOLS_VERSION)"
        @touch $@
 
 $(PYTHON_DEPENDS): requirements.txt
        @bash -c "source $(VENV_PATH)/bin/activate && \
                  CUSTOM_COMPILE_COMMAND='make test-refresh-deps (or update requirements.txt)' \
-                 $(PYTHON_INTERP) -m piptools compile -q --generate-hashes requirements.txt --output-file $@"
+                 python3 -m piptools compile -q --generate-hashes requirements.txt --output-file $@"
 
 $(PIP_INSTALL_DONE): $(PIP_TOOLS_INSTALL_DONE) $(PYTHON_DEPENDS)
        @bash -c "source $(VENV_PATH)/bin/activate && \
-                 $(PYTHON_INTERP) -m piptools sync $(PYTHON_DEPENDS)"
+                 python3 -m piptools sync $(PYTHON_DEPENDS)"
        @touch $@
 
 $(PIP_PATCH_DONE): $(PIP_INSTALL_DONE)
@@ -154,7 +127,7 @@ $(PIP_PATCH_DONE): $(PIP_INSTALL_DONE)
        touch $@
 
 $(PAPI_INSTALL_DONE): $(PIP_PATCH_DONE)
-       @bash -c "source $(VENV_PATH)/bin/activate && $(PYTHON_INTERP) -m pip install -e $(PAPI_PYTHON_SRC_DIR)"
+       @bash -c "source $(VENV_PATH)/bin/activate && python3 -m pip install -e $(PAPI_PYTHON_SRC_DIR)"
        @touch $@
 
 .PHONY: refresh-deps
@@ -171,33 +144,126 @@ else
 PLUGIN_SRC_DIR=$(INTERN_PLUGIN_SRC_DIR)
 endif
 
-define retest-func
-@env FORCE_FOREGROUND=$(FORCE_FOREGROUND) FAILED_DIR=$(FAILED_DIR) VENV_PATH=$(VENV_PATH) scripts/setsid_wrapper.sh $(FORCE_FOREGROUND) $(VENV_PATH)/bin/activate $(PYTHON_INTERP) $(PYTHON_PROFILE_OPTS) $(BUILD_TEST_SRC)/run_tests.py -d $(BUILD_TEST_SRC) $(UNITTEST_EXTRA_OPTS) || env FAILED_DIR=$(FAILED_DIR) COMPRESS_FAILED_TEST_LOGS=$(COMPRESS_FAILED_TEST_LOGS) scripts/compress_failed.sh
-endef
-
 .PHONY: sanity
 
-ifeq ($(SANITY),no)
-SANITY_IMPORT_VPP_PAPI_CMD=true
-SANITY_RUN_VPP_CMD=true
-else
-SANITY_IMPORT_VPP_PAPI_CMD=source $(VENV_PATH)/bin/activate && $(PYTHON_INTERP) $(BUILD_TEST_SRC)/sanity_import_vpp_papi.py
-SANITY_RUN_VPP_CMD=source $(VENV_PATH)/bin/activate && $(PYTHON_INTERP) $(BUILD_TEST_SRC)/sanity_run_vpp.py
-endif
-
 ifndef TEST_JOBS
 PARALLEL_ILLEGAL=0
+TEST_JOBS=1
 else ifeq ($(FORCE_FOREGROUND),0)
 PARALLEL_ILLEGAL=0
-else ifeq ($(TEST_JOBS),auto)
-PARALLEL_ILLEGAL=0
-else ifeq ($(TEST_JOBS),1)
+else ifneq ($(findstring $(TEST_JOBS),1 auto),)
 PARALLEL_ILLEGAL=0
 else
 PARALLEL_ILLEGAL=1
 endif
 
-sanity: test-dep verify-no-running-vpp
+ifneq ($(DEBUG),)
+SANITY=no
+endif
+
+ifneq ($(findstring $(SANITY),0 n no),)
+SANITY_IMPORT_VPP_PAPI_CMD=true
+ARG0=
+else
+SANITY_IMPORT_VPP_PAPI_CMD=source $(VENV_PATH)/bin/activate && $(PYTHON_INTERP) sanity_import_vpp_papi.py
+ARG0=--sanity
+endif
+
+ARG1=
+ifneq ($(findstring $(FAILFAST),1 y yes),)
+ARG1=--failfast
+endif
+
+ARG2=
+ifneq ($(findstring $(EXTENDED_TESTS),1 y yes),)
+ARG2=--extended
+endif
+
+ARG3=
+ifneq ($(EXTERN_TESTS),)
+ARG3=--test-src-dir $(EXTERN_TESTS)
+endif
+
+ARG4=
+ifneq ($(findstring $(FORCE_FOREGROUND),1 y yes),)
+ARG4=--force-foreground
+endif
+
+ARG5=
+ifneq ($(findstring $(COREDUMP_COMPRESS),1 y yes),)
+ARG5=--compress-core
+endif
+
+ARG6=
+ifneq ($(findstring $(STEP),1 y yes),)
+ARG6=--step
+endif
+
+ARG7=
+ifneq ($(findstring $(TESTS_GCOV),1 y yes),)
+ARG7=--gcov
+endif
+
+ARG8=
+ifneq ($(EXTERN_PLUGINS),)
+ARG8=--extern-plugin-dir=$(EXTERN_PLUGINS)
+endif
+
+ARG9=
+ifneq ($(DEBUG),)
+ARG9=--debug=$(DEBUG)
+endif
+
+ARG10=
+ifneq ($(COREDUMP_SIZE),)
+ARG10=--coredump-size=$(COREDUMP_SIZE)
+endif
+
+ARG11=
+ifneq ($(VARIANT),)
+ARG11=--variant=$(VARIANT)
+endif
+
+ARG12=--cache-vpp-output
+ifneq ($(findstring $(CACHE_OUTPUT),0 n no),)
+ARG12=
+endif
+
+ARG13=
+ifneq ($(MAX_VPP_CPUS),)
+ARG13=--max-vpp-cpus=$(MAX_VPP_CPUS)
+endif
+
+ARG14=
+ifneq ($(TIMEOUT),)
+ARG14=--timeout=$(TIMEOUT)
+endif
+
+ARG15=
+ifneq ($(findstring $(TEST_DEBUG),1 y yes),)
+ARG15=--debug-framework
+endif
+
+ARG16=
+ifneq ($(findstring $(API_FUZZ),1 y yes),)
+ARG16=--api-fuzz=on
+endif
+
+ARG17=
+ifneq ($(EXTERN_APIDIR),)
+ARG17=--extern-apidir=$(EXTERN_APIDIR)
+endif
+
+EXTRA_ARGS=$(ARG0) $(ARG1) $(ARG2) $(ARG3) $(ARG4) $(ARG5) $(ARG6) $(ARG7) $(ARG8) $(ARG9) $(ARG10) $(ARG11) $(ARG12) $(ARG13) $(ARG14) $(ARG15) $(ARG16) $(ARG17)
+
+RUN_TESTS_ARGS=--failed-dir=$(FAILED_DIR) --verbose=$(V) --jobs=$(TEST_JOBS) --filter=$(TEST) --retries=$(RETRIES) --venv-dir=$(VENV_PATH) --vpp-ws-dir=$(WS_ROOT) --vpp-tag=$(TAG) --rnd-seed=$(RND_SEED) --vpp-worker-count="$(VPP_WORKER_COUNT)" --keep-pcaps $(PLUGIN_PATH_ARGS) $(TEST_PLUGIN_PATH_ARGS) $(EXTRA_ARGS)
+RUN_SCRIPT_ARGS=--python-opts=$(PYTHON_OPTS)
+
+define retest-func
+@scripts/run.sh $(RUN_SCRIPT_ARGS) $(RUN_TESTS_ARGS) || env FAILED_DIR=$(FAILED_DIR) COMPRESS_FAILED_TEST_LOGS=$(COMPRESS_FAILED_TEST_LOGS) scripts/compress_failed.sh
+endef
+
+sanity: test-dep
        @bash -c "test $(PARALLEL_ILLEGAL) -eq 0 ||\
            (echo \"*******************************************************************\" &&\
                 echo \"* Sanity check failed, TEST_JOBS is not 1 or 'auto' and DEBUG, STEP or PROFILE is set\" &&\
@@ -211,29 +277,15 @@ sanity: test-dep verify-no-running-vpp
                 echo \"* 2. execute debugger:   gdb python -ex 'run sanity_import_vpp_papi.py'\" &&\
                 echo \"*******************************************************************\" &&\
                 false)"
-       @bash -c "$(SANITY_RUN_VPP_CMD) ||\
-               (echo \"*******************************************************************\" &&\
-                echo \"* Sanity check failed, cannot run vpp\" &&\
-                echo \"*******************************************************************\" &&\
-                false)"
-
-.PHONY: ext-test-apps
-ext-test-apps:
-       make -C ext test-apps
-
-$(BUILD_TEST_SRC): verify-env
-       @rm -rf $@
-       @mkdir -p $@
-       @for file in $(VPP_TEST_SRC); do if [ ! -L $$file ] && [ ! -e $(BUILD_TEST_SRC)/$$(basename $$file) ] ; then ln -s $$file $(BUILD_TEST_SRC) ; fi ; done
 
 $(FAILED_DIR): reset
        @mkdir -p $@
 
 .PHONY: test-dep
-test-dep: $(BUILD_TEST_SRC) $(PAPI_INSTALL_DONE) $(FAILED_DIR)
+test-dep: $(PAPI_INSTALL_DONE) $(FAILED_DIR)
 
 .PHONY: test
-test: test-dep ext-test-apps sanity
+test: test-dep sanity
        $(call retest-func)
 
 .PHONY: retest
@@ -243,20 +295,17 @@ retest: verify-env sanity $(FAILED_DIR)
 .PHONY: shell
 shell: test-dep
        @echo "source $(VENV_PATH)/bin/activate;\
-               cd $(BUILD_TEST_SRC);\
-               export PYTHONPATH=$(PYTHONPATH);\
                export RND_SEED=$(RND_SEED);\
                echo '***';\
                echo PYTHONPATH=$(PYTHONPATH);\
                echo RND_SEED=$(RND_SEED);\
                echo VPP_BUILD_DIR=$(VPP_BUILD_DIR);\
-               echo VPP_BIN=$(VPP_BIN);\
                echo VPP_PLUGIN_PATH=$(VPP_PLUGIN_PATH);\
                echo VPP_TEST_PLUGIN_PATH=$(VPP_TEST_PLUGIN_PATH);\
                echo VPP_INSTALL_PATH=$(VPP_INSTALL_PATH);\
                echo EXTERN_TESTS=$(EXTERN_TESTS);\
                echo EXTERN_PLUGINS=$(EXTERN_PLUGINS);\
-                echo EXTERN_COV_DIR=$(EXTERN_COV_DIR);\
+        echo EXTERN_COV_DIR=$(EXTERN_COV_DIR);\
                echo LD_LIBRARY_PATH=$(LD_LIBRARY_PATH);\
                echo '***';\
                exec </dev/tty" | bash -i
@@ -264,48 +313,49 @@ shell: test-dep
 .PHONY: reset
 reset:
        @rm -f /dev/shm/vpp-unittest-*
-       @rm -rf /tmp/vpp-unittest-*
+       @if [ $(FORCE_NO_WIPE) -eq "0" ] ; then rm -rf /tmp/vpp-unittest-*;  fi
        @rm -f /tmp/api_post_mortem.*
        @rm -rf $(FAILED_DIR)
+       @rm -rf /tmp/vpp-vm-tests
 
 .PHONY: wipe
 wipe: reset
-       @make -C ext clean
-       @rm -rf $(VENV_PATH)
+       @rm -rf $(VENV_BR_DIR)
        @rm -rf $(patsubst %,%/__pycache__, $(VPP_TEST_DIRS))
-       @rm -rf  $(BUILD_TEST_SRC)
-
-$(TEST_DOC_BR): $(PIP_INSTALL_DONE)
-       @mkdir -p $@
-       @bash -c "source $(VENV_PATH)/bin/activate && make -C doc html"
-
-.PHONY: doc
-doc: $(BUILD_TEST_SRC) $(PIP_PATCH_DONE) $(TEST_DOC_BR)
-       @echo
-       @echo "Test Documentation URL: $(TEST_DOC_BR)/html/index.html"
-       @echo "Run 'make test-wipe-doc test-doc' to rebuild the test docs"
-       @echo
-
-.PHONY: wipe-doc
-wipe-doc:
-       @rm -rf $(TEST_DOC_BR)
 
 $(BUILD_COV_DIR):
        @mkdir -p $@
 
-.PHONY: cov
-cov: wipe-cov test-dep ext $(BUILD_COV_DIR)
+.PHONY: cov-prep
+cov-prep: test-dep
        @lcov --zerocounters --directory $(VPP_BUILD_DIR)
        @test -z "$(EXTERN_COV_DIR)" || lcov --zerocounters --directory $(EXTERN_COV_DIR)
-       $(call retest-func)
-       @lcov --capture --directory $(VPP_BUILD_DIR) --output-file $(BUILD_COV_DIR)/coverage.info
-       @test -z "$(EXTERN_COV_DIR)" || lcov --capture --directory $(EXTERN_COV_DIR) --output-file $(BUILD_COV_DIR)/extern-coverage.info
-       @genhtml $(BUILD_COV_DIR)/coverage.info --output-directory $(BUILD_COV_DIR)/html
-       @test -z "$(EXTERN_COV_DIR)" || genhtml $(BUILD_COV_DIR)/extern-coverage.info --output-directory $(BUILD_COV_DIR)/extern-html
+
+.PHONY: cov-post
+cov-post: wipe-cov $(BUILD_COV_DIR)
+       @lcov --capture \
+               --directory $(VPP_BUILD_DIR) \
+               --output-file $(BUILD_COV_DIR)/coverage.info
+       @test -z "$(EXTERN_COV_DIR)" || \
+               lcov --capture \
+               --directory $(EXTERN_COV_DIR) \
+               --output-file $(BUILD_COV_DIR)/extern-coverage.info
+       @lcov --remove $(BUILD_COV_DIR)/coverage.info \
+               "/usr/include/*" "*/build-root/*" "/opt/*" "/usr/lib/*" \
+               -o $(BUILD_COV_DIR)/coverage-filtered.info
+       @genhtml $(BUILD_COV_DIR)/coverage-filtered.info \
+               --output-directory $(BUILD_COV_DIR)/html
+       @test -z "$(EXTERN_COV_DIR)" || \
+               genhtml $(BUILD_COV_DIR)/extern-coverage.info \
+                       --output-directory $(BUILD_COV_DIR)/extern-html
        @echo
        @echo "Build finished. Code coverage report is in $(BUILD_COV_DIR)/html/index.html"
        @test -z "$(EXTERN_COV_DIR)" || echo "Code coverage report for out-of-tree objects is in $(BUILD_COV_DIR)/extern-html/index.html"
 
+.PHONY: cov
+cov:
+       make -C . cov-prep test cov-post
+
 .PHONY: wipe-cov
 wipe-cov: wipe
        @rm -rf $(BUILD_COV_DIR)
@@ -315,36 +365,38 @@ wipe-papi:
        @rm -rf $(PAPI_INSTALL_DONE) $(PAPI_WIPE_DIST)
 
 .PHONY: wipe-all
-wipe-all: wipe wipe-papi wipe-doc wipe-cov
+wipe-all: wipe wipe-papi wipe-cov
        @rm -rf $(TEST_BR)
 
-.PHONY: checkstyle-diff
-checkstyle-diff: $(BUILD_TEST_SRC) $(PIP_INSTALL_DONE)
-       @bash -c "source $(VENV_PATH)/bin/activate &&\
-                 $(PYTHON_INTERP) -m pip install pycodestyle"
+.PHONY: start-gdb
+start-gdb: sanity
+       @bash -c "source $(VENV_PATH)/bin/activate && python3 -c 'from debug import start_vpp_in_gdb; start_vpp_in_gdb()' $(RUN_TESTS_ARGS)"
+
+.PHONY: checkstyle-python-all
+checkstyle-python-all: $(PIP_INSTALL_DONE)
        @bash -c "source $(VENV_PATH)/bin/activate &&\
-               cd $(WS_ROOT) && git diff --name-only --no-color --relative HEAD~1 ':!*.patch' | grep '.py$$' | xargs -n 1 -I XXX \
-               pycodestyle --show-source --ignore=W504,E126,E241,E226,E305,E704,E741,E722 -v XXX ||\
-               (echo \"*********************************************************************\" &&\
-                echo \"* Test framework PEP8 compliance check FAILED (checked changed files)\" &&\
-                echo \"*********************************************************************\" &&\
-                false)"
-       @echo "*********************************************************************"
-       @echo "* Test framework PEP8 compliance check passed (checked changed files)"
-       @echo "*********************************************************************"
+               black -t py39 --check --diff $(WS_ROOT) ||\
+               (echo \"*************************************************************************\" &&\
+               echo \"* Test framework PEP8 compliance check FAILED (maybe: make fixstyle-python)\" &&\
+               echo \"*************************************************************************\" &&\
+               false)"
+       @echo "*******************************************************************"
+       @echo "* Test framework PEP8 compliance check passed"
+       @echo "*******************************************************************"
 
 .PHONY: checkstyle
-checkstyle: $(BUILD_TEST_SRC) $(PIP_INSTALL_DONE)
-       @bash -c "source $(VENV_PATH)/bin/activate &&\
-                 $(PYTHON_INTERP) -m pip install pycodestyle"
+checkstyle: checkstyle-python-all
+
+.PHONY: fixstyle-python-all
+fixstyle-python-all: $(PIP_INSTALL_DONE)
        @bash -c "source $(VENV_PATH)/bin/activate &&\
-               pycodestyle --show-source --ignore=W504,E126,E241,E226,E305,E704,E741,E722 -v $(BUILD_TEST_SRC)/*.py ||\
-               (echo \"*******************************************************************\" &&\
-                echo \"* Test framework PEP8 compliance check FAILED (checked all files)\" &&\
-                echo \"*******************************************************************\" &&\
-                false)"
+               black -t py39 $(WS_ROOT) ||\
+               (echo \"*************************************************************************\" &&\
+               echo \"* Test framework PEP8 compliance check FAILED (maybe: make fixstyle-python)\" &&\
+               echo \"*************************************************************************\" &&\
+               false)"
        @echo "*******************************************************************"
-       @echo "* Test framework PEP8 compliance check passed (checked all files)"
+       @echo "* Test framework PEP8 compliance check passed"
        @echo "*******************************************************************"
 
 .PHONY: help
@@ -353,84 +405,191 @@ help:
        @echo ""
        @echo " test                   - build and run (basic) functional tests"
        @echo " test-debug             - build and run (basic) functional tests (debug build)"
+       @echo " test-cov               - generate code coverage report for functional tests"
+       @echo " test-cov-prep          - coverage phase #1 : prepare lcov"
+       @echo " test-cov-build         - coverage phase #2 : build gcov image & run tests against it (use TEST=)"
+       @echo " test-cov-post          - coverage phase #3 : generate lcov html report"
        @echo " test-all               - build and run functional and extended tests"
        @echo " test-all-debug         - build and run functional and extended tests (debug build)"
+       @echo " test-all-cov           - generate code coverage report for functional and extended tests"
        @echo " retest                 - run functional tests"
        @echo " retest-debug           - run functional tests (debug build)"
        @echo " retest-all             - run functional and extended tests"
        @echo " retest-all-debug       - run functional and extended tests (debug build)"
-       @echo " test-cov               - generate code coverage report for test framework"
-       @echo " test-gcov                      - build and run functional tests (gcov build)"
-
        @echo " test-wipe              - wipe (temporary) files generated by unit tests"
        @echo " test-wipe-cov          - wipe code coverage report for test framework"
-       @echo " test-wipe-doc          - wipe documentation for test framework"
        @echo " test-wipe-papi         - rebuild vpp_papi sources"
-       @echo " test-wipe-all          - wipe (temporary) files generated by unit tests, docs, and coverage"
+       @echo " test-wipe-all          - wipe (temporary) files generated by unit tests, and coverage"
        @echo " test-shell             - enter shell with test environment"
        @echo " test-shell-debug       - enter shell with test environment (debug build)"
-       @echo " test-checkstyle      - check PEP8 compliance for test framework"
-       @echo " test-refresh-deps    - refresh the Python dependencies for the tests"
-       @echo ""
-       @echo "Arguments controlling test runs:"
-       @echo " V=[0|1|2]              - set test verbosity level"
-       @echo "                          0=ERROR, 1=INFO, 2=DEBUG"
-       @echo " TEST_JOBS=[<n>|auto]   - use <n> parallel processes for test execution or automatic discovery of maximum acceptable processes (default: 1)"
-       @echo " CACHE_OUTPUT=[0|1]     - cache VPP stdout/stderr and log as one block after test finishes (default: 1)"
-       @echo " FAILFAST=[0|1]         - fail fast if 1, complete all tests if 0"
-       @echo " TIMEOUT=<timeout>      - fail test suite if any single test takes longer than <timeout> (in seconds) to finish (default: 600)"
-       @echo " RETRIES=<n>            - retry failed tests <n> times"
-       @echo " DEBUG=<type>           - set VPP debugging kind"
-       @echo "    DEBUG=core          - detect coredump and load it in gdb on crash"
-       @echo "    DEBUG=gdb           - allow easy debugging by printing VPP PID"
-       @echo "                          and waiting for user input before running"
-       @echo "                          and tearing down a testcase"
-       @echo "    DEBUG=gdbserver     - run gdb inside a gdb server, otherwise"
-       @echo "                          same as above"
-       @echo " STEP=[yes|no]          - ease debugging by stepping through a testcase"
-       @echo " SANITY=[yes|no]        - perform sanity import of vpp-api/sanity vpp run before running tests (default: yes)"
-       @echo " EXTENDED_TESTS=[1|y]   - used by '[re]test-all' & '[re]test-all-debug' to run extended tests"
-       @echo " TEST=<filter>          - filter the set of tests:"
-       @echo "    by file-name        - only run tests from specified file, e.g. TEST=test_bfd selects all tests from test_bfd.py"
-       @echo "    by file-suffix      - same as file-name, but 'test_' is omitted e.g. TEST=bfd selects all tests from test_bfd.py"
-       @echo "    by wildcard         - wildcard filter is <file>.<class>.<test function>, each can be replaced by '*'"
-       @echo "                          e.g. TEST='test_bfd.*.*' is equivalent to above example of filter by file-name"
-       @echo "                               TEST='bfd.*.*' is equivalent to above example of filter by file-suffix"
-       @echo "                               TEST='bfd.BFDAPITestCase.*' selects all tests from test_bfd.py which are part of BFDAPITestCase class"
-       @echo "                               TEST='bfd.BFDAPITestCase.test_add_bfd' selects a single test named test_add_bfd from test_bfd.py/BFDAPITestCase"
-       @echo "                               TEST='*.*.test_add_bfd' selects all test functions named test_add_bfd from all files/classes"
-       @echo ""
-       @echo " VARIANT=<variant>      - specify which march node variant to unit test"
-       @echo "                          e.g. VARIANT=skx test the skx march variants"
-       @echo "                          e.g. VARIANT=icl test the icl march variants"
-       @echo ""
-       @echo " VPP_ZOMBIE_NOCHECK=1   - skip checking for vpp (zombie) processes (CAUTION)"
-       @echo " COREDUMP_SIZE=<size>   - pass <size> as unix { coredump-size <size> } argument to vpp"
-       @echo "                          e.g. COREDUMP_SIZE=4g"
-       @echo "                               COREDUMP_SIZE=unlimited"
-       @echo " COREDUMP_COMPRESS=1    - compress core files if not debugging them"
-       @echo " EXTERN_TESTS=<path>    - path to out-of-tree test_<name>.py files containing test cases"
-       @echo " EXTERN_PLUGINS=<path>  - path to out-of-tree plugins to be loaded by vpp under test"
-       @echo " EXTERN_COV_DIR=<path>  - path to out-of-tree prefix, where source, object and .gcda files can be found for coverage report"
-       @echo ""
-       @echo " PROFILE=1              - enable profiling of test framework via cProfile module"
-       @echo " PROFILE_SORT_BY=opt    - sort profiling report by opt - consult cProfile documentation for possible values (default: cumtime)"
-       @echo " PROFILE_OUTPUT=file    - output profiling info to file - use absolute path (default: stdout)"
-       @echo ""
-       @echo " TEST_DEBUG=1           - turn on debugging of the test framework itself (expert)"
-       @echo ""
-       @echo " SKIP_AARCH64=1         - skip tests that are failing on the ARM platorm in FD.io CI"
-       @echo ""
-       @echo " RND_SEED=seed          - Seed RND with given seed"
-       @echo ""
-       @echo "Creating test documentation"
-       @echo " test-doc               - generate documentation for test framework"
-       @echo " test-wipe-doc          - wipe documentation for test framework"
-       @echo ""
-       @echo "Creating test code coverage report"
-       @echo " test-cov               - generate code coverage report for test framework"
-       @echo " test-wipe-cov          - wipe code coverage report for test framework"
+       @echo " test-refresh-deps      - refresh the Python dependencies for the tests"
+       @echo ""
+       @echo "Environment variables controlling test runs:"
+       @echo ""
+       @echo "   V=[0|1|2]"
+       @echo "       set test verbosity level: 0=ERROR, 1=INFO, 2=DEBUG"
+       @echo ""
+       @echo "   TEST_JOBS=[<n>|auto]"
+       @echo "       use at most <n> parallel python processes for test"
+       @echo "       execution, if auto, set to number of available cpus"
+       @echo "       (default: 1)"
+       @echo ""
+       @echo "   MAX_VPP_CPUS=[<n>|auto]"
+       @echo "       use at most <n> cpus for running vpp"
+       @echo "       'auto' sets to number of available cpus"
+       @echo "       (default: auto)"
+       @echo ""
+       @echo "   CACHE_OUTPUT=[0|n|no]"
+       @echo "       disable caching VPP stdout/stderr and logging it"
+       @echo "       as one block after test finishes"
+       @echo "       (default: yes)"
+       @echo ""
+       @echo "   FAILFAST=[1|y|yes]"
+       @echo "       if enabled, stop running tests on first failure"
+       @echo "       otherwise finish running whole suite"
+       @echo "       (default: no)"
+       @echo ""
+       @echo "   TIMEOUT=<timeout>"
+       @echo "       fail test suite if any single test takes longer"
+       @echo "       than <timeout> (in seconds) to finish"
+       @echo "       (default: 600)"
+       @echo ""
+       @echo "   RETRIES=<n>"
+       @echo "       retry failed tests <n> times"
+       @echo "       (default: 0)"
+       @echo ""
+       @echo "   DEBUG=<type>"
+       @echo "       configure VPP debugging:"
+       @echo "       DEBUG=core"
+       @echo "           detect coredump and load it in gdb on crash"
+       @echo ""
+       @echo "       DEBUG=gdb"
+       @echo "           print VPP PID and wait for user input before"
+       @echo "           running and tearing down a testcase, allowing"
+       @echo "           easy gdb attach"
+       @echo ""
+       @echo "       DEBUG=gdbserver"
+       @echo "           same as above, but run gdb inside a gdb server"
+       @echo ""
+       @echo "       DEBUG=attach"
+       @echo "           attach to existing vpp in running in gdb"
+       @echo "           (see test-start-vpp-in-gdb)"
+       @echo "       (default: none)"
+       @echo ""
+       @echo "   STEP=[1|y|yes]"
+       @echo "       enable stepping through a testcase"
+       @echo "       (default: no)"
+       @echo ""
+       @echo "   SANITY=[0|n|no]"
+       @echo "       disable sanity import of vpp-api/vpp sanity"
+       @echo "       run before running tests"
+       @echo "       (default: yes)"
+       @echo ""
+       @echo "   EXTENDED_TESTS=[1|y|yes]"
+       @echo "       run extended tests"
+       @echo "       (default: no)"
+       @echo ""
+       @echo "   TEST=<filter>,[<filter>],..."
+       @echo "       only run tests matching one or more comma-delimited"
+       @echo "       filter expressions"
+       @echo ""
+       @echo "       simple filter:"
+       @echo "           file name or file suffix select all tests from a file"
+       @echo "           examples:"
+       @echo "               TEST=test_bfd"
+       @echo "               TEST=bfd"
+       @echo "                    equivalent expressions selecting all"
+       @echo "                    tests defined in test_bfd.py"
+       @echo ""
+       @echo "       wildcard filter:"
+       @echo "           advanced filtering based on test file, test class"
+       @echo "           and test function"
+       @echo "           each filter expression is in the form of"
+       @echo "               <file>.<class>.<test function>"
+       @echo "           each of the tokens can be left empty or replaced"
+       @echo "           with '*' to select all objects available"
+       @echo "           examples:"
+       @echo "               TEST=test_bfd.*.*"
+       @echo "               TEST=test_bfd.."
+       @echo "               TEST=bfd.*.*"
+       @echo "               TEST=bfd.."
+       @echo "                    select all tests defined in test_bfd.py"
+       @echo "               TEST=bfd.BFDAPITestCase.*"
+       @echo "               TEST=bfd.BFDAPITestCase."
+       @echo "                    select all tests from test_bfd.py"
+       @echo "                    which are part of BFDAPITestCase class"
+       @echo "               TEST=bfd.BFDAPITestCase.test_add_bfd"
+       @echo "                    select a single test named test_add_bfd"
+       @echo "                    from test_bfd.py/BFDAPITestCase"
+       @echo "               TEST=..test_add_bfd"
+       @echo "               TEST=*.*.test_add_bfd"
+       @echo "                    select all test functions named test_add_bfd"
+       @echo "                    from all files/classes"
+       @echo "               TEST=bfd,ip4,..test_icmp_error"
+       @echo "                    select all test functions in test_bfd.py,"
+       @echo "                    test_ip4.py and all test functions named"
+       @echo "                    'test_icmp_error' in all files"
+       @echo "       (default: '')"
+       @echo ""
+       @echo "   VARIANT=<variant>"
+       @echo "       specify which march node variant to unit test"
+       @echo "           e.g. VARIANT=skx test the skx march variants"
+       @echo "           e.g. VARIANT=icl test the icl march variants"
+       @echo "       (default: '')"
+       @echo ""
+       @echo "   COREDUMP_SIZE=<size>"
+       @echo "       pass <size> as unix { coredump-size <size> } argument"
+       @echo "       to vpp, e.g. COREDUMP_SIZE=4g or COREDUMP_SIZE=unlimited"
+       @echo "       (default: '')"
+       @echo ""
+       @echo "   COREDUMP_COMPRESS=[1|y|yes]"
+       @echo "       if no debug option is set, compress any core files"
+       @echo "       (default: no)"
+       @echo ""
+       @echo "   EXTERN_TESTS=<path>"
+       @echo "       include out-of-tree test_*.py files under <path>"
+       @echo "       (default: '')"
+       @echo ""
+       @echo "   EXTERN_PLUGINS=<path>"
+       @echo "       load out-of-tree vpp plugins in <path>"
+       @echo "       (default: '')"
+       @echo ""
+       @echo "   EXTERN_COV_DIR=<path>"
+       @echo "       path to out-of-tree prefix, where source, object"
+       @echo "       and .gcda files can be found for coverage report"
+       @echo "       (default: '')"
+       @echo ""
+       @echo "   PROFILE=[1|y|yes]"
+       @echo "       enable profiling of test framework via cProfile module"
+       @echo "       (default: no)"
+       @echo ""
+       @echo "   PROFILE_SORT_BY=opt"
+       @echo "       sort profiling report by opt - see cProfile documentation"
+       @echo "       for possible values"
+       @echo "       (default: cumtime)"
+       @echo ""
+       @echo "   PROFILE_OUTPUT=file"
+       @echo "       output profiling info to file - use absolute path"
+       @echo "       (default: stdout)"
+       @echo ""
+       @echo "   TEST_DEBUG=[1|y|yes]"
+       @echo "       enable debugging of the test framework itself (expert)"
+       @echo "       (default: no)"
+       @echo ""
+       @echo "   TEST_GCOV=[1|y|yes]"
+       @echo "       enable tests specifically designed soley for code coverage"
+       @echo "       (default: no)"
+       @echo ""
+       @echo "   API_FUZZ=[1|y|yes]"
+       @echo "       enable VPP api fuzz testing"
+       @echo "       (default: no)"
+       @echo ""
+       @echo "   RND_SEED=<seed>"
+       @echo "       random seed used by test framework"
+       @echo "       (default: time.time())"
+       @echo ""
+       @echo "Starting VPP in GDB for use with DEBUG=attach:"
        @echo ""
-       @echo "Verifying code-style"
-       @echo " test-checkstyle        - check PEP8 compliance"
+       @echo " test-start-vpp-in-gdb       - start VPP in gdb (release)"
+       @echo " test-start-vpp-debug-in-gdb - start VPP in gdb (debug)"
        @echo ""