test: enable back tests for LuaJIT#10824
Merged
Buristan merged 2 commits intotarantool:masterfrom Dec 17, 2024
Merged
Conversation
37775ab to
9ba5ac0
Compare
efff08e to
9ebb380
Compare
9ebb380 to
49de4f3
Compare
49de4f3 to
e6f7058
Compare
ligurio
reviewed
Dec 17, 2024
This patch is a follow-up for the commit 9542cc2 ("lua: add custom allocator with memory limit"). LuaJIT tests with enabled GC64 may consume more than 2 GiB of memory. To avoid failure for these tests, just set the limit to 128 TiB (LuaJIT maximum for GC64 mode). Follows up tarantool#8881 NO_DOC=fix LuaJIT tests NO_CHANGELOG=fix LuaJIT tests NO_TEST=fix LuaJIT tests
Since the commit db351d3 ("luajit: bump new version"), which introduces CTest as a launcher for LuaJIT's test suites, the LuaJIT tests are not run by Tarantool since `LUAJIT_USE_TEST` is disabled. This patch enables all tests, except ASAN build, due to tarantool#10733. NO_DOC=testing NO_CHANGELOG=testing NO_TEST=enable tests back
e6f7058 to
55f68a9
Compare
Totktonada
approved these changes
Dec 17, 2024
Contributor
Totktonada
left a comment
There was a problem hiding this comment.
No objections from me.
This was referenced Dec 17, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Since the commit db351d3 ("luajit: bump new version"), which introduces CTest as a launcher for LuaJIT's test suites, the LuaJIT tests are not run by Tarantool since
LUAJIT_USE_TESTis disabled. This patch enables all tests, except ASAN build, due to #10733.NO_DOC=testing
NO_CHANGELOG=testing
NO_TEST=enable tests back