From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id 37F1B25764 for ; Fri, 12 Jul 2019 17:44:53 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0gFFbryOpkA4 for ; Fri, 12 Jul 2019 17:44:53 -0400 (EDT) Received: from mail-lf1-f68.google.com (mail-lf1-f68.google.com [209.85.167.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id CFC0E25749 for ; Fri, 12 Jul 2019 17:44:52 -0400 (EDT) Received: by mail-lf1-f68.google.com with SMTP id v85so7371020lfa.6 for ; Fri, 12 Jul 2019 14:44:52 -0700 (PDT) Date: Sat, 13 Jul 2019 00:44:49 +0300 From: Cyrill Gorcunov Subject: [tarantool-patches] Re: [PATCH 2/3] box/lua/console: Add support for lua output format Message-ID: <20190712214449.GG4602@uranus.lan> References: <20190712190438.32349-1-gorcunov@gmail.com> <20190712190438.32349-3-gorcunov@gmail.com> <20190712204527.GB31930@atlas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190712204527.GB31930@atlas> Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-Help: List-Unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-Subscribe: List-Owner: List-post: List-Archive: To: Konstantin Osipov Cc: tml , Alexander Turenko , Kirill Yukhin On Fri, Jul 12, 2019 at 11:45:27PM +0300, Konstantin Osipov wrote: > * Cyrill Gorcunov [19/07/12 22:08]: > > Historically we use YAML format to print results of operation to > > a console. Moreover our test engine is aiming YAML as a primary format > > to compare results of test runs. Still we need an ability to print > > results in a different fasion, in particular one may need to use > > the console in a REPL way so that the results would be copied and > > pased back to further processing. > > Can you please come up with a patch for test-run that changes the > test engine to set the output explicitly in suite.ini, so that > it's not broken by changing the default? I think about it and this gonna be new patches on top of the series. Look, the main purpose of the patchset is to be completely transparent to current codebase and the patches must not break anything already running (including tests). That's why modifying test engine is my next step, also I need to provide tests for this new console output. But again, my initial main purpose -- to not break anything now.