Chromium Code Reviews| Index: components/safe_browsing_db/v4_store_unittest.cc |
| diff --git a/components/safe_browsing_db/v4_store_unittest.cc b/components/safe_browsing_db/v4_store_unittest.cc |
| index c0138b7b379aa1da9a6ea74a7739ae9f05bea6b1..2d44b8e2b0e9316eb273cb09051e8bcf5b4ea552 100644 |
| --- a/components/safe_browsing_db/v4_store_unittest.cc |
| +++ b/components/safe_browsing_db/v4_store_unittest.cc |
| @@ -135,6 +135,7 @@ TEST_F(V4StoreTest, TestReadFromNoHashPrefixesFile) { |
| V4Store store(task_runner_, store_path_); |
| EXPECT_EQ(READ_SUCCESS, store.ReadFromDisk()); |
| EXPECT_TRUE(store.hash_prefix_map_.empty()); |
| + EXPECT_EQ(14l, store.file_size_); |
|
Nathan Parker
2016/10/25 19:01:44
Is there a case where it should eq 0, like on a fa
vakh (use Gerrit instead)
2016/10/25 19:12:34
Done.
|
| } |
| TEST_F(V4StoreTest, TestAddUnlumpedHashesWithInvalidAddition) { |
| @@ -562,6 +563,7 @@ TEST_F(V4StoreTest, TestReadFullResponseWithValidHashPrefixMap) { |
| ASSERT_EQ(2u, read_store.hash_prefix_map_.size()); |
| EXPECT_EQ("00000abc", read_store.hash_prefix_map_[4]); |
| EXPECT_EQ("00000abcde", read_store.hash_prefix_map_[5]); |
| + EXPECT_EQ(71l, read_store.file_size_); |
| } |
| // This tests fails to read the prefix map from the disk because the file on |
| @@ -783,6 +785,7 @@ TEST_F(V4StoreTest, TestChecksumErrorOnStartup) { |
| EXPECT_TRUE(store.expected_checksum_.empty()); |
| EXPECT_EQ(READ_SUCCESS, store.ReadFromDisk()); |
| EXPECT_TRUE(!store.expected_checksum_.empty()); |
| + EXPECT_EQ(69l, store.file_size_); |
| EXPECT_EQ("test_client_state", store.state()); |
| EXPECT_FALSE(store.VerifyChecksum()); |
| @@ -811,6 +814,7 @@ TEST_F(V4StoreTest, TestChecksumErrorOnStartup) { |
| EXPECT_EQ(READ_SUCCESS, another_store.ReadFromDisk()); |
| EXPECT_TRUE(!another_store.expected_checksum_.empty()); |
| EXPECT_EQ("test_client_state", another_store.state()); |
| + EXPECT_EQ(69l, store.file_size_); |
| EXPECT_TRUE(another_store.VerifyChecksum()); |
| } |