Skip to content

Commit 420b909

Browse files
committed
Fix tests
1 parent e4f3c4b commit 420b909

File tree

3 files changed

+35
-33
lines changed

3 files changed

+35
-33
lines changed

src/Index.zig

+1
Original file line numberDiff line numberDiff line change
@@ -531,6 +531,7 @@ pub fn search(self: *Self, hashes: []const u32, allocator: std.mem.Allocator, de
531531
try segments.memory_segments.value.search(sorted_hashes, &results, deadline);
532532

533533
results.sort();
534+
534535
return results;
535536
}
536537

src/index_tests.zig

+33-33
Original file line numberDiff line numberDiff line change
@@ -98,49 +98,49 @@ test "index create, update, reopen and search" {
9898
}
9999
}
100100

101-
// test "index many updates" {
102-
// var tmp_dir = std.testing.tmpDir(.{});
103-
// defer tmp_dir.cleanup();
101+
test "index many updates" {
102+
var tmp_dir = std.testing.tmpDir(.{});
103+
defer tmp_dir.cleanup();
104104

105-
// var hashes: [100]u32 = undefined;
105+
var hashes: [100]u32 = undefined;
106106

107-
// {
108-
// var index = try Index.init(std.testing.allocator, tmp_dir.dir, .{ .create = true });
109-
// defer index.deinit();
107+
{
108+
var index = try Index.init(std.testing.allocator, tmp_dir.dir, .{ .create = true });
109+
defer index.deinit();
110110

111-
// try index.open();
111+
try index.open();
112112

113-
// for (0..100) |i| {
114-
// try index.update(&[_]Change{.{ .insert = .{
115-
// .id = @as(u32, @intCast(i % 20)) + 1,
116-
// .hashes = generateRandomHashes(&hashes, i),
117-
// } }});
118-
// }
119-
// }
113+
for (0..100) |i| {
114+
try index.update(&[_]Change{.{ .insert = .{
115+
.id = @as(u32, @intCast(i % 20)) + 1,
116+
.hashes = generateRandomHashes(&hashes, i),
117+
} }});
118+
}
119+
}
120120

121-
// var index = try Index.init(std.testing.allocator, tmp_dir.dir, .{ .create = false });
122-
// defer index.deinit();
121+
var index = try Index.init(std.testing.allocator, tmp_dir.dir, .{ .create = false });
122+
defer index.deinit();
123123

124-
// try index.open();
124+
try index.open();
125125

126-
// {
127-
// var results = try index.search(generateRandomHashes(&hashes, 0), std.testing.allocator, .{});
128-
// defer results.deinit();
126+
{
127+
var results = try index.search(generateRandomHashes(&hashes, 0), std.testing.allocator, .{});
128+
defer results.deinit();
129129

130-
// const result = results.get(1);
131-
// try std.testing.expect(result == null or result.?.score == 0);
132-
// }
130+
const result = results.get(1);
131+
try std.testing.expect(result == null or result.?.score == 0);
132+
}
133133

134-
// {
135-
// var results = try index.search(generateRandomHashes(&hashes, 80), std.testing.allocator, .{});
136-
// defer results.deinit();
134+
{
135+
var results = try index.search(generateRandomHashes(&hashes, 80), std.testing.allocator, .{});
136+
defer results.deinit();
137137

138-
// const result = results.get(1);
139-
// try std.testing.expect(result != null);
140-
// try std.testing.expectEqual(1, result.?.id);
141-
// try std.testing.expectEqual(hashes.len, result.?.score);
142-
// }
143-
// }
138+
const result = results.get(1);
139+
try std.testing.expect(result != null);
140+
try std.testing.expectEqual(1, result.?.id);
141+
try std.testing.expectEqual(hashes.len, result.?.score);
142+
}
143+
}
144144

145145
test "index, multiple fingerprints with the same hashes" {
146146
var tmp_dir = std.testing.tmpDir(.{});

src/segment_list.zig

+1
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,7 @@ pub fn SegmentList(Segment: type) type {
110110
}
111111
try node.value.search(hashes, results);
112112
}
113+
results.removeOutdatedResults(self);
113114
}
114115

115116
pub fn getMaxCommitId(self: Self) u64 {

0 commit comments

Comments
 (0)