commit | 9f4aa159da8d06e75095a1210cff94e39f2899e9 | [log] [tgz] |
---|---|---|
author | Wade Guthrie <wdg@chromium.org> | Mon Jul 29 16:03:14 2013 -0700 |
committer | ChromeBot <chrome-bot@google.com> | Wed Jul 31 12:32:47 2013 -0700 |
tree | 3257c4a3ce0ceca96386c92ca6868dfe41e3c9b9 | |
parent | 2269bb6c1aebbdad70004969187d4f28ab9de2d8 [diff] |
shill: Factors some common unittest code into own methods. A code review on a previous CL suggested refactoring the StartScan and AttemptConnection code from the unittests into their own methods. This does that. Found and fixed a tiny buglet where transitioning from scanning(progressive) to scanning(full) emitted an extra dbus signal(scanning=true). Also made VerifyScanState 'const'. BUG=None. TEST=unittest. Change-Id: Ia4d993dfd0ee425cba1a38fed2561e213f42e699 Reviewed-on: https://gerrit.chromium.org/gerrit/63712 Commit-Queue: Wade Guthrie <wdg@chromium.org> Reviewed-by: Wade Guthrie <wdg@chromium.org> Tested-by: Wade Guthrie <wdg@chromium.org>