[Tarantool-patches] [PATCH 2/2] luacheck: fix warnings in test/box-tap

Serge Petrenko sergepetrenko at tarantool.org
Fri Jan 15 15:40:58 MSK 2021



15.01.2021 14:22, sergeyb at tarantool.org пишет:
> From: Sergey Bronnikov <sergeyb at tarantool.org>
>
> Follows up #5457
> ---
>   test/box-tap/feedback_daemon.test.lua | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/test/box-tap/feedback_daemon.test.lua b/test/box-tap/feedback_daemon.test.lua
> index 123033cda..8cfbf31d7 100755
> --- a/test/box-tap/feedback_daemon.test.lua
> +++ b/test/box-tap/feedback_daemon.test.lua
> @@ -124,8 +124,8 @@ local fh = fio.open("feedback.json")
>   test:ok(fh, "file is created")
>   local file_data = fh:read()
>   -- Ignore the report time. The data should be equal other than that.
> -feedback_save = string.gsub(feedback_save, 'time:(%d+)', 'time:0')
> -file_data = string.gsub(feedback_save, 'time:(%d+)', 'time:0')
> +feedback_save = string.gsub(feedback_save, '"time":(%d+)', 'time:0')
> +file_data = string.gsub(file_data, '"time":(%d+)', 'time:0')
Thanks for finding this typo!
Please also replace `time:0` with `"time":0`

Other than that LGTM.

>   test:is(file_data, feedback_save, "data is equal")
>   fh:close()
>   fio.unlink("feedback.json")
> @@ -247,7 +247,7 @@ box.space.features_memtx_empty:drop()
>   box.space.features_memtx:drop()
>   box.space.features_sync:drop()
>   
> -function check_stats(stat)
> +local function check_stats(stat)
>       local sub = test:test('feedback operation stats')
>       sub:plan(18)
>       local box_stat = box.stat()

-- 
Serge Petrenko



More information about the Tarantool-patches mailing list