Skip to content

Commit d9d32e8

Browse files
committed
fix(tests): add Lua 5.5 support to gc, userdata, and resuming tests
- gc test: include lua55 in gcSetGenerational check - userdata test: include lua55 in setUserValue/getUserValue check - resuming test: remove lua54 skip, refactor to use switch for clarity
1 parent 9e151be commit d9d32e8

1 file changed

Lines changed: 15 additions & 6 deletions

File tree

src/tests.zig

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -728,7 +728,7 @@ test "garbage collector" {
728728
if (zlua.lang == .lua52) {
729729
lua.gcSetGenerational();
730730
lua.gcSetGenerational();
731-
} else if (zlua.lang == .lua54) {
731+
} else if (zlua.lang == .lua54 or zlua.lang == .lua55) {
732732
try expect(lua.gcSetGenerational(0, 10));
733733
try expect(lua.gcSetIncremental(0, 0, 0));
734734
try expect(!lua.gcSetIncremental(0, 0, 0));
@@ -967,7 +967,7 @@ test "userdata and uservalues" {
967967

968968
_ = lua.getUserValue(1);
969969
try expectEqual(.nil, lua.typeOf(-1));
970-
} else if (zlua.lang == .lua54) {
970+
} else if (zlua.lang == .lua54 or zlua.lang == .lua55) {
971971
// assign the user values
972972
lua.pushNumber(1234.56);
973973
try lua.setUserValue(1, 1);
@@ -1235,8 +1235,6 @@ test "yielding no continuation" {
12351235
}
12361236

12371237
test "resuming" {
1238-
if (zlua.lang == .lua54) return;
1239-
12401238
const lua: *Lua = try .init(testing.allocator);
12411239
defer lua.deinit();
12421240

@@ -1259,11 +1257,22 @@ test "resuming" {
12591257
var num_results: i32 = 0;
12601258
var i: i32 = 1;
12611259
while (i <= 5) : (i += 1) {
1262-
try expectEqual(.yield, if (zlua.lang == .lua51 or zlua.lang == .luajit) try thread.resumeThread(0) else if (zlua.lang == .lua54 or zlua.lang == .lua55) try thread.resumeThread(lua, 0, &num_results) else try thread.resumeThread(lua, 0));
1260+
try expectEqual(.yield, switch (zlua.lang) {
1261+
.lua51, .luajit => thread.resumeThread(0),
1262+
.lua54, .lua55 => thread.resumeThread(lua, 0, &num_results),
1263+
else => thread.resumeThread(lua, 0),
1264+
});
1265+
12631266
try expectEqual(i, thread.toInteger(-1));
12641267
lua.pop(lua.getTop());
12651268
}
1266-
try expectEqual(.ok, if (zlua.lang == .lua51 or zlua.lang == .luajit) try thread.resumeThread(0) else if (zlua.lang == .lua54 or zlua.lang == .lua55) try thread.resumeThread(lua, 0, &num_results) else try thread.resumeThread(lua, 0));
1269+
1270+
try expectEqual(.ok, switch (zlua.lang) {
1271+
.lua51, .luajit => thread.resumeThread(0),
1272+
.lua54, .lua55 => thread.resumeThread(lua, 0, &num_results),
1273+
else => thread.resumeThread(lua, 0),
1274+
});
1275+
12671276
try expectEqualStrings("done", try thread.toString(-1));
12681277
}
12691278

0 commit comments

Comments
 (0)