Commit 9f6e73e4 authored by Vasilii Sukhanov's avatar Vasilii Sukhanov

Fix PasswordStoreTest.RemoveLoginsCreatedBetweenCallbackIsCalled

The test is flaky on Android
[ RUN      ]
PasswordStoreTest.RemoveLoginsCreatedBetweenCallbackIsCalled
[FATAL:scoped_task_environment.cc(192)] Check failed:
!queue_empty_closure_.

Bug: 355145
Change-Id: I53a73974d4eed6d8b6c895e4ec5515547449b231
Reviewed-on: https://chromium-review.googlesource.com/602149Reviewed-by: default avatarVadym Doroshenko <dvadym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#492003}
parent 91d01230
...@@ -331,10 +331,11 @@ TEST_F(PasswordStoreTest, RemoveLoginsCreatedBetweenCallbackIsCalled) { ...@@ -331,10 +331,11 @@ TEST_F(PasswordStoreTest, RemoveLoginsCreatedBetweenCallbackIsCalled) {
store->AddObserver(&mock_observer); store->AddObserver(&mock_observer);
EXPECT_CALL(mock_observer, OnLoginsChanged(testing::SizeIs(1u))); EXPECT_CALL(mock_observer, OnLoginsChanged(testing::SizeIs(1u)));
store->RemoveLoginsCreatedBetween( base::RunLoop run_loop;
base::Time::FromDoubleT(0), base::Time::FromDoubleT(2), store->RemoveLoginsCreatedBetween(base::Time::FromDoubleT(0),
base::MessageLoop::current()->QuitWhenIdleClosure()); base::Time::FromDoubleT(2),
WaitForPasswordStore(); run_loop.QuitClosure());
run_loop.Run();
testing::Mock::VerifyAndClearExpectations(&mock_observer); testing::Mock::VerifyAndClearExpectations(&mock_observer);
store->RemoveObserver(&mock_observer); store->RemoveObserver(&mock_observer);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment