diff options
author | Paul Stewart <pstew@chromium.org> | 2014-10-15 21:26:44 -0700 |
---|---|---|
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-10-30 09:57:10 +0000 |
commit | 22ce765226cfdcf267b7dc80c22892e6b29065ea (patch) | |
tree | d33286f3c6621828fc5a924bef54216d349fe75b /mock_service.h | |
parent | 68b73d232e8a03ed55401f36ea1a72a2b69cf912 (diff) | |
download | shill-22ce765226cfdcf267b7dc80c22892e6b29065ea.tar.gz |
shill: Service: Move ManagerTest::SortServices to ServiceTest::Compare
We have long considered the SortServices test to be misplaced.
Finally move this to the Service unit tests, since after all,
this is testing the Service::Compare static method.
While here, mock EnableAndRetainAutoConnect() in MockService since
the move of SortServices was blocking this change.
BUG=chromium:206367
TEST=Unit tests
Change-Id: I4ec7d5a098edfb938d1ffd0ae78116f8004646c9
Reviewed-on: https://chromium-review.googlesource.com/223615
Reviewed-by: Paul Stewart <pstew@chromium.org>
Commit-Queue: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>
Diffstat (limited to 'mock_service.h')
-rw-r--r-- | mock_service.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/mock_service.h b/mock_service.h index 0df41b3c..2afae160 100644 --- a/mock_service.h +++ b/mock_service.h @@ -72,6 +72,7 @@ class MockService : public Service { MOCK_CONST_METHOD0(is_dns_auto_fallback_allowed, bool()); MOCK_METHOD0(NotifyIPConfigChanges, void()); MOCK_CONST_METHOD0(link_monitor_disabled, bool()); + MOCK_METHOD0(EnableAndRetainAutoConnect, void()); // Set a string for this Service via |store|. Can be wired to Save() for // test purposes. |