PM: RealShillProvider starts even when shill is not available.
The RealShillProvider used to query shill for the default connection
status during its initialization, failing the latter if this query did
not go through. However, we would like this provider to initialize
properly even if shill is not responding (e.g. hasn't started yet). This
change relies on the assumption that, when shill comes up and detects
a connection, it will signal the detected properties via DBus and so
there's no need to reattempt querying.
That said, the RealShillProvider's variables need to be aware of the
possibility that their underlying connection values are not available,
and respond accordingly to GetValue queries. In doing so, we extend the
generic CopyVariable class to also accept a flag (Boolean), indicating
whether the underlying value object is set and available for copying.
BUG=chromium:356949
TEST=Unit tests.
Change-Id: Ibe29b6cd1fb8dce2e227418c721dbc47a75763be
Reviewed-on: https://chromium-review.googlesource.com/193748
Tested-by: Gilad Arnold <garnold@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Reviewed-by: Alex Deymo <deymo@chromium.org>
Commit-Queue: Gilad Arnold <garnold@chromium.org>
diff --git a/policy_manager/variable.h b/policy_manager/variable.h
index da8eee9..2ef0fe1 100644
--- a/policy_manager/variable.h
+++ b/policy_manager/variable.h
@@ -170,10 +170,10 @@
friend class PmRealRandomProviderTest;
FRIEND_TEST(PmRealRandomProviderTest, GetRandomValues);
friend class PmRealShillProviderTest;
- FRIEND_TEST(PmRealShillProviderTest, ReadDefaultValues);
- FRIEND_TEST(PmRealShillProviderTest, ReadChangedValuesConnectedViaEthernet);
+ FRIEND_TEST(PmRealShillProviderTest, ReadBaseValues);
FRIEND_TEST(PmRealShillProviderTest, ReadChangedValuesConnectedViaVpn);
FRIEND_TEST(PmRealShillProviderTest, ReadChangedValuesConnectedTwoSignals);
+ FRIEND_TEST(PmRealShillProviderTest, NoInitConnStatusReadBaseValues);
friend class PmRealTimeProviderTest;
FRIEND_TEST(PmRealTimeProviderTest, CurrDateValid);
FRIEND_TEST(PmRealTimeProviderTest, CurrHourValid);