[Tarantool-patches] [PATCH v3 1/2] test: cleanup tests code

Alexander Tikhonov avtikhon at tarantool.org
Fri Mar 20 17:10:17 MSK 2020


Igor, I’ve corrected these messages, thank you.

  
>Пятница, 20 марта 2020, 16:52 +03:00 от Igor Munkin <imun at tarantool.org>:
> 
>Sasha,
>
>Thanks for the patch! LGTM except the one general nit: please adjust
>test:ok messages considering those I proposed in the previous review.
>
>New series is not needed, just update the remote branch.
>
>On 20.03.20, Alexander V. Tikhonov wrote:
>> Cleaned up the tests code according to the Lua style guide:
>> - made scoped variables local
>> - added os.exit call at the end of the test chunk
>> - adjusted the messages in test:ok calls
>>
>> Part of #4655
>> ---
>> test/fix_string_find_recording.test.lua | 6 +++---
>> test/fold_bug_LuaJIT_505.test.lua | 7 ++++---
>> test/fold_bug_LuaJIT_524.test.lua | 4 ++--
>> test/gh.test.lua | 8 ++++----
>> test/table_chain_bug_LuaJIT_494.test.lua | 23 ++++++++++++-----------
>> test/unsink_64_kptr.test.lua | 7 ++++---
>> 6 files changed, 29 insertions(+), 26 deletions(-)
>>
>
><snipped>
>
>> --
>> 2.17.1
>>
>
>--
>Best regards,
>IM 
 
 
--
Alexander Tikhonov
 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20200320/dc1fdd9f/attachment.html>


More information about the Tarantool-patches mailing list