ReviewOS

also looking at this

zig-utils/zig-test-framework

test: make CI exercise maintained integrations

#18
Merged glennmichael123 wants to merge codex/issue-3-test-coverage into main
16 files +390 -139
tests/test_runner_test.zigmodified+22-11
Changes to tests/test_runner_test.zig
@@ -1,19 +1,30 @@
11const std = ;
2const ztf = ;
23
3// Placeholder tests for test_runner module
4// TODO: Implement comprehensive tests for TestRunner
4fn passingTest(_: std.mem.Allocator) !void {}
55
66test "test runner initialization" {
7 // Test that TestRunner can be initialized
8 try std.testing.expect(true);
7 var registry = ztf.TestRegistry.init(std.testing.allocator);
8 defer registry.deinit();
9 var runner = ztf.TestRunner.init(std.testing.allocator, &registry, .{});
10 defer runner.deinit();
11 try std.testing.expectEqual(@as(usize, 0), runner.results.total);
12 try std.testing.expectEqual(ztf.ReporterType.spec, runner.options.reporter_type);
913}
1014
11test "test runner executes tests" {
12 // Test that TestRunner can execute registered tests
13 try std.testing.expect(true);
14}
15test "test runner executes matching tests and skips filtered tests" {
16 const allocator = std.testing.allocator;
17 var registry = ztf.TestRegistry.init(allocator);
18 defer registry.deinit();
19 const suite = try ztf.TestSuite.init(allocator, "runner suite");
20 try suite.addTest(ztf.TestCase.init("selected", passingTest));
21 try suite.addTest(ztf.TestCase.init("ignored", passingTest));
22 try registry.registerSuite(suite);
1523
16test "test runner reports results" {
17 // Test that TestRunner correctly reports test results
18 try std.testing.expect(true);
24 var runner = ztf.TestRunner.init(allocator, &registry, .{ .filter = "selected", .use_colors = false });
25 defer runner.deinit();
26 try std.testing.expect(try runner.run());
27 try std.testing.expectEqual(@as(usize, 2), runner.results.total);
28 try std.testing.expectEqual(@as(usize, 1), runner.results.passed);
29 try std.testing.expectEqual(@as(usize, 1), runner.results.skipped);
1930}