New issue
Advanced search Search tips

Issue 728731 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Switch some of v4_local_database_manager_unittest.cc to MOCKs

Project Member Reported by nparker@chromium.org, Jun 1 2017

Issue description

Any place we're using a fake to just set an expected value and then test that it got called, we could (I posit) reduce code complexity and increase coverage by using a mock.

Test my theory. :-)
 

Comment 1 by vakh@chromium.org, Jun 2 2017

Labels: SafeBrowsing-Triaged
Status: Assigned (was: Untriaged)

Comment 2 by est...@chromium.org, Nov 10 2017

Labels: Hotlist-EnamelAndFriendsFixIt

Comment 3 by est...@chromium.org, Feb 18 2018

Labels: -Hotlist-EnamelAndFriendsFixIt

Sign in to add a comment