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>
next prev parent 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