test: cover discovery filtering end to end

Author: glennmichael123

README.md

@@ -94,8 +94,15 @@ This will automatically discover and run all `*.test.zig` files in the `tests` d
 - `--pattern <pattern>` - File pattern to match (default: `*.test.zig`)
 - `--no-recursive` - Disable recursive directory search
 - `--bail` - Stop on first failure
+- `--filter <text>` / `--grep <text>` - Run only Zig tests whose names contain the text
+- `--no-color` - Disable color in child Zig test processes
 - `--verbose` - Show detailed output
 
+Discovery mode rejects options that it cannot honor yet instead of silently
+ignoring them. Reporter selection, quiet output, timeouts, parallel jobs,
+snapshot updates, memory profiling, configuration files, JUnit output, and the
+web UI require the shared execution pipeline planned for future releases.
+
 **Examples:**
 
 ```bash

build.zig

@@ -130,6 +130,9 @@ pub fn build(b: *std.Build) void {
             .target = target,
             .optimize = optimize,
             .link_libc = true,
+            .imports = &.{
+                .{ .name = "zig_test_framework", .module = lib_module },
+            },
         }),
     });
     const run_filter_tests = b.addRunArtifact(filter_tests);

tests/filter_test.zig

@@ -1,19 +1,21 @@
 const std = @import("std");
+const ztf = @import("zig_test_framework");
 
-// Placeholder tests for filter module
-// TODO: Implement comprehensive tests for test filtering
+test "discovery forwards test-name filters to zig test" {
+    const allocator = std.testing.allocator;
+    var discovered = ztf.DiscoveryResult.init(allocator);
+    defer discovered.deinit();
 
-test "filter by test name" {
-    // Test filtering tests by name
-    try std.testing.expect(true);
-}
+    try discovered.addFile(
+        "tests/fixtures/filter_fixture.zig",
+        "fixtures/filter_fixture.zig",
+        "filter_fixture.zig",
+    );
 
-test "filter by pattern" {
-    // Test filtering tests by pattern
-    try std.testing.expect(true);
-}
+    const passed = try ztf.runDiscoveredTests(allocator, &discovered, .{
+        .filter = "selected test passes",
+        .use_colors = false,
+    });
 
-test "filter combination" {
-    // Test combined filters
-    try std.testing.expect(true);
+    try std.testing.expect(passed);
 }

tests/fixtures/filter_fixture.zig

@@ -0,0 +1,9 @@
+const std = @import("std");
+
+test "selected test passes" {
+    try std.testing.expect(true);
+}
+
+test "unselected test fails" {
+    return error.ShouldHaveBeenFiltered;
+}