[tarantool-patches] Re: [PATCH v2] box: allow vinyl_memory set to 0 in config

  • From: Vladimir Davydov <vdavydov.dev@xxxxxxxxx>
  • To: Sergey Petrenko <sergepetrenko@xxxxxxxxxxxxx>
  • Date: Wed, 4 Jul 2018 17:23:46 +0300

On Wed, Jul 04, 2018 at 05:11:10PM +0300, Sergey Petrenko wrote:

diff --git a/test/box-tap/cfg.test.lua b/test/box-tap/cfg.test.lua
index b93a21f35..5608f6578 100755
--- a/test/box-tap/cfg.test.lua
+++ b/test/box-tap/cfg.test.lua
@@ -6,7 +6,7 @@ local socket = require('socket')
 local fio = require('fio')
 local uuid = require('uuid')
 local msgpack = require('msgpack')
-test:plan(91)
+test:plan(92)

 
--------------------------------------------------------------------------------
 -- Invalid values
@@ -164,6 +164,13 @@ function run_script(code)
     return res
 end

+-- gh-3468: should allow box.cfg with vinyl_memory=0
+code =[[
+box.cfg{vinyl_memory=0}
+os.exit(box.cfg.vinyl_memory == 0 and 0 or 1)
+]]
+test:is(run_script(code), 0, "actually set vinyl_memory to 0")

Looks good to me.

Other related posts: