[Tarantool-patches] [PATCH 1/1] test: fix test for #4672
Alexander Turenko
alexander.turenko at tarantool.org
Sun Dec 29 20:54:38 MSK 2019
I rewrote the commit message and pushed it to master, 2.2 and 1.10.
The new commit message:
| test: trim field name update from serializer test
|
| The test re encoding of -2^63 Lua number value did use update by a field
| name, which does not supported in 1.10 and 2.2 branches. Field name
| updates are orthogonal to Lua number serialization and we don't intend
| to test them here. So it is safe and logical to get rid of them in the
| test.
|
| This change allow the test to pass on 1.10 and 2.2 branches.
|
| Follows up #4672.
|
| Reviewed-by: Alexander Tikhonov <avtikhon at tarantool.org>
| Reviewed-by: Alexander Turenko <alexander.turenko at tarantool.org>
WBR, Alexander Turenko.
> test: fix test for #4672
Please, don't include an issue number into a commit header. A header
should describe what is changed in brief rather then provide tracking
information.
On Thu, Dec 19, 2019 at 05:56:57PM +0300, imeevma at tarantool.org wrote:
> ---
> https://github.com/tarantool/tarantool/tree/imeevma/gh-4672-fix-test-for-1.10
>
> test/box/gh-4672-min-integer-value-in-serializer.result | 8 ++++----
> test/box/gh-4672-min-integer-value-in-serializer.test.lua | 8 ++++----
> 2 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/test/box/gh-4672-min-integer-value-in-serializer.result b/test/box/gh-4672-min-integer-value-in-serializer.result
> index f4d7adb..a9540d9 100644
> --- a/test/box/gh-4672-min-integer-value-in-serializer.result
> +++ b/test/box/gh-4672-min-integer-value-in-serializer.result
> @@ -36,7 +36,7 @@ s:insert({3, 0})
> | ---
> | - [3, 0]
> | ...
> -s:update(3, {{'=', 'i', -2^63}})
> +s:update(3, {{'=', 2, -2^63}})
> | ---
> | - [3, -9223372036854775808]
> | ...
> @@ -44,7 +44,7 @@ s:insert({4, 0})
> | ---
> | - [4, 0]
> | ...
> -s:update(4, {{'=', 'i', -9223372036854775808LL}})
> +s:update(4, {{'=', 2, -9223372036854775808LL}})
> | ---
> | - [4, -9223372036854775808]
> | ...
> @@ -72,7 +72,7 @@ s:insert({13, 0})
> | ---
> | - [13, 0]
> | ...
> -s:update(13, {{'=', 'i', -2^63}})
> +s:update(13, {{'=', 2, -2^63}})
> | ---
> | - [13, -9223372036854775808]
> | ...
> @@ -80,7 +80,7 @@ s:insert({14, 0})
> | ---
> | - [14, 0]
> | ...
> -s:update(14, {{'=', 'i', -9223372036854775808LL}})
> +s:update(14, {{'=', 2, -9223372036854775808LL}})
> | ---
> | - [14, -9223372036854775808]
> | ...
> diff --git a/test/box/gh-4672-min-integer-value-in-serializer.test.lua b/test/box/gh-4672-min-integer-value-in-serializer.test.lua
> index f718a56..fcabb3a 100644
> --- a/test/box/gh-4672-min-integer-value-in-serializer.test.lua
> +++ b/test/box/gh-4672-min-integer-value-in-serializer.test.lua
> @@ -14,9 +14,9 @@ _ = s:create_index('ii')
> s:insert({1, -2^63})
> s:insert({2, -9223372036854775808LL})
> s:insert({3, 0})
> -s:update(3, {{'=', 'i', -2^63}})
> +s:update(3, {{'=', 2, -2^63}})
> s:insert({4, 0})
> -s:update(4, {{'=', 'i', -9223372036854775808LL}})
> +s:update(4, {{'=', 2, -9223372036854775808LL}})
>
> box.schema.user.grant('guest', 'read, write', 'space', 'serializer_test_space')
>
> @@ -26,9 +26,9 @@ s = cn.space.serializer_test_space
> s:insert({11, -2^63})
> s:insert({12, -9223372036854775808LL})
> s:insert({13, 0})
> -s:update(13, {{'=', 'i', -2^63}})
> +s:update(13, {{'=', 2, -2^63}})
> s:insert({14, 0})
> -s:update(14, {{'=', 'i', -9223372036854775808LL}})
> +s:update(14, {{'=', 2, -9223372036854775808LL}})
>
> cn:close()
> box.schema.user.revoke('guest', 'read, write', 'space', 'serializer_test_space')
> --
> 2.7.4
>
More information about the Tarantool-patches
mailing list