tarantool / tarantool / 16021742437

88%

push

github

test: drop box/alter-primary-index-tuple-leak-long.test.lua

The test takes too long, especially on ASAN-enabled builds. Here's what
test-run reports for this test on debug-asan-clang workflow (duration is
in seconds):

* 367.76 box/alter-primary-index-tuple-leak-long.test.lua

The test takes that long because it tries to run out of memory while
checking that there's no memory leaks. Note that it has no assertions.

The test was added a long time ago, when we had a lot of suppressions
that prevented ASAN from reporting memory leaks. By today, most of the
suppressions have been removed so we shouldn't need tests like this
anymore.

I tried to reintroduce the leak checked by this test by removing
the code unreferencing stmt->old_tuple from memtx_build_on_replace.
Turned out that the test passes unless ASAN is enabled. At the same
time, at least two other tests fail as well because ASAN reports leaks:

* box/gh-5958-rolled-back-replace-background-built-index.test.lua
* engine-luatest/gh_4973_concurrent_alter_fails_test.lua

In other words, the test is totally useless now so let's drop it.

NO_DOC=test fix
NO_CHANGELOG=test fix

70639 of 124437 branches covered (56.77%)

103851 of 118642 relevant lines covered (87.53%)

2599744.9 hits per line