From: olegrok@tarantool.org To: tarantool-patches@dev.tarantool.org, sergos@tarantool.org, alexander.turenko@tarantool.org Cc: Oleg Babin <babinoleg@mail.ru> Subject: [Tarantool-patches] [PATCH v3] error: add __concat method to error object Date: Fri, 10 Jan 2020 18:48:05 +0300 [thread overview] Message-ID: <20200110154805.94649-1-olegrok@tarantool.org> (raw) From: Oleg Babin <babinoleg@mail.ru> Usually functions return pair {nil, err} and expected that err is string. Let's make the behaviour of error object closer to string and define __concat metamethod. Closes #4489 The case of error "error_mt.__concat(): neither of args is an error" is not covered by tests because of #4723 --- Changes in v3: - Improve concatenation logic - Add more test cases Changes in v2: - Added tests Issue: https://github.com/tarantool/tarantool/issues/4489 Branch: olegrok/4489-concat-for-errors src/lua/error.lua | 11 +++++++ test/box/misc.result | 75 +++++++++++++++++++++++++++++++++++++++++- test/box/misc.test.lua | 24 +++++++++++++- 3 files changed, 108 insertions(+), 2 deletions(-) diff --git a/src/lua/error.lua b/src/lua/error.lua index d091ee48b..7f249864a 100644 --- a/src/lua/error.lua +++ b/src/lua/error.lua @@ -160,9 +160,20 @@ local function error_index(err, key) return error_methods[key] end +local function error_concat(lhs, rhs) + if ffi.istype('struct error', lhs) then + return tostring(lhs) .. rhs + elseif ffi.istype('struct error', rhs) then + return lhs .. tostring(rhs) + else + error('error_mt.__concat(): neither of args is an error') + end +end + local error_mt = { __index = error_index; __tostring = error_message; + __concat = error_concat; }; ffi.metatype('struct error', error_mt); diff --git a/test/box/misc.result b/test/box/misc.result index 3ca4e31ac..5ac5e0f26 100644 --- a/test/box/misc.result +++ b/test/box/misc.result @@ -199,6 +199,79 @@ box.error.new() - error: 'Usage: box.error.new(code, args)' ... -- +-- gh-4489: box.error has __concat metamethod +-- +test_run:cmd("push filter '(.builtin/.*.lua):[0-9]+' to '\\1'") +--- +- true +... +e = box.error.new(box.error.UNKNOWN) +--- +... +'left side: ' .. e +--- +- 'left side: Unknown error' +... +e .. ': right side' +--- +- 'Unknown error: right side' +... +e .. nil +--- +- error: 'builtin/error.lua: attempt to concatenate local ''rhs'' (a nil value)' +... +nil .. e +--- +- error: 'builtin/error.lua: attempt to concatenate local ''lhs'' (a nil value)' +... +e .. box.NULL +--- +- error: 'builtin/error.lua: attempt to concatenate ''string'' and ''void *''' +... +box.NULL .. e +--- +- error: 'builtin/error.lua: attempt to concatenate ''void *'' and ''string''' +... +123 .. e +--- +- 123Unknown error +... +e .. 123 +--- +- Unknown error123 +... +e .. e +--- +- Unknown errorUnknown error +... +e .. {} +--- +- error: 'builtin/error.lua: attempt to concatenate local ''rhs'' (a table value)' +... +{} .. e +--- +- error: 'builtin/error.lua: attempt to concatenate local ''lhs'' (a table value)' +... +-1ULL .. e +--- +- error: 'builtin/error.lua: attempt to concatenate ''uint64_t'' and ''string''' +... +e .. -1ULL +--- +- error: 'builtin/error.lua: attempt to concatenate ''string'' and ''uint64_t''' +... +1LL .. e +--- +- error: 'builtin/error.lua: attempt to concatenate ''int64_t'' and ''string''' +... +e .. 1LL +--- +- error: 'builtin/error.lua: attempt to concatenate ''string'' and ''int64_t''' +... +e = nil +--- +... +-- -- System errors expose errno as a field. -- _, err = require('fio').open('not_existing_file') @@ -1065,7 +1138,7 @@ error() --- - error: null ... --- A test case for bitwise operations +-- A test case for bitwise operations bit.lshift(1, 32) --- - 1 diff --git a/test/box/misc.test.lua b/test/box/misc.test.lua index c3e992a48..e1c2f990f 100644 --- a/test/box/misc.test.lua +++ b/test/box/misc.test.lua @@ -59,6 +59,28 @@ e = box.error.new(box.error.CREATE_SPACE, "space", "error") e box.error.new() +-- +-- gh-4489: box.error has __concat metamethod +-- +test_run:cmd("push filter '(.builtin/.*.lua):[0-9]+' to '\\1'") +e = box.error.new(box.error.UNKNOWN) +'left side: ' .. e +e .. ': right side' +e .. nil +nil .. e +e .. box.NULL +box.NULL .. e +123 .. e +e .. 123 +e .. e +e .. {} +{} .. e +-1ULL .. e +e .. -1ULL +1LL .. e +e .. 1LL +e = nil + -- -- System errors expose errno as a field. -- @@ -281,7 +303,7 @@ dostring('return abc') dostring('return ...', 1, 2, 3) -- A test case for Bug#1043804 lua error() -> server crash error() --- A test case for bitwise operations +-- A test case for bitwise operations bit.lshift(1, 32) bit.band(1, 3) bit.bor(1, 2) -- 2.23.0
next reply other threads:[~2020-01-10 15:48 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-01-10 15:48 olegrok [this message] 2020-01-14 14:21 ` Sergey Ostanevich 2020-01-14 15:02 ` Oleg Babin 2020-01-15 11:26 ` Alexander Turenko 2020-01-15 12:35 ` Oleg Babin 2020-01-16 22:02 ` Alexander Turenko
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=20200110154805.94649-1-olegrok@tarantool.org \ --to=olegrok@tarantool.org \ --cc=alexander.turenko@tarantool.org \ --cc=babinoleg@mail.ru \ --cc=sergos@tarantool.org \ --cc=tarantool-patches@dev.tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH v3] error: add __concat method to error object' \ /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