Fix test for new Tarantool version and debug build (#42)

* fix assign to undeclared variable error

Tarantool in debug build has strict mode enabled by default.
As result before this patch we faced an error "assign to
undeclared variable".

* fix error messages for new Tarantool versions

  - netbox timeout error was changed (tarantool/tarantool#6538)
  - error on connection refused was extended

Author: olegrok

test/unit/errors_test.lua

@@ -9,7 +9,7 @@ local my_error = errors.new_class('My error')
 local current_file = debug.getinfo(1, 'S').short_src
 
 g.before_all = function()
-    _G.my_error = my_error
+    rawset(_G, 'my_error', my_error)
 end
 
 function g.test_error_new()

test/unit/netbox_test.lua

@@ -35,12 +35,16 @@ g.before_all = function()
         'universe', nil, {if_not_exists = true}
     )
 
-    _G.my_error = my_error
-    _G.remote_4args_fn = remote_4args_fn
+    rawset(_G, 'my_error', my_error)
+    rawset(_G, 'remote_4args_fn', remote_4args_fn)
 end
 
 g.before_each(function()
     g.conn = netbox.connect('127.0.0.1:3301')
+    local ok, err = pcall(g.conn.call, g.conn, 'table.insert', {{}, {}}, {timeout = 0})
+    t.assert_not(ok)
+    g.timeout_error = tostring(err)
+    t.assert(g.timeout_error == 'Timeout exceeded' or g.timeout_error == 'timed out')
 end)
 
 function g.test_errors_netbox_eval()
@@ -109,7 +113,7 @@ function g.test_errors_netbox_eval()
 
 
     local _l1, remote_fn = h.get_line(), function() return nil, my_error:new('Fuschia Platinum') end
-    _G.remote_fn = remote_fn
+    rawset(_G, 'remote_fn', remote_fn)
 
     local _l2, _, err = h.get_line(), errors.netbox_eval(g.conn, 'return remote_fn()')
     h.check_error(err, {
@@ -159,8 +163,8 @@ function g.test_errors_netbox_eval()
     local _l, _, err = h.get_line(), errors.netbox_eval(conn, 'return true')
     h.check_error(err, {
         file = 'builtin/box/net_box.lua',
-        err = '"127.0.0.1:9": Connection refused',
-        str = '^NetboxEvalError: "127.0.0.1:9": Connection refused\n' ..
+        err = '"127.0.0.1:9": ' .. conn.error,
+        str = '^NetboxEvalError: "127.0.0.1:9": ' .. conn.error .. '\n' ..
             'stack traceback:\n' ..
             '.+\n' ..
             ('\t%s:%d: .*$'):format(current_file, _l)
@@ -179,7 +183,7 @@ function g.test_errors_netbox_call()
     }, 'netbox_call(fn_undefined)')
 
     local _l1, remote_fn = h.get_line(), function() return nil, my_error:new('Yellow Iron') end
-    _G.remote_fn = remote_fn
+    rawset(_G, 'remote_fn', remote_fn)
 
     local _l2, _, err = h.get_line(), errors.netbox_call(g.conn, 'remote_fn')
     h.check_error(err, {
@@ -220,16 +224,16 @@ end
 function g.test_errors_netbox_wait_async()
     -- future returns error
     local long_call = function() fiber.sleep(10) return 5 end
-    _G.long_call = long_call
+    rawset(_G, 'long_call', long_call)
 
     local csw1 = h.fiber_csw()
 
     local future_call = errors.netbox_call(g.conn, '_G.long_call', nil, {is_async = true})
     local _l, _, err = h.get_line(), errors.netbox_wait_async(future_call, 0)
     h.check_error(err, {
         file = debug.getinfo(errors.netbox_wait_async).source:gsub('@', ''),
-        err = '"127.0.0.1:3301": Timeout exceeded',
-        str = '^NetboxCallError: "127.0.0.1:3301": Timeout exceeded\n' ..
+        err = '"127.0.0.1:3301": ' .. g.timeout_error,
+        str = '^NetboxCallError: "127.0.0.1:3301": ' .. g.timeout_error .. '\n' ..
             'stack traceback:\n' ..
             '.+\n' ..
             ('\t%s:%d: .*$'):format(current_file, _l)
@@ -239,8 +243,8 @@ function g.test_errors_netbox_wait_async()
     local _l, _, err = h.get_line(), errors.netbox_wait_async(future_eval, 0)
     h.check_error(err, {
         file = debug.getinfo(errors.netbox_wait_async).source:gsub('@', ''),
-        err = '"127.0.0.1:3301": Timeout exceeded',
-        str = '^NetboxEvalError: "127.0.0.1:3301": Timeout exceeded\n' ..
+        err = '"127.0.0.1:3301": ' .. g.timeout_error,
+        str = '^NetboxEvalError: "127.0.0.1:3301": '.. g.timeout_error .. '\n' ..
             'stack traceback:\n' ..
             '.+\n' ..
             ('\t%s:%d: .*$'):format(current_file, _l)
@@ -421,7 +425,7 @@ function g.test_errors_wrap_remote()
     }, 'wrap conn:eval("return nil, e:new()")')
 
     local _l1, remote_fn = h.get_line(), function() return nil, my_error:new('Fuschia Platinum') end
-    _G.remote_fn = remote_fn
+    rawset(_G, 'remote_fn', remote_fn)
 
     local _l2, _, err = h.get_line(), errors.wrap(g.conn:eval('return remote_fn()'))
     h.check_error(err, {
@@ -438,7 +442,7 @@ function g.test_errors_wrap_remote()
     }, 'wrap conn:eval("return remote_fn()")')
 
     local _l1, remote_fn = h.get_line(), function() return nil, my_error:new('Yellow Iron') end
-    _G.remote_fn = remote_fn
+    rawset(_G, 'remote_fn', remote_fn)
 
     local _l2, _, err = h.get_line(), errors.wrap(g.conn:call('remote_fn'))
     h.check_error(err, {