Merge pull request #8308 from roc-lang/better-zig-test-ux

Print nr of passed tests

Author: Anton-4

build.zig

@@ -20,6 +20,38 @@ fn configureBackend(step: *Step.Compile, target: ResolvedTarget) void {
     }
 }
 
+const TestsSummaryStep = struct {
+    step: Step,
+
+    fn create(b: *std.Build) *TestsSummaryStep {
+        const self = b.allocator.create(TestsSummaryStep) catch @panic("OOM");
+        self.* = .{
+            .step = Step.init(.{
+                .id = Step.Id.custom,
+                .name = "tests_summary",
+                .owner = b,
+                .makeFn = make,
+            }),
+        };
+        return self;
+    }
+
+    fn addRun(self: *TestsSummaryStep, run_step: *Step) void {
+        self.step.dependOn(run_step);
+    }
+
+    fn make(step: *Step, options: Step.MakeOptions) !void {
+        _ = options;
+
+        var passed: u64 = 0;
+        for (step.dependencies.items) |dependency| {
+            passed += @intCast(dependency.test_results.passCount());
+        }
+
+        std.debug.print("{d} tests passed.\n", .{passed});
+    }
+};
+
 pub fn build(b: *std.Build) void {
     // build steps
     const run_step = b.step("run", "Build and run the roc cli");
@@ -394,6 +426,7 @@ pub fn build(b: *std.Build) void {
     }
 
     // Create and add module tests
+    const tests_summary = TestsSummaryStep.create(b);
     const module_tests = roc_modules.createModuleTests(b, target, optimize, zstd, test_filters);
     for (module_tests) |module_test| {
         // Add compiled builtins to check module tests
@@ -419,7 +452,7 @@ pub fn build(b: *std.Build) void {
         individual_test_step.dependOn(&individual_run.step);
 
         b.default_step.dependOn(&module_test.test_step.step);
-        test_step.dependOn(&module_test.run_step.step);
+        tests_summary.addRun(&module_test.run_step.step);
     }
 
     // Add snapshot tool test
@@ -444,7 +477,7 @@ pub fn build(b: *std.Build) void {
         if (run_args.len != 0) {
             run_snapshot_test.addArgs(run_args);
         }
-        test_step.dependOn(&run_snapshot_test.step);
+        tests_summary.addRun(&run_snapshot_test.step);
     }
 
     // Add CLI test
@@ -468,7 +501,7 @@ pub fn build(b: *std.Build) void {
         if (run_args.len != 0) {
             run_cli_test.addArgs(run_args);
         }
-        test_step.dependOn(&run_cli_test.step);
+        tests_summary.addRun(&run_cli_test.step);
     }
 
     // Add watch tests
@@ -499,9 +532,11 @@ pub fn build(b: *std.Build) void {
         if (run_args.len != 0) {
             run_watch_test.addArgs(run_args);
         }
-        test_step.dependOn(&run_watch_test.step);
+        tests_summary.addRun(&run_watch_test.step);
     }
 
+    test_step.dependOn(&tests_summary.step);
+
     b.default_step.dependOn(playground_step);
     {
         const install = playground_test_install;

src/cli/cli_args.zig

@@ -1124,6 +1124,7 @@ test "roc help" {
         try testing.expectEqual(.help, std.meta.activeTag(result));
     }
 }
+
 test "roc licenses" {
     const gpa = testing.allocator;
     {