[Tarantool-patches] [PATCH v7 0/3] Fix luacheck warnings in test/app-tap and test/app
sergeyb at tarantool.org
sergeyb at tarantool.org
Wed Dec 16 18:59:58 MSK 2020
From: Sergey Bronnikov <sergeyb at tarantool.org>
Changelog v7:
- reverted changes with shadowing
- reverted changes with redefinitions
- rebased to master branch
- splitted a huge patch series for test/ to a separate issues and patches series
(other patches will follow)
Changelog v6:
- splitted patch in test/ for patches per sub-directory
- adjusted supressions in .luacheckrc
- fixed formatting issues in .luacheckrc
- fixed warnings in test/vinyl/vinyl.lua and test/vinyl/large.lua
- added commit with bumping luajit version
- fixed warnings in test/app-tap/lua/serializer_test.lua
NOTE: patch v6 for app-tap sent in [1] got LGTM from Igor Munkin in [2],
but later Alexander Turenko initiated discussion [3] and we decided verbally to
revert some changes in all patches.
1. https://lists.tarantool.org/pipermail/tarantool-patches/2020-May/017237.html
2. https://lists.tarantool.org/pipermail/tarantool-patches/2020-June/017301.html
3. https://lists.tarantool.org/pipermail/tarantool-patches/2020-June/017312.html
GH issues:
- https://github.com/tarantool/tarantool/issues/5453
- https://github.com/tarantool/tarantool/issues/5454
GitLab: https://gitlab.com/tarantool/tarantool/-/pipelines/230796285
Branch: ligurio/gh-5453-luacheck-warnings-test-app-tap
Sergey Bronnikov (3):
luacheck: fix warnings in test/app-tap
luacheck: fix warnings in test/app
luacheck: remove unneeded comment
.luacheckrc | 15 ++++-
test/app-tap/cfg.test.lua | 2 +-
test/app-tap/clock.test.lua | 4 +-
test/app-tap/console.test.lua | 12 ++--
test/app-tap/csv.test.lua | 56 +++++++++----------
test/app-tap/fail_main.test.lua | 4 +-
.../gh-4761-json-per-call-options.test.lua | 4 +-
test/app-tap/gh-5013-fiber-cancel.test.lua | 2 +-
.../gh-5130-panic-on-invalid-log.test.lua | 4 +-
test/app-tap/http_client.test.lua | 25 +++------
test/app-tap/iconv.test.lua | 4 +-
test/app-tap/init_script.test.lua | 16 +++---
test/app-tap/inspector.test.lua | 5 +-
test/app-tap/json.test.lua | 1 -
test/app-tap/logger.test.lua | 12 ++--
test/app-tap/lua/serializer_test.lua | 28 +++-------
test/app-tap/module_api.test.lua | 13 ++---
test/app-tap/msgpackffi.test.lua | 3 +-
test/app-tap/pcall.test.lua | 2 +-
test/app-tap/snapshot.test.lua | 15 +++--
test/app-tap/string.test.lua | 30 +++++-----
test/app-tap/tap.test.lua | 4 +-
test/app-tap/tarantoolctl.test.lua | 46 ++++++---------
test/app-tap/trigger.test.lua | 8 +--
test/app-tap/yaml.test.lua | 14 ++---
test/app-tap/yield-in-gc-finalizer.test.lua | 3 +-
26 files changed, 150 insertions(+), 182 deletions(-)
--
2.25.1
More information about the Tarantool-patches
mailing list