[Lldb-commits] [lldb] 9be5c13 - [lldb/Test] Add check-lldb-repro target

[Martin Storsjö via lldb-commits]

On Wed, 22 Jan 2020, Jonas Devlieghere via lldb-commits wrote:

>
> Author: Jonas Devlieghere
> Date: 2020-01-22T17:35:45-08:00
> New Revision: 9be5c13538898c7632c2de7300de9479688a2460
>
> URL: https://github.com/llvm/llvm-project/commit/9be5c13538898c7632c2de7300de9479688a2460
> DIFF: https://github.com/llvm/llvm-project/commit/9be5c13538898c7632c2de7300de9479688a2460.diff
>
> LOG: [lldb/Test] Add check-lldb-repro target
>
> This adds a new target check-lldb-repro which runs the shell tests with
> the lldb-repo utility. It runs the shell tests twice, once while
> capturing a reproducer and then again by replaying that reproducer.
>
> Added:
> 
>
> Modified:
>    lldb/test/Shell/CMakeLists.txt
>
> Removed:
> 
>
>
> ################################################################################
> diff  --git a/lldb/test/Shell/CMakeLists.txt b/lldb/test/Shell/CMakeLists.txt
> index d203f1e093c7..a843f11f3a06 100644
> --- a/lldb/test/Shell/CMakeLists.txt
> +++ b/lldb/test/Shell/CMakeLists.txt
> @@ -15,3 +15,17 @@ if (CMAKE_GENERATOR STREQUAL "Xcode")
>     ${CMAKE_CURRENT_BINARY_DIR}
>     DEPENDS lldb-test-deps)
> endif()
> +
> +# Add a lit test suite that runs the shell test while capturing a reproducer.
> +add_lit_testsuite(check-lldb-repro-capture
> +  "Running lldb shell test suite with reproducer capture"
> +  ${CMAKE_CURRENT_BINARY_DIR}
> +  PARAMS "lldb-run-with-repro=capture"
> +  DEPENDS lldb-test-deps)
> +
> +# Add a lit test suite that runs the shell test by replaying a reproducer.
> +add_lit_testsuite(check-lldb
> +  "Running lldb shell test suite with reproducer replay"
> +  ${CMAKE_CURRENT_BINARY_DIR}
> +  PARAMS "lldb-run-with-repro=capture"
> +  DEPENDS lldb-test-deps check-lldb-repro-capture)

FYI (the commit mails from my commits don't seem to make it to the list), 
I made a small tweak to this in 
https://github.com/llvm/llvm-project/commit/1db1b8b8b35727a01387c1bc0bbf25701ad05d3f 
as the above construct with add_lit_testsuite(... DEPENDS check-*) will 
cause that check-* testsuite to be run when just building test dependncies 
with "ninja test-depends".

// Martin