[1924] prioritize a method where possible.

diff --git a/src/lib/cc/tests/session_unittests.cc b/src/lib/cc/tests/session_unittests.cc
index c218c76387a8267ee9fe2398dad45ddd55da20f4..1f66157ef78df8d91a12bb9b264cec419bcefa9a 100644 (file)
--- a/src/lib/cc/tests/session_unittests.cc
+++ b/src/lib/cc/tests/session_unittests.cc
@@ -173,15 +173,6 @@ protected:
         delete tds;
     }
 
-    // Check two elements are equal
-    void elementsEqual(const string& expected,
-                       const ConstElementPtr& actual) const
-    {
-        EXPECT_TRUE(Element::fromJSON(expected)->equals(*actual)) <<
-            "Elements differ, expected: " << expected <<
-            ", got: " << actual->toWire();
-    }
-
     // Check the session sent a message with the given header. The
     // message is hardcoded.
     void checkSentMessage(const string& expected_hdr, const char* description)
@@ -192,6 +183,16 @@ protected:
         elementsEqual("{\"test\": 42}", msg.second);
     }
 
+private:
+    // Check two elements are equal
+    void elementsEqual(const string& expected,
+                       const ConstElementPtr& actual) const
+    {
+        EXPECT_TRUE(Element::fromJSON(expected)->equals(*actual)) <<
+            "Elements differ, expected: " << expected <<
+            ", got: " << actual->toWire();
+    }
+
 public:
     // used in the handler test
     // This handler first reads (and ignores) whatever message caused