Skip to content

Commit feadb9d

Browse files
author
Lei Jin
committed
fix cuckoo table builder test
Summary: as title Test Plan: ./cuckoo_table_builder_test Reviewers:igor CC:leveldb Task ID: # Blame Rev:
1 parent 3c232e1 commit feadb9d

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

table/cuckoo_table_builder_test.cc

+8-8
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ TEST(CuckooBuilderTest, WriteSuccessNoCollisionFullKey) {
170170
ASSERT_OK(builder.status());
171171
}
172172
uint32_t bucket_size = keys[0].size() + values[0].size();
173-
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
173+
ASSERT_EQ(expected_table_size * bucket_size - 1, builder.FileSize());
174174
ASSERT_OK(builder.Finish());
175175
ASSERT_OK(writable_file->Close());
176176
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
@@ -210,7 +210,7 @@ TEST(CuckooBuilderTest, WriteSuccessWithCollisionFullKey) {
210210
ASSERT_OK(builder.status());
211211
}
212212
uint32_t bucket_size = keys[0].size() + values[0].size();
213-
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
213+
ASSERT_EQ(expected_table_size * bucket_size - 1, builder.FileSize());
214214
ASSERT_OK(builder.Finish());
215215
ASSERT_OK(writable_file->Close());
216216
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
@@ -251,7 +251,7 @@ TEST(CuckooBuilderTest, WriteSuccessWithCollisionAndCuckooBlock) {
251251
ASSERT_OK(builder.status());
252252
}
253253
uint32_t bucket_size = keys[0].size() + values[0].size();
254-
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
254+
ASSERT_EQ(expected_table_size * bucket_size - 1, builder.FileSize());
255255
ASSERT_OK(builder.Finish());
256256
ASSERT_OK(writable_file->Close());
257257
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
@@ -296,7 +296,7 @@ TEST(CuckooBuilderTest, WithCollisionPathFullKey) {
296296
ASSERT_OK(builder.status());
297297
}
298298
uint32_t bucket_size = keys[0].size() + values[0].size();
299-
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
299+
ASSERT_EQ(expected_table_size * bucket_size - 1, builder.FileSize());
300300
ASSERT_OK(builder.Finish());
301301
ASSERT_OK(writable_file->Close());
302302
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
@@ -338,7 +338,7 @@ TEST(CuckooBuilderTest, WithCollisionPathFullKeyAndCuckooBlock) {
338338
ASSERT_OK(builder.status());
339339
}
340340
uint32_t bucket_size = keys[0].size() + values[0].size();
341-
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
341+
ASSERT_EQ(expected_table_size * bucket_size - 1, builder.FileSize());
342342
ASSERT_OK(builder.Finish());
343343
ASSERT_OK(writable_file->Close());
344344
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
@@ -374,7 +374,7 @@ TEST(CuckooBuilderTest, WriteSuccessNoCollisionUserKey) {
374374
ASSERT_OK(builder.status());
375375
}
376376
uint32_t bucket_size = user_keys[0].size() + values[0].size();
377-
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
377+
ASSERT_EQ(expected_table_size * bucket_size - 1, builder.FileSize());
378378
ASSERT_OK(builder.Finish());
379379
ASSERT_OK(writable_file->Close());
380380
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
@@ -410,7 +410,7 @@ TEST(CuckooBuilderTest, WriteSuccessWithCollisionUserKey) {
410410
ASSERT_OK(builder.status());
411411
}
412412
uint32_t bucket_size = user_keys[0].size() + values[0].size();
413-
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
413+
ASSERT_EQ(expected_table_size * bucket_size - 1, builder.FileSize());
414414
ASSERT_OK(builder.Finish());
415415
ASSERT_OK(writable_file->Close());
416416
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
@@ -448,7 +448,7 @@ TEST(CuckooBuilderTest, WithCollisionPathUserKey) {
448448
ASSERT_OK(builder.status());
449449
}
450450
uint32_t bucket_size = user_keys[0].size() + values[0].size();
451-
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());
451+
ASSERT_EQ(expected_table_size * bucket_size - 1, builder.FileSize());
452452
ASSERT_OK(builder.Finish());
453453
ASSERT_OK(writable_file->Close());
454454
ASSERT_LE(expected_table_size * bucket_size, builder.FileSize());

0 commit comments

Comments
 (0)