[Tarantool-patches] [PATCH] test: enable all LuaJIT tests for M1

[Tarantool-patches] [PATCH] test: enable all LuaJIT tests for M1

[Igor Munkin via Tarantool-patches]

This patch enables all tests in tarantool/luajit to be run in CI, since
all issues found with them are finally resolved.

Follows up #6098
Follows up tarantool/tarantool-qa#120

Signed-off-by: Igor Munkin <imun@tarantool.org>
---

Issue: https://github.com/tarantool/tarantool/issues/6098
Branch: https://github.com/tarantool/tarantool/tree/imun/gh-6098-fix-luajit-tests-suite-failures
CI: https://github.com/tarantool/tarantool/commit/67f92d2

 .travis.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.travis.mk b/.travis.mk
index db5116b25..e1d7ea194 100644
--- a/.travis.mk
+++ b/.travis.mk
@@ -396,7 +396,7 @@ test_osx_no_deps: build_osx
 # FIXME: Temporary target with reduced number of tests.
 # Use test_osx_no_deps target, when all M1 issues are resolved.
 test_osx_arm64_no_deps: build_osx
-	make PUC-Rio-Lua-5.1-tests lua-Harness-tests tarantool-tests
+	make LuaJIT-test
 
 test_osx: deps_osx test_osx_no_deps
 
-- 
2.25.0

Re: [Tarantool-patches] [PATCH] test: enable all LuaJIT tests for M1

[Sergey Kaplun via Tarantool-patches]

Hi, Igor!

Thanks for the patch, LGTM!

P.S. Can we enable Tarantool tests too?

-- 
Best regards,
Sergey Kaplun

Re: [Tarantool-patches] [PATCH] test: enable all LuaJIT tests for M1

[Igor Munkin via Tarantool-patches]

Sergey,

Thanks for your review!

On 19.08.21, Sergey Kaplun wrote:
> Hi, Igor!
> 
> Thanks for the patch, LGTM!

Added your tag:
| Reviewed-by: Sergey Kaplun <skaplun@tarantool.org>

> 
> P.S. Can we enable Tarantool tests too?

Unfortunately, not today: there are still some issues open for M1.

> 
> -- 
> Best regards,
> Sergey Kaplun

-- 
Best regards,
IM

Re: [Tarantool-patches] [PATCH] test: enable all LuaJIT tests for M1

[Vitaliia Ioffe via Tarantool-patches]

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


Hi team, 
 
QA LGTM
 
 
--
Vitaliia Ioffe
 
  
>Четверг, 19 августа 2021, 9:29 +03:00 от Igor Munkin <imun@tarantool.org>:
> 
>Sergey,
>
>Thanks for your review!
>
>On 19.08.21, Sergey Kaplun wrote:
>> Hi, Igor!
>>
>> Thanks for the patch, LGTM!
>
>Added your tag:
>| Reviewed-by: Sergey Kaplun < skaplun@tarantool.org >
>
>>
>> P.S. Can we enable Tarantool tests too?
>
>Unfortunately, not today: there are still some issues open for M1.
>
>>
>> --
>> Best regards,
>> Sergey Kaplun
>
>--
>Best regards,
>IM
 

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

Re: [Tarantool-patches] [PATCH] test: enable all LuaJIT tests for M1

[Igor Munkin via Tarantool-patches]

Vitaliia,

Thanks for your review!

On 19.08.21, Vitaliia Ioffe wrote:
> 
> Hi team, 
>  
> QA LGTM

Added your tag:
| Reviewed-by: Vitaliia Ioffe <v.ioffe@tarantool.org>

>  
>  
> --
> Vitaliia Ioffe
>  

<snipped>

>  

-- 
Best regards,
IM

Re: [Tarantool-patches] [PATCH] test: enable all LuaJIT tests for M1

[Igor Munkin via Tarantool-patches]

I've checked the patch into master.

On 18.08.21, Igor Munkin wrote:
> This patch enables all tests in tarantool/luajit to be run in CI, since
> all issues found with them are finally resolved.
> 
> Follows up #6098
> Follows up tarantool/tarantool-qa#120
> 
> Signed-off-by: Igor Munkin <imun@tarantool.org>
> ---
> 
> Issue: https://github.com/tarantool/tarantool/issues/6098
> Branch: https://github.com/tarantool/tarantool/tree/imun/gh-6098-fix-luajit-tests-suite-failures
> CI: https://github.com/tarantool/tarantool/commit/67f92d2
> 
>  .travis.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

<snipped>

> -- 
> 2.25.0
> 

-- 
Best regards,
IM