tree 3257c4a3ce0ceca96386c92ca6868dfe41e3c9b9
parent 2269bb6c1aebbdad70004969187d4f28ab9de2d8
author Wade Guthrie <wdg@chromium.org> 1375138994 -0700
committer ChromeBot <chrome-bot@google.com> 1375299167 -0700

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>