commit 67ed78d1f0fcf22e26afc11b6cb597f3b04ffacf
author Gilad Arnold <garnold@chromium.org> Wed Apr 23 13:17:46 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Thu Apr 24 23:50:49 2014 +0000
tree e7c3e847a937c25f108e3ee84282ae30402c0ff2
parent 075248bffd57c4d2f13019ca75b6b63d7a0c9717

PM: Move Variable content testing logic into a common unittest module.

Testing the content of variables is done frequently when testing various
providers. This results in plenty of boilerplate code, as well as
numerous friend declarations in the different FooVariable class
definitions.

This CL introduces a PmTestUtils helper class providing common Variable
testing methods that test fixtures can use.

BUG=chromium:366259
TEST=Unit tests.

Change-Id: I7bb1cea080c3b79c203607550259f4c277e84f3b
Reviewed-on: https://chromium-review.googlesource.com/196529
Tested-by: Gilad Arnold <garnold@chromium.org>
Reviewed-by: Alex Deymo <deymo@chromium.org>
Commit-Queue: Gilad Arnold <garnold@chromium.org>

policy_manager/variable.h

diff --git a/policy_manager/variable.h b/policy_manager/variable.h
index e855e03..5b16b5d 100644
--- a/policy_manager/variable.h
+++ b/policy_manager/variable.h
@@ -115,7 +115,6 @@
 
  private:
   friend class PmEvaluationContextTest;
-  friend class PmBaseVariableTest;
   FRIEND_TEST(PmBaseVariableTest, RepeatedObserverTest);
   FRIEND_TEST(PmBaseVariableTest, NotifyValueChangedTest);
   FRIEND_TEST(PmBaseVariableTest, NotifyValueRemovesObserversTest);
@@ -174,25 +173,9 @@
   // directly from the variable.
   friend class EvaluationContext;
 
-  friend class PmRealConfigProviderTest;
-  friend class PmRealDevicePolicyProviderTest;
-  FRIEND_TEST(PmRealDevicePolicyProviderTest,
-              NonExistentDevicePolicyEmptyVariables);
-  friend class PmRealRandomProviderTest;
+  // Needed to be able to verify variable contents during unit testing.
+  friend class PmTestUtils;
   FRIEND_TEST(PmRealRandomProviderTest, GetRandomValues);
-  friend class PmRealShillProviderTest;
-  FRIEND_TEST(PmRealShillProviderTest, ReadBaseValues);
-  FRIEND_TEST(PmRealShillProviderTest, ReadConnTypeVpn);
-  FRIEND_TEST(PmRealShillProviderTest, ReadLastChangedTimeTwoSignals);
-  FRIEND_TEST(PmRealShillProviderTest, ConnTypeCacheUsed);
-  FRIEND_TEST(PmRealShillProviderTest, ConnTypeCacheRemainsValid);
-  FRIEND_TEST(PmRealShillProviderTest, ConnTetheringCacheUsed);
-  FRIEND_TEST(PmRealShillProviderTest, ConnTetheringCacheRemainsValid);
-  FRIEND_TEST(PmRealShillProviderTest, NoInitConnStatusReadBaseValues);
-  friend class PmRealTimeProviderTest;
-  FRIEND_TEST(PmRealTimeProviderTest, CurrDateValid);
-  FRIEND_TEST(PmRealTimeProviderTest, CurrHourValid);
-  friend class PmRealUpdaterProviderTest;
 
   Variable(const std::string& name, VariableMode mode)
       : BaseVariable(name, mode) {}