Tarantool development patches archive
 help / color / mirror / Atom feed
From: Sergey Bronnikov <sergeyb@tarantool.org>
To: "Alexander V. Tikhonov" <avtikhon@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org
Subject: Re: [Tarantool-patches] [PATCH v1] Divide test box/net.box
Date: Mon, 18 May 2020 13:00:58 +0300	[thread overview]
Message-ID: <20200518100058.GC20053@pony.bronevichok.ru> (raw)
In-Reply-To: <20200515170409.GA10228@hpalx>

Hi, Alexander

On 20:04 Fri 15 May , Alexander V. Tikhonov wrote:
> Hi Sergey, thanks a lot for the review. As we discussed the ability to
> reproduce the issues on the fixing commits I've checked group of tests:
> 10 tests as you asked in the issue and 3 that I've already checked.
> Whithin these tests results were:
> 3 tests: all reproduced the issue
> 10 tests:
>   4 - reproduced the issue
>   1 - didn't show the issue

could you reproduce issue using original test with source code without fix?

>   5 - couldn't build and test too old fixes committed in 1.6 Tarantool
>   version.

could you reproduce them using tarantool from packages?

> To see the results with names please check the issue in github, where
> I've created the table of results for the checked tests.
> 
> On Thu, Apr 02, 2020 at 06:44:44PM +0300, Sergey Bronnikov wrote:
> > Hello!
> > 
> > before splitting test we had a net.box.skipcond file due to an issue
> > https://github.com/tarantool/tarantool/issues/4271. I believe that file
> > is useless now and we should add similar files for a new tests.
> > 
> > I have looked on tests without postfix "gh-xxx" in a filenames.
> > LGTM for them:
> > 
> > net.box_connect_triggers.test.lua
> > net.box_permissions.test.lua
> > net.box_reconnect_after.test.lua
> > net.box_get_connection_object.test.lua
> > 
> > I propose to run tests with postfix "gh-xxx" in filenames on commits
> > before fix where bug covered by test was added. Broken test would mean
> > it still works.
> > 
> > Sergey
> > 

<snipped>

  reply	other threads:[~2020-05-18 10:01 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-02  6:38 Alexander V. Tikhonov
2020-04-02 15:44 ` Sergey Bronnikov
2020-05-15 17:04   ` Alexander V. Tikhonov
2020-05-18 10:00     ` Sergey Bronnikov [this message]
2020-06-01  9:24       ` Alexander V. Tikhonov
2020-06-04 10:33         ` Sergey Bronnikov
2020-06-04 15:50           ` Alexander V. Tikhonov
2020-06-08 20:10             ` Sergey Bronnikov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200518100058.GC20053@pony.bronevichok.ru \
    --to=sergeyb@tarantool.org \
    --cc=avtikhon@tarantool.org \
    --cc=tarantool-patches@dev.tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH v1] Divide test box/net.box' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox