<!DOCTYPE html>
<html data-lt-installed="true">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body style="padding-bottom: 1px;">
<p>Hi, Sergey,<br>
</p>
<div class="moz-cite-prefix">On 24.09.2024 14:45, Sergey Kaplun
wrote:<br>
</div>
<blockquote type="cite" cite="mid:ZvKmTM83r7scDSqg@root">
<pre class="moz-quote-pre" wrap="">Hi, Sergey!
Thanks for the review!
On 24.09.24, Sergey Bronnikov wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">Hi, Sergey,
test statuses from TAP report produced by tests is not propagated to CTest.
I propose to disable these tests in CTest:
--- a/test/tarantool-tests/CMakeLists.txt
+++ b/test/tarantool-tests/CMakeLists.txt
@@ -166,3 +167,14 @@ if(LUAJIT_USE_ASAN AND CMAKE_C_COMPILER_ID STREQUAL
"GNU")
LD_PRELOAD ${LIB_ASAN}
)
endif()
+
+set_tests_properties(
+ test/tarantool-tests/profilers/gh-5688-tool-cli-flag.test.lua
+ test/tarantool-tests/profilers/gh-5813-resolving-of-c-symbols.test.lua
+ test/tarantool-tests/profilers/gh-5994-memprof-human-readable.test.lua
+
test/tarantool-tests/profilers/gh-7264-add-proto-trace-sysprof-default.test.lua
+
test/tarantool-tests/profilers/gh-9217-profile-parsers-error-handling.test.lua
+ test/tarantool-tests/profilers/misclib-memprof-lapi.test.lua
+ test/tarantool-tests/profilers/misclib-sysprof-lapi.test.lua
+ PROPERTIES DISABLED TRUE
+)
CTest will always report that these tests are disabled and we will never
forget to enable them back:
100% tests passed, 0 tests failed out of 158
Label Time Summary:
tarantool-tests = 14.66 sec*proc (165 tests)
Total Test time (real) = 14.73 sec
The following tests did not run:
222 -
test/tarantool-tests/profilers/gh-5688-tool-cli-flag.test.lua (Disabled)
223 -
test/tarantool-tests/profilers/gh-5813-resolving-of-c-symbols.test.lua
(Disabled)
224 -
test/tarantool-tests/profilers/gh-5994-memprof-human-readable.test.lua
(Disabled)
225 -
test/tarantool-tests/profilers/gh-7264-add-proto-trace-sysprof-default.test.lua
(Disabled)
226 -
test/tarantool-tests/profilers/gh-9217-profile-parsers-error-handling.test.lua
(Disabled)
227 -
test/tarantool-tests/profilers/misclib-memprof-lapi.test.lua (Disabled)
228 -
test/tarantool-tests/profilers/misclib-sysprof-lapi.test.lua (Disabled)
What do you think?
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
Side note: this is new in 3.9, so we need to update CMake first.
I'm not sure about this one.
We also have all skipconditions inside the test itself, which makes it
"self-sufficient" -- so if we look at the test, we know all builds when
it is disabled. Adding another location may be confusing.</pre>
</blockquote>
<p>I agree. The ideal option for us is when TAP statuses are
propagated to CTest,</p>
<p>but it is not an option now.<br>
</p>
<blockquote type="cite" cite="mid:ZvKmTM83r7scDSqg@root">
<pre class="moz-quote-pre" wrap="">
OTOH, we can see skipped tests in -V mode (but I agree, that it isn't
very convenient).
I am open for discussion :).
</pre>
</blockquote>
<p>Feel free to ignore, I'll not insist.</p>
<p><br>
</p>
<p>LGTM<br>
</p>
<blockquote type="cite" cite="mid:ZvKmTM83r7scDSqg@root">
<pre class="moz-quote-pre" wrap="">
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">
On 24.09.2024 13:29, Sergey Kaplun wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">Profilers tests with LUAJIT_ENABLE_TABLE_BUMP are flaky due to the bug
in the TDUP patching. Disable them for now.
---
test/tarantool-tests/CMakeLists.txt | 6 ++++++
.../profilers/gh-5688-tool-cli-flag.test.lua | 2 ++
.../gh-5813-resolving-of-c-symbols.test.lua | 2 ++
.../gh-5994-memprof-human-readable.test.lua | 2 ++
...7264-add-proto-trace-sysprof-default.test.lua | 2 ++
...-9217-profile-parsers-error-handling.test.lua | 2 ++
.../profilers/misclib-memprof-lapi.test.lua | 16 +++++++++-------
.../profilers/misclib-sysprof-lapi.test.lua | 2 ++
8 files changed, 27 insertions(+), 7 deletions(-)
diff --git a/test/tarantool-tests/CMakeLists.txt b/test/tarantool-tests/CMakeLists.txt
</pre>
</blockquote>
</blockquote>
<pre class="moz-quote-pre" wrap="">
<snipped>
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre class="moz-quote-pre" wrap=""> test:plan(19)
</pre>
</blockquote>
</blockquote>
<pre class="moz-quote-pre" wrap="">
</pre>
</blockquote>
</body>
<lt-container></lt-container>
</html>