Skip to content

Commit 6968178

Browse files
committed
update tests
1 parent 995e4c7 commit 6968178

File tree

1 file changed

+9
-13
lines changed

1 file changed

+9
-13
lines changed

src/TrigramStore.zig

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,8 @@
33
const std = @import("std");
44
const ast = @import("ast.zig");
55
const Ast = std.zig.Ast;
6-
const builtin = @import("builtin");
76
const assert = std.debug.assert;
87
const offsets = @import("offsets.zig");
9-
const URI = @import("uri.zig");
10-
const log = std.log.scoped(.store);
118

129
pub const TrigramStore = @This();
1310

@@ -239,7 +236,7 @@ fn mergeIntersection(
239236
b: []const Declaration.Index,
240237
out: []Declaration.Index,
241238
) u32 {
242-
std.debug.assert(@min(a.len, b.len) <= out.len);
239+
assert(@min(a.len, b.len) <= out.len);
243240

244241
var out_idx: u32 = 0;
245242

@@ -438,17 +435,17 @@ test CuckooFilter {
438435
try entries.ensureTotalCapacity(allocator, element_count);
439436

440437
var buckets: [filter_size]CuckooFilter.Bucket = undefined;
441-
var filter = CuckooFilter{ .buckets = &buckets };
442-
var filter_prng = std.Random.DefaultPrng.init(42);
438+
var filter: CuckooFilter = .{ .buckets = &buckets };
439+
var filter_prng: std.Random.DefaultPrng = .init(42);
443440

444441
for (0..2_500) |gen_prng_seed| {
445442
entries.clearRetainingCapacity();
446443
filter.reset();
447444

448-
var gen_prng = std.Random.DefaultPrng.init(gen_prng_seed);
445+
var gen_prng: std.Random.DefaultPrng = .init(gen_prng_seed);
449446
for (0..element_count) |_| {
450447
const trigram: Trigram = @bitCast(gen_prng.random().int(u24));
451-
try entries.put(allocator, trigram, {});
448+
entries.putAssumeCapacity(trigram, {});
452449
try filter.append(filter_prng.random(), trigram);
453450
}
454451

@@ -460,7 +457,7 @@ test CuckooFilter {
460457
// Reasonable false positive rate
461458
const fpr_count = 2_500;
462459
var false_positives: usize = 0;
463-
var negative_prng = std.Random.DefaultPrng.init(~gen_prng_seed);
460+
var negative_prng: std.Random.DefaultPrng = .init(~gen_prng_seed);
464461
for (0..fpr_count) |_| {
465462
var trigram: Trigram = @bitCast(negative_prng.random().int(u24));
466463
while (entries.contains(trigram)) {
@@ -471,9 +468,8 @@ test CuckooFilter {
471468
}
472469

473470
const fpr = @as(f32, @floatFromInt(false_positives)) / fpr_count;
474-
std.testing.expect(fpr < 0.035) catch |err| {
475-
std.log.err("fpr: {d}%", .{fpr * 100});
476-
return err;
477-
};
471+
472+
errdefer std.log.err("fpr: {d}%", .{fpr * 100});
473+
try std.testing.expect(fpr < 0.035);
478474
}
479475
}

0 commit comments

Comments
 (0)