[Tarantool-patches] [PATCHv4 1/3] test: change timeout in -e assert(false) test

Artem artemreyt at tarantool.org
Tue Jan 26 13:26:39 MSK 2021


Hi, thanks for review!

21.01.2021 19:20, Sergey Bronnikov пишет:
> Thanks for the patch!
>
> I think it would be more clear if you will add reasons
>
> why we need to reduce timeout for two times.
Done.
>
> On 18.01.2021 15:45, Artem Starshov wrote:
>> Set timeout to 60 sec for waiting operations with process,
>> like in default timeout for `cond_wait` in test-run.
>>
>> Follows up #4983
>> ---
>>   test/app-tap/gh-4983-tnt-e-assert-false-hangs.test.lua | 8 +++++---
>>   1 file changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/test/app-tap/gh-4983-tnt-e-assert-false-hangs.test.lua 
>> b/test/app-tap/gh-4983-tnt-e-assert-false-hangs.test.lua
>> index 656fe212b..c8313770e 100755
>> --- a/test/app-tap/gh-4983-tnt-e-assert-false-hangs.test.lua
>> +++ b/test/app-tap/gh-4983-tnt-e-assert-false-hangs.test.lua
>> @@ -60,10 +60,12 @@ local TARANTOOL_PATH = arg[-1]
>>   local output_file = fio.abspath('out.txt')
>>   local line = ('%s -e "assert(false)" > %s 2>&1 & echo $!'):
>>           format(TARANTOOL_PATH, output_file)
>> -local process_waiting_timeout = 30.0
>> -local file_read_timeout = 30.0
>> +
>> +-- Like a default timeout for `cond_wait` in test-run
>> +local process_waiting_timeout = 60.0
>> +local file_read_timeout = 60.0
>> +local file_open_timeout = 60.0
>>   local file_read_interval = 0.2
>> -local file_open_timeout = 30.0
>>     local res = tap.test('gh-4983-tnt-e-assert-false-hangs', 
>> function(test)
>>       test:plan(2)


More information about the Tarantool-patches mailing list