[Tarantool-patches] [PATCH luajit 0/2] Enable CLI-related tests in lua-Harness
Igor Munkin
imun at tarantool.org
Sun Feb 25 13:41:05 MSK 2024
There are two reasons to apply this series to the trunk:
1. The positive one: Tarantool supports -b and -j options to use LuaJIT
modules since the commit bf8b76a4dfc9dd62d4131e90e2ae5d83843b6630 ("lua:
proxy -j and -b flags"), so the related tests from lua-Harness suite can
be partially (since -O is still not implemented in Tarantool) enabled.
2. The negative one: Tarantool diff-based tests for CLI interfaces are
hard to maintain if any change occurrs in LuaJIT modules, since the
aforementioned tests implement dumb comparison of the output, produced
by the current CLI version against the expected one, provided by the
.result file. Hence, to rule the tests related to LuaJIT CLI interface
in a more convenient way, the corresponding tests should be moved from
the tests in Tarantool repository to the tests in LuaJIT repository. The
branch enables the nice checks implemented in the lua-Harness suite; the
PR #9726 removes the unmaintainable diff-based tests in Tarantool repo.
The changeset is split into the two patches. One can find more reasoning
within the particular commit message.
Branch: https://github.com/tarantool/luajit/tree/imun/enable-tarantool-cli-tests-in-lua-Harness
Tarantool PR(*): https://github.com/tarantool/tarantool/pull/9726
(*) There is a Tarantool-related patch on top of the corresponding
LuaJIT bump.
Igor Munkin (2):
test: introduce routine to build error message
test: enable CLI-related lua-Harness tests back
test/lua-Harness-tests/241-standalone.t | 2 +-
test/lua-Harness-tests/411-luajit.t | 23 +++++++++++++++++------
2 files changed, 18 insertions(+), 7 deletions(-)
--
2.39.2
More information about the Tarantool-patches
mailing list