Skip to content

Commit dab3200

Browse files
committed
refactor(test): rename skip error for Zig version checks
- Replace SkipZigVersionCheck with SkipZigTest for clarity - Apply changes across all PackerIO test functions - Simplify test skip error naming convention
1 parent 9a70ef4 commit dab3200

File tree

1 file changed

+33
-33
lines changed

1 file changed

+33
-33
lines changed

src/test.zig

Lines changed: 33 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ fn u8eql(a: []const u8, b: []const u8) bool {
1515
// ============================================================================
1616

1717
test "PackerIO: truncated data error" {
18-
if (!has_new_io) return error.SkipZigVersionCheck;
18+
if (!has_new_io) return error.SkipZigTest;
1919

2020
// Test reading truncated integer data (simpler case without allocations)
2121
var full_buffer: [100]u8 = undefined;
@@ -50,7 +50,7 @@ test "PackerIO: truncated data error" {
5050
}
5151

5252
test "PackerIO: invalid msgpack marker" {
53-
if (!has_new_io) return error.SkipZigVersionCheck;
53+
if (!has_new_io) return error.SkipZigTest;
5454

5555
// 0xc1 is a reserved/invalid marker byte in MessagePack
5656
var buffer: [10]u8 = [_]u8{ 0xc1, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 };
@@ -70,7 +70,7 @@ test "PackerIO: invalid msgpack marker" {
7070
}
7171

7272
test "PackerIO: corrupted length field" {
73-
if (!has_new_io) return error.SkipZigVersionCheck;
73+
if (!has_new_io) return error.SkipZigTest;
7474

7575
var buffer: [100]u8 = undefined;
7676
var input = if (builtin.zig_version.minor == 14)
@@ -126,7 +126,7 @@ test "PackerIO: corrupted length field" {
126126
}
127127

128128
test "PackerIO: multiple payloads with error recovery" {
129-
if (!has_new_io) return error.SkipZigVersionCheck;
129+
if (!has_new_io) return error.SkipZigTest;
130130

131131
var buffer: [4096]u8 = std.mem.zeroes([4096]u8);
132132
var writer = std.Io.Writer.fixed(&buffer);
@@ -171,7 +171,7 @@ test "PackerIO: multiple payloads with error recovery" {
171171
// ============================================================================
172172

173173
test "PackerIO: sequential writes and reads with fixed buffer" {
174-
if (!has_new_io) return error.SkipZigVersionCheck;
174+
if (!has_new_io) return error.SkipZigTest;
175175

176176
var buffer: [4096]u8 = undefined;
177177
var writer = std.Io.Writer.fixed(&buffer);
@@ -212,7 +212,7 @@ test "PackerIO: sequential writes and reads with fixed buffer" {
212212
// ============================================================================
213213

214214
test "PackerIO: large string near limit" {
215-
if (!has_new_io) return error.SkipZigVersionCheck;
215+
if (!has_new_io) return error.SkipZigTest;
216216

217217
const allocator_heap = std.heap.page_allocator;
218218

@@ -249,7 +249,7 @@ test "PackerIO: large string near limit" {
249249
}
250250

251251
test "PackerIO: large binary data" {
252-
if (!has_new_io) return error.SkipZigVersionCheck;
252+
if (!has_new_io) return error.SkipZigTest;
253253

254254
const allocator_heap = std.heap.page_allocator;
255255

@@ -291,7 +291,7 @@ test "PackerIO: large binary data" {
291291
}
292292

293293
test "PackerIO: large array (1000 elements)" {
294-
if (!has_new_io) return error.SkipZigVersionCheck;
294+
if (!has_new_io) return error.SkipZigTest;
295295

296296
const allocator_heap = std.heap.page_allocator;
297297

@@ -327,7 +327,7 @@ test "PackerIO: large array (1000 elements)" {
327327
}
328328

329329
test "PackerIO: large map (500 entries)" {
330-
if (!has_new_io) return error.SkipZigVersionCheck;
330+
if (!has_new_io) return error.SkipZigTest;
331331

332332
const allocator_heap = std.heap.page_allocator;
333333

@@ -377,7 +377,7 @@ test "PackerIO: large map (500 entries)" {
377377
// ============================================================================
378378

379379
test "PackerIO: empty buffer write error" {
380-
if (!has_new_io) return error.SkipZigVersionCheck;
380+
if (!has_new_io) return error.SkipZigTest;
381381

382382
var buffer: [0]u8 = undefined;
383383
var writer = std.Io.Writer.fixed(&buffer);
@@ -397,7 +397,7 @@ test "PackerIO: empty buffer write error" {
397397
}
398398

399399
test "PackerIO: minimal buffer size" {
400-
if (!has_new_io) return error.SkipZigVersionCheck;
400+
if (!has_new_io) return error.SkipZigTest;
401401

402402
// Test with exactly 1 byte buffer (enough for nil marker)
403403
var buffer: [1]u8 = undefined;
@@ -417,7 +417,7 @@ test "PackerIO: minimal buffer size" {
417417
}
418418

419419
test "PackerIO: exact buffer size for small payload" {
420-
if (!has_new_io) return error.SkipZigVersionCheck;
420+
if (!has_new_io) return error.SkipZigTest;
421421

422422
// positive fixint uses exactly 1 byte
423423
var buffer: [1]u8 = undefined;
@@ -436,7 +436,7 @@ test "PackerIO: exact buffer size for small payload" {
436436
}
437437

438438
test "PackerIO: off-by-one buffer size" {
439-
if (!has_new_io) return error.SkipZigVersionCheck;
439+
if (!has_new_io) return error.SkipZigTest;
440440

441441
// uint8 needs 2 bytes (marker + value), provide only 1
442442
var buffer: [1]u8 = undefined;
@@ -457,7 +457,7 @@ test "PackerIO: off-by-one buffer size" {
457457
}
458458

459459
test "PackerIO: empty string edge case" {
460-
if (!has_new_io) return error.SkipZigVersionCheck;
460+
if (!has_new_io) return error.SkipZigTest;
461461

462462
var buffer: [10]u8 = undefined;
463463
var writer = std.Io.Writer.fixed(&buffer);
@@ -479,7 +479,7 @@ test "PackerIO: empty string edge case" {
479479
}
480480

481481
test "PackerIO: empty array edge case" {
482-
if (!has_new_io) return error.SkipZigVersionCheck;
482+
if (!has_new_io) return error.SkipZigTest;
483483

484484
var buffer: [10]u8 = undefined;
485485
var writer = std.Io.Writer.fixed(&buffer);
@@ -501,7 +501,7 @@ test "PackerIO: empty array edge case" {
501501
}
502502

503503
test "PackerIO: empty map edge case" {
504-
if (!has_new_io) return error.SkipZigVersionCheck;
504+
if (!has_new_io) return error.SkipZigTest;
505505

506506
var buffer: [10]u8 = undefined;
507507
var writer = std.Io.Writer.fixed(&buffer);
@@ -4877,7 +4877,7 @@ test "memory alignment: large integer array serialization" {
48774877
const has_new_io = builtin.zig_version.minor >= 15;
48784878

48794879
test "PackerIO: basic write and read with fixed Reader/Writer" {
4880-
if (!has_new_io) return error.SkipZigVersionCheck;
4880+
if (!has_new_io) return error.SkipZigTest;
48814881

48824882
var buffer: [1024]u8 = undefined;
48834883
var writer = std.Io.Writer.fixed(&buffer);
@@ -4901,7 +4901,7 @@ test "PackerIO: basic write and read with fixed Reader/Writer" {
49014901
}
49024902

49034903
test "PackerIO: nil type" {
4904-
if (!has_new_io) return error.SkipZigVersionCheck;
4904+
if (!has_new_io) return error.SkipZigTest;
49054905

49064906
var buffer: [1024]u8 = undefined;
49074907
var writer = std.Io.Writer.fixed(&buffer);
@@ -4920,7 +4920,7 @@ test "PackerIO: nil type" {
49204920
}
49214921

49224922
test "PackerIO: bool type" {
4923-
if (!has_new_io) return error.SkipZigVersionCheck;
4923+
if (!has_new_io) return error.SkipZigTest;
49244924

49254925
var buffer: [1024]u8 = undefined;
49264926

@@ -4960,7 +4960,7 @@ test "PackerIO: bool type" {
49604960
}
49614961

49624962
test "PackerIO: signed integers" {
4963-
if (!has_new_io) return error.SkipZigVersionCheck;
4963+
if (!has_new_io) return error.SkipZigTest;
49644964

49654965
var buffer: [1024]u8 = undefined;
49664966

@@ -4988,7 +4988,7 @@ test "PackerIO: signed integers" {
49884988
}
49894989

49904990
test "PackerIO: unsigned integers" {
4991-
if (!has_new_io) return error.SkipZigVersionCheck;
4991+
if (!has_new_io) return error.SkipZigTest;
49924992

49934993
var buffer: [1024]u8 = undefined;
49944994

@@ -5012,7 +5012,7 @@ test "PackerIO: unsigned integers" {
50125012
}
50135013

50145014
test "PackerIO: float type" {
5015-
if (!has_new_io) return error.SkipZigVersionCheck;
5015+
if (!has_new_io) return error.SkipZigTest;
50165016

50175017
var buffer: [1024]u8 = undefined;
50185018

@@ -5051,7 +5051,7 @@ test "PackerIO: float type" {
50515051
}
50525052

50535053
test "PackerIO: string type" {
5054-
if (!has_new_io) return error.SkipZigVersionCheck;
5054+
if (!has_new_io) return error.SkipZigTest;
50555055

50565056
var buffer: [1024]u8 = undefined;
50575057

@@ -5083,7 +5083,7 @@ test "PackerIO: string type" {
50835083
}
50845084

50855085
test "PackerIO: binary type" {
5086-
if (!has_new_io) return error.SkipZigVersionCheck;
5086+
if (!has_new_io) return error.SkipZigTest;
50875087

50885088
var buffer: [1024]u8 = undefined;
50895089

@@ -5106,7 +5106,7 @@ test "PackerIO: binary type" {
51065106
}
51075107

51085108
test "PackerIO: array type" {
5109-
if (!has_new_io) return error.SkipZigVersionCheck;
5109+
if (!has_new_io) return error.SkipZigTest;
51105110

51115111
var buffer: [1024]u8 = undefined;
51125112
var writer = std.Io.Writer.fixed(&buffer);
@@ -5152,7 +5152,7 @@ test "PackerIO: array type" {
51525152
}
51535153

51545154
test "PackerIO: map type" {
5155-
if (!has_new_io) return error.SkipZigVersionCheck;
5155+
if (!has_new_io) return error.SkipZigTest;
51565156

51575157
var buffer: [2048]u8 = undefined;
51585158
var writer = std.Io.Writer.fixed(&buffer);
@@ -5185,7 +5185,7 @@ test "PackerIO: map type" {
51855185
}
51865186

51875187
test "PackerIO: nested structures" {
5188-
if (!has_new_io) return error.SkipZigVersionCheck;
5188+
if (!has_new_io) return error.SkipZigTest;
51895189

51905190
var buffer: [4096]u8 = undefined;
51915191
var writer = std.Io.Writer.fixed(&buffer);
@@ -5221,7 +5221,7 @@ test "PackerIO: nested structures" {
52215221
}
52225222

52235223
test "PackerIO: timestamp extension type" {
5224-
if (!has_new_io) return error.SkipZigVersionCheck;
5224+
if (!has_new_io) return error.SkipZigTest;
52255225

52265226
var buffer: [1024]u8 = undefined;
52275227

@@ -5251,7 +5251,7 @@ test "PackerIO: timestamp extension type" {
52515251
}
52525252

52535253
test "PackerIO: extension type" {
5254-
if (!has_new_io) return error.SkipZigVersionCheck;
5254+
if (!has_new_io) return error.SkipZigTest;
52555255

52565256
var buffer: [1024]u8 = undefined;
52575257
var writer = std.Io.Writer.fixed(&buffer);
@@ -5275,7 +5275,7 @@ test "PackerIO: extension type" {
52755275
}
52765276

52775277
test "PackerIO: deeply nested structures" {
5278-
if (!has_new_io) return error.SkipZigVersionCheck;
5278+
if (!has_new_io) return error.SkipZigTest;
52795279

52805280
var buffer: [4096]u8 = undefined;
52815281
var writer = std.Io.Writer.fixed(&buffer);
@@ -5308,7 +5308,7 @@ test "PackerIO: deeply nested structures" {
53085308
}
53095309

53105310
test "PackerIO: multiple writes and reads" {
5311-
if (!has_new_io) return error.SkipZigVersionCheck;
5311+
if (!has_new_io) return error.SkipZigTest;
53125312

53135313
var buffer: [4096]u8 = undefined;
53145314
var writer = std.Io.Writer.fixed(&buffer);
@@ -5337,7 +5337,7 @@ test "PackerIO: multiple writes and reads" {
53375337
}
53385338

53395339
test "PackerIO: large array" {
5340-
if (!has_new_io) return error.SkipZigVersionCheck;
5340+
if (!has_new_io) return error.SkipZigTest;
53415341

53425342
var buffer: [16384]u8 = undefined;
53435343
var writer = std.Io.Writer.fixed(&buffer);
@@ -5367,7 +5367,7 @@ test "PackerIO: large array" {
53675367
}
53685368

53695369
test "PackerIO: packIO convenience function" {
5370-
if (!has_new_io) return error.SkipZigVersionCheck;
5370+
if (!has_new_io) return error.SkipZigTest;
53715371

53725372
var buffer: [1024]u8 = undefined;
53735373
var writer = std.Io.Writer.fixed(&buffer);

0 commit comments

Comments
 (0)