Re: [Tarantool-patches] [PATCH v2 luajit 24/45] test: enable <misc/parse_esc.lua> LuaJIT test

[Sergey Bronnikov via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

[-- Attachment #1: Type: text/plain, Size: 1898 bytes --]

Sergey,

On 21.08.2024 11:58, Sergey Kaplun wrote:
> This patch moves the aforementioned test from the <misc> to the <lang/>
> directory, includes it in <index>, and names the subtest.
>
> Part of tarantool/tarantool#9398
> ---
thanks for the patch! LGTM
>   test/LuaJIT-tests/lang/index         |  1 +
>   test/LuaJIT-tests/lang/parse_esc.lua | 10 ++++++++++
>   test/LuaJIT-tests/misc/parse_esc.lua |  7 -------
>   3 files changed, 11 insertions(+), 7 deletions(-)
>   create mode 100644 test/LuaJIT-tests/lang/parse_esc.lua
>   delete mode 100644 test/LuaJIT-tests/misc/parse_esc.lua
>
> diff --git a/test/LuaJIT-tests/lang/index b/test/LuaJIT-tests/lang/index
> index 371a5768..0d1175a8 100644
> --- a/test/LuaJIT-tests/lang/index
> +++ b/test/LuaJIT-tests/lang/index
> @@ -18,6 +18,7 @@ concat.lua
>   self.lua
>   table.lua
>   parse_comp.lua
> +parse_esc.lua
>   upvalue
>   tail_recursion.lua
>   vararg_jit.lua
> diff --git a/test/LuaJIT-tests/lang/parse_esc.lua b/test/LuaJIT-tests/lang/parse_esc.lua
> new file mode 100644
> index 00000000..c14d347a
> --- /dev/null
> +++ b/test/LuaJIT-tests/lang/parse_esc.lua
> @@ -0,0 +1,10 @@
> +do --- Base parsing of escape sequences.
> +  assert("\79\126" == "O~")
> +  assert("\x4f\x7e" == "O~")
> +  assert(loadstring[[return "\xxx"]] == nil)
> +  assert(loadstring[[return "\xxx"]] == nil)
> +
> +  assert(assert(loadstring[[return "abc   \z
> +
> +     def"]])() == "abc   def")
> +end
> diff --git a/test/LuaJIT-tests/misc/parse_esc.lua b/test/LuaJIT-tests/misc/parse_esc.lua
> deleted file mode 100644
> index 4bcce0e8..00000000
> --- a/test/LuaJIT-tests/misc/parse_esc.lua
> +++ /dev/null
> @@ -1,7 +0,0 @@
> -assert("\79\126" == "O~")
> -assert("\x4f\x7e" == "O~")
> -assert(loadstring[[return "\xxx"]] == nil)
> -assert(loadstring[[return "\xxx"]] == nil)
> -assert(assert(loadstring[[return "abc   \z
> -
> -   def"]])() == "abc   def")

[-- Attachment #2: Type: text/html, Size: 2477 bytes --]