tarantool / tarantool / 25701

83%

master: 88%

push

travis-ci

<a href="https://github.com/tarantool/tarantool/commit/<a class=hub.com/tarantool/tarantool/commit/0ef4958792a9be2f3d92f9b6cf56a5ff1a3584d7">0ef495879<a href="https://github.com/tarantool/tarantool/commit/0ef4958792a9be2f3d92f9b6cf56a5ff1a3584d7">&quot;&gt;test: update test-run

The following commits are land into the submodule with this update:

- 96dea99 Enable luacheck on Travis CI
- daff045 Add luacheck config and fix warnings found by luacheck

Most of the changes are about test-run&amp;#39;s Makefile and CI, but there is
one change in the testing framework itself:

 | diff --git a/test_run.lua b/test_run.lua
 | index 2a8d713..06fbf6c 100644
 | --- a/test_run.lua
 | +++ b/test_run.lua
 | @@ -163,7 +163,6 @@ local function wait_fullmesh(self, servers)
 |                  log.info(&amp;quot;%s: bootstrapped&amp;quot;, server)
 |                  break
 |              end
 | -            local info = self:eval(server, &amp;quot;box.info&amp;quot;)
 |              fiber.sleep(0.01)
 |          end
 |          -- wait all for full mesh

This change removes the call, whose result is not used. On the first
glance, it should not change any behaviour. However `test_run:eval()`
performs a network request and it is good to get rid of it.

The change is tiny and should not affect anything in tarantool testing,
but I decided to commit it separately from others to don&amp;#39;t hide anything
and ease bisect if we&amp;#39;ll observe any positive or negative impact. IOW,
just in case.

(cherry picked from commit </a><a class="double-link" href="https://github.com/tarantool/tarantool/commit/<a class="double-link" href="https://github.com/tarantool/tarantool/commit/546499c9c001e30cbd2598946c9c0589b8e30f53">546499c9c</a>">546499c9c</a><a href="https://github.com/tarantool/tarantool/commit/0ef4958792a9be2f3d92f9b6cf56a5ff1a3584d7">)

42771 of 51635 relevant lines covered (82.83%)

981905.37 hits per line