<HTML><BODY><br><br><br><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">
        Вторник, 17 июля 2018, 16:23 +03:00 от Vladimir Davydov <vdavydov.dev@gmail.com>:<br><br><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_15318338060000000094_BODY">On Tue, Jul 17, 2018 at 04:13:49PM +0300, Vladimir Davydov wrote:<br>
> >  test/box-tap/cfg.test.lua | 8 +++++++-<br>
> >  1 file changed, 7 insertions(+), 1 deletion(-)<br>
> <br>
> Please squash the test into the patch with the fix (`git rebase -i`)</div></div></div></div></blockquote>Ok<br><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_15318338060000000094_BODY"><br><br>
Hmm, the test doesn't pass on certain distros (see travis). I guess,<br>
this is because they don't have syslog configured. What about wrapping<br>
box.cfg in pcall and checking the status? Since we just want to check<br>
that there's no crash, this should do. </div></div></div></div></blockquote>I'll do that way.<br><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_15318338060000000094_BODY">Or alternatively you can test<br>
this by setting a custom syslog destination (you have to test it anyway<br>
in your other patches AFAIR). </div></div></div></div></blockquote>Do you mean in syslog destination series? I had a thought about it, but did not do in v2. Will keep in mind<br><br><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_15318338060000000094_BODY">In this case, this particular fix doesn't<br>
need a test case at all - it's suffices to mention that the fix is going<br>
to be tested in the following patches.<br><br>
> <br>
> > <br>
> > diff --git a/test/box-tap/cfg.test.lua b/test/box-tap/cfg.test.lua<br>
> > index ffafdbe42..605623013 100755<br>
> > --- a/test/box-tap/cfg.test.lua<br>
> > +++ b/test/box-tap/cfg.test.lua<br>
> > @@ -6,7 +6,7 @@ local socket = require('socket')<br>
> >  local fio = require('fio')<br>
> >  local uuid = require('uuid')<br>
> >  local msgpack = require('msgpack')<br>
> > -test:plan(95)<br>
> > +test:plan(96)<br>
> >  <br>
> >  --------------------------------------------------------------------------------<br>
> >  -- Invalid values<br>
> > @@ -464,5 +464,11 @@ code = string.format(code_fmt, dir, instance_uuid, uuid.new())<br>
> >  test:is(run_script(code), PANIC, "replicaset_uuid mismatch")<br>
> >  fio.rmdir(dir)<br>
> >  <br>
> > +--<br>
> > +-- Check syslog configuration<br>
> > +--<br>
> > +code = [[box.cfg{log = 'syslog:identity=tarantool'}]]<br>
> > +test:is(run_script(code), 0, "valid log configuration is broken")<br>
> > +<br>
> >  test:check()<br>
> >  os.exit(0)<br></div></div></div></div></blockquote>
<br>
<br>-- <br>Olga Arkhangelskaia<br></BODY></HTML>