isolate test cases

Author: joachimschmidt557

src/boyer_moore.zig

@@ -3,6 +3,8 @@ const Allocator = std.mem.Allocator;
 const assert = std.debug.assert;
 const testing = std.testing;
 
+const test_suites = @import("test_cases.zig").test_suites;
+
 pub fn StringFinder(comptime T: type) type {
     assert(std.meta.trait.isIndexable(T));
     const ElemType = std.meta.Elem(T);
@@ -128,41 +130,15 @@ pub fn StringFinder(comptime T: type) type {
     };
 }
 
-test "empty pattern" {
-    const allocator = testing.allocator;
-
-    var sf = try StringFinder([]const u8).init(allocator, "");
-    defer sf.deinit();
-
-    testing.expectEqual(@as(?usize, 0), sf.next("zig"));
-    testing.expectEqual(@as(?usize, 0), sf.next(""));
-    testing.expectEqual(@as(?usize, 0), sf.next("a"));
-    testing.expectEqual(@as(?usize, 0), sf.next("lang"));
-}
-
-test "pattern with length 1" {
+test "boyer moore" {
     const allocator = testing.allocator;
 
-    var sf = try StringFinder([]const u8).init(allocator, "a");
-    defer sf.deinit();
+    for (test_suites) |suite| {
+        var sf = try StringFinder([]const u8).init(allocator, suite.pattern);
+        defer sf.deinit();
 
-    testing.expectEqual(@as(?usize, null), sf.next("zig"));
-    testing.expectEqual(@as(?usize, null), sf.next(""));
-    testing.expectEqual(@as(?usize, 0), sf.next("a"));
-    testing.expectEqual(@as(?usize, 1), sf.next("lang"));
-}
-
-test "test matching" {
-    const allocator = testing.allocator;
-
-    var sf = try StringFinder([]const u8).init(allocator, "zig");
-    defer sf.deinit();
-
-    testing.expectEqual(@as(?usize, 0), sf.next("zig"));
-    testing.expectEqual(@as(?usize, 0), sf.next("ziglang"));
-    testing.expectEqual(@as(?usize, 4), sf.next("langzig"));
-    testing.expectEqual(@as(?usize, 4), sf.next("langziglang"));
-    testing.expectEqual(@as(?usize, null), sf.next(""));
-    testing.expectEqual(@as(?usize, null), sf.next("firefox"));
-    testing.expectEqual(@as(?usize, 8), sf.next("abc abc ziglang"));
+        for (suite.cases) |case| {
+            testing.expectEqual(case.expected, sf.next(case.text));
+        }
+    }
 }

src/test_cases.zig

@@ -0,0 +1,42 @@
+pub const Testcase = struct {
+    text: []const u8,
+    expected: ?usize,
+};
+
+pub const Testsuite = struct {
+    pattern: []const u8,
+    cases: []const Testcase,
+};
+
+pub const test_suites = [_]Testsuite{
+    Testsuite{
+        .pattern = "",
+        .cases = &[_]Testcase{
+            .{ .text = "zig", .expected = 0 },
+            .{ .text = "", .expected = 0 },
+            .{ .text = "a", .expected = 0 },
+            .{ .text = "lang", .expected = 0 },
+        },
+    },
+    Testsuite{
+        .pattern = "a",
+        .cases = &[_]Testcase{
+            .{ .text = "zig", .expected = null },
+            .{ .text = "", .expected = null },
+            .{ .text = "a", .expected = 0 },
+            .{ .text = "lang", .expected = 1 },
+        },
+    },
+    Testsuite{
+        .pattern = "zig",
+        .cases = &[_]Testcase{
+            .{ .text = "zig", .expected = 0 },
+            .{ .text = "ziglang", .expected = 0 },
+            .{ .text = "langzig", .expected = 4 },
+            .{ .text = "langziglang", .expected = 4 },
+            .{ .text = "", .expected = null },
+            .{ .text = "firefox", .expected = null },
+            .{ .text = "abc abc ziglang", .expected = 8 },
+        },
+    },
+};