Merge pull request #1089 from brave/failing-tests

Fixes failing tests
brave · Dec 13, 2018 · bfa3c54 · bfa3c54
2 parents 38ba23c + 95b797c
commit bfa3c54
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/components/brave_rewards/browser/rewards_service_impl_unittest.cc b/components/brave_rewards/browser/rewards_service_impl_unittest.cc
@@ -157,9 +157,9 @@ TEST_F(RewardsServiceTest, HandleFlags) {
 }
 
 TEST_F(RewardsServiceTest, OnWalletProperties) {
-  // wallet properties are empty (no call should be made)
-  EXPECT_CALL(*observer(), OnWalletProperties(_, _, _)).Times(0);
-  rewards_service()->OnWalletProperties(ledger::Result::LEDGER_OK, nullptr);
+  // We always need to call observer as we report errors back even when we have null pointer
+  EXPECT_CALL(*observer(), OnWalletProperties(_, 1, _)).Times(1);
+  rewards_service()->OnWalletProperties(ledger::Result::LEDGER_ERROR, nullptr);
 }
 
 // add test for strange entries