[Tarantool-patches] [PATCH luajit v2 1/5] test: disable `lj-603-snap-restore` test
Sergey Kaplun
skaplun at tarantool.org
Fri Mar 17 14:25:04 MSK 2023
Hi, Maxim!
Thanks for the patch!
Please, consider my comment below.
On 14.03.23, Maxim Kokryashkin wrote:
> The test is extremely fragile and it is burdening
> to fix it in scope of every major change.
> ---
> test/tarantool-tests/lj-603-err-snap-restore.test.lua | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/test/tarantool-tests/lj-603-err-snap-restore.test.lua b/test/tarantool-tests/lj-603-err-snap-restore.test.lua
> index 6eb53dfd..2f2eec7d 100644
> --- a/test/tarantool-tests/lj-603-err-snap-restore.test.lua
> +++ b/test/tarantool-tests/lj-603-err-snap-restore.test.lua
> @@ -2,7 +2,9 @@ local tap = require('tap')
> -- Test to demonstrate the incorrect JIT behaviour when an error
> -- is raised on restoration from the snapshot.
> -- See also https://github.com/LuaJIT/LuaJIT/issues/603.
> -local test = tap.test('lj-603-err-snap-restore')
> +local test = tap.test('lj-603-err-snap-restore'):skipcond({
> + ['Too fragile, temporarily disabled.'] = true
> +})
I've reverted your patch at the last commit (4d5e9afe) and everything
working just fine. I suggest to drop these changes.
>
> test:plan(2)
>
> --
> 2.39.0
>
--
Best regards,
Sergey Kaplun
More information about the Tarantool-patches
mailing list