autobuild: Make autobuild test suite more granular
Split each test into its own function. Oncoming patches will expand on this approach. Signed-off-by: Michal Berger <michal.berger@intel.com> Change-Id: I61f3a8847ae65a25504acdf22301757d35c0408d Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15722 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Kamil Godzwon <kamilx.godzwon@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Karol Latecki <karol.latecki@intel.com>
This commit is contained in:
parent
92109c88d7
commit
533039a235
@ -283,7 +283,7 @@ test_make_uninstall() {
|
|||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
|
|
||||||
build_doc() {
|
_build_doc() {
|
||||||
local doxygenv
|
local doxygenv
|
||||||
doxygenv=$(doxygen --version)
|
doxygenv=$(doxygen --version)
|
||||||
|
|
||||||
@ -326,19 +326,44 @@ build_doc() {
|
|||||||
rm -rf "$rootdir"/doc/output
|
rm -rf "$rootdir"/doc/output
|
||||||
}
|
}
|
||||||
|
|
||||||
autobuild_test_suite() {
|
check_format() {
|
||||||
run_test "autobuild_check_format" "$rootdir/scripts/check_format.sh"
|
run_test "autobuild_check_format" "$rootdir/scripts/check_format.sh"
|
||||||
run_test "autobuild_check_so_deps" $rootdir/test/make/check_so_deps.sh $1
|
}
|
||||||
|
|
||||||
|
check_so_deps() {
|
||||||
|
run_test "autobuild_check_so_deps" "$rootdir/test/make/check_so_deps.sh" "$1"
|
||||||
|
}
|
||||||
|
|
||||||
|
external_code() {
|
||||||
|
run_test "autobuild_external_code" "$rootdir/test/external_code/test_make.sh" "$rootdir"
|
||||||
|
}
|
||||||
|
|
||||||
|
dpdk_pci_api() {
|
||||||
run_test "autobuild_check_dpdk_pci_api" check_dpdk_pci_api
|
run_test "autobuild_check_dpdk_pci_api" check_dpdk_pci_api
|
||||||
if [[ $SPDK_TEST_AUTOBUILD == 'full' ]]; then
|
}
|
||||||
run_test "autobuild_external_code" $rootdir/test/external_code/test_make.sh $rootdir
|
|
||||||
|
build_files() {
|
||||||
"$rootdir/configure" $config_params --without-shared
|
"$rootdir/configure" $config_params --without-shared
|
||||||
$MAKE $MAKEFLAGS
|
$MAKE $MAKEFLAGS
|
||||||
run_test "autobuild_generated_files_check" porcelain_check
|
run_test "autobuild_generated_files_check" porcelain_check
|
||||||
run_test "autobuild_header_dependency_check" header_dependency_check
|
run_test "autobuild_header_dependency_check" header_dependency_check
|
||||||
run_test "autobuild_make_install" test_make_install
|
run_test "autobuild_make_install" test_make_install
|
||||||
run_test "autobuild_make_uninstall" test_make_uninstall
|
run_test "autobuild_make_uninstall" test_make_uninstall
|
||||||
run_test "autobuild_build_doc" build_doc
|
}
|
||||||
|
|
||||||
|
build_doc() {
|
||||||
|
"$rootdir/configure" $config_params --without-shared
|
||||||
|
run_test "autobuild_build_doc" _build_doc
|
||||||
|
}
|
||||||
|
|
||||||
|
autobuild_test_suite() {
|
||||||
|
check_format
|
||||||
|
check_so_deps "$1"
|
||||||
|
dpdk_pci_api
|
||||||
|
if [[ $SPDK_TEST_AUTOBUILD == 'full' ]]; then
|
||||||
|
external_code
|
||||||
|
build_files
|
||||||
|
build_doc
|
||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user