diff options
author | Brad King <brad.king@kitware.com> | 2021-06-09 12:15:30 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2021-06-09 12:15:30 (GMT) |
commit | d4056f24a17ab7d35a0539583cadfe08699d14e7 (patch) | |
tree | 72d0f17616e5206d9411824c2be154debb496ee2 /Tests/RunCMake/CPack | |
parent | 5e26887c35be116a83ba049098f5780309ced073 (diff) | |
download | CMake-d4056f24a17ab7d35a0539583cadfe08699d14e7.zip CMake-d4056f24a17ab7d35a0539583cadfe08699d14e7.tar.gz CMake-d4056f24a17ab7d35a0539583cadfe08699d14e7.tar.bz2 |
Tests: Improve RunCMake.CPack*.SOURCE_PACKAGE test error formatting
Format message content for `message(FATAL_ERROR)`.
Apply the change from commit c54c284de7 (Tests: Improve RunCMake.CPack*
test error formatting, 2021-04-22) to the `SOURCE_PACKAGE` script too.
Also fix a typo from commit f5d79dec53 (Tests: Cleanup CPack message when
files are expected and none are found, 2021-01-15, v3.20.0-rc1~89^2~22).
Diffstat (limited to 'Tests/RunCMake/CPack')
-rw-r--r-- | Tests/RunCMake/CPack/VerifyResult.cmake | 2 | ||||
-rw-r--r-- | Tests/RunCMake/CPack/tests/SOURCE_PACKAGE/VerifyResult.cmake | 37 |
2 files changed, 28 insertions, 11 deletions
diff --git a/Tests/RunCMake/CPack/VerifyResult.cmake b/Tests/RunCMake/CPack/VerifyResult.cmake index 294b9e8..76c1bea 100644 --- a/Tests/RunCMake/CPack/VerifyResult.cmake +++ b/Tests/RunCMake/CPack/VerifyResult.cmake @@ -87,7 +87,7 @@ if(NOT EXPECTED_FILES_COUNT EQUAL 0) "which does not match:${msg_expected}\n" "${output_error_message}") endif() - elseif(foundFilescount_ EQUAL 0) + elseif(foundFilesCount_ EQUAL 0) message(FATAL_ERROR "Found no files for file ${file_no_}!\n" "Globbing expression:\n '${EXPECTED_FILE_${file_no_}}'\n" diff --git a/Tests/RunCMake/CPack/tests/SOURCE_PACKAGE/VerifyResult.cmake b/Tests/RunCMake/CPack/tests/SOURCE_PACKAGE/VerifyResult.cmake index 73d7481..42e8384 100644 --- a/Tests/RunCMake/CPack/tests/SOURCE_PACKAGE/VerifyResult.cmake +++ b/Tests/RunCMake/CPack/tests/SOURCE_PACKAGE/VerifyResult.cmake @@ -18,14 +18,20 @@ execute_process(COMMAND ${RPMBUILD_EXECUTABLE} --define "_topdir ${CMAKE_CURRENT OUTPUT_QUIET ) -set(output_error_message_ - "\n${RPMBUILD_EXECUTABLE} error: '${error_}';\nresult: '${result_}';\n${output_error_message}") +string(REPLACE "\n" "\n rpmbuild-err> " rpmbuild_err "\n${error_}") +string(REPLACE "\n" "\n rpmbuild-res> " rpmbuild_res "\n${result_}") +string(CONCAT output_error_message_ + "${RPMBUILD_EXECUTABLE} error:${rpmbuild_err}\n" + "${RPMBUILD_EXECUTABLE} result:${rpmbuild_res}\n" + "${output_error_message}" + ) # expected file content are test_prog and optional build-id links that are # generated by rpmbuild (introduced in rpm 4.13.0.1) set(EXPECTED_FILE_CONTENT_ "^/foo${whitespaces_}/foo/test_prog(${whitespaces_}.*\.build-id.*)*$") +set(EXPECTED_FILE_ "${CMAKE_CURRENT_BINARY_DIR}/test_rpm/RPMS/*.rpm") -file(GLOB_RECURSE FOUND_FILE_ RELATIVE "${CMAKE_CURRENT_BINARY_DIR}/test_rpm/RPMS" "${CMAKE_CURRENT_BINARY_DIR}/test_rpm/RPMS/*.rpm") +file(GLOB_RECURSE FOUND_FILE_ RELATIVE "${CMAKE_CURRENT_BINARY_DIR}/test_rpm/RPMS" "${EXPECTED_FILE_}") list(APPEND foundFiles_ "${FOUND_FILE_}") list(LENGTH FOUND_FILE_ foundFilesCount_) @@ -37,17 +43,27 @@ if(foundFilesCount_ EQUAL 1) expected_content_list "${PACKAGE_CONTENT}") if(NOT expected_content_list) + string(REPLACE "\n" "\n actual> " msg_actual "\n${PACKAGE_CONTENT}") + string(REPLACE "\n" "\n expect> " msg_expected "\n${EXPECTED_FILE_CONTENT_}") + string(REPLACE "\r" "\\r" msg_expected "${msg_expected}") + string(REPLACE "\t" "\\t" msg_expected "${msg_expected}") message(FATAL_ERROR "Unexpected file content!\n" - " Content: '${PACKAGE_CONTENT}'\n\n" - " Expected: '${EXPECTED_FILE_CONTENT_}'" + "The content was:${msg_actual}\n" + "which does not match:${msg_expected}\n" "${output_error_message_}") endif() +elseif(foundFilesCount_ EQUAL 0) + message(FATAL_ERROR + "Found no files!\n" + "Globbing expression:\n '${EXPECTED_FILE_}'\n" + "${output_error_message_}") else() message(FATAL_ERROR - "Found more than one file!" - " Found files count '${foundFilesCount_}'." - " Files: '${FOUND_FILE_}'" + "Found more than one file!\n" + "Found files count '${foundFilesCount_}'.\n" + "Files:\n '${FOUND_FILE_}'\n" + "Globbing expression:\n '${EXPECTED_FILE_}'\n" "${output_error_message_}") endif() @@ -62,6 +78,7 @@ list(LENGTH allFoundFiles_ allFoundFilesCount_) if(NOT foundFilesCount_ EQUAL allFoundFilesCount_) message(FATAL_ERROR - "Found more files than expected! Found files: '${allFoundFiles_}'" - "${output_error_message_}") + "Found more files than expected!\n" + "Found files:\n '${allFoundFiles_}'\n" + "${output_error_message_}") endif() |