[Tarantool-patches] [PATCH] lua: introduce function for uuid comparison
Sergey Bronnikov
sergeyb at tarantool.org
Fri Nov 13 09:37:45 MSK 2020
Hi, Oleg
On 12.11.2020 12:55, Oleg Babin wrote:
> Hi, Sergey!
>
> The main reason is "uuid tests were implemented with diff-based approach".
Got it. Missed it that you haven't add new test and just enhanced the
old one.
>
> However I can easily convert my tests to TAP but I don't want touch
> existing tests of course.
>
> Do you think we should start moving towards TAP-based approach for
> "uuid" module testing?
I think no.
> On 12/11/2020 12:44, Sergey Bronnikov wrote:
>> Hi, Oleg
>>
>> thanks for the patch!
>> Looks like you expect a review from Leonid and Vlad, but I have a
>> question regarding tests in the patch.
>>
>> On 23:04 Wed 11 Nov , olegrok at tarantool.org wrote:
>>> --- a/test/app/uuid.result
>>> +++ b/test/app/uuid.result
>> For Tarantool core we have two kind of tests: with diff-based approach
>> and TAP based tests. Your tests can be easily implemented with a TAP
>> approach, when test data and oracle are in the code. But you preferred a
>> diff-based approach here. Why?
More information about the Tarantool-patches
mailing list