Fix up tests: no more copying

Author: andreilitvin

src/app/server-cluster/StandardServerCluster.h

@@ -38,11 +38,12 @@ class StandardServerCluster : public ServerClusterInterface
     StandardServerCluster();
     virtual ~StandardServerCluster() = default;
 
-    StandardServerCluster(const StandardServerCluster & other)             = default;
     StandardServerCluster(StandardServerCluster && other)                  = default;
-    StandardServerCluster & operator=(const StandardServerCluster & other) = default;
     StandardServerCluster & operator=(StandardServerCluster && other)      = default;
 
+    StandardServerCluster(const StandardServerCluster & other)             = delete;
+    StandardServerCluster & operator=(const StandardServerCluster & other) = delete;
+
     void IncreaseDataVersion() { mDataVersion++; }
 
     //////////////////////////// ServerClusterInterface implementation ////////////////////////////////////////

src/app/server-cluster/tests/TestStandardServerCluster.cpp

@@ -73,21 +73,6 @@ TEST(TestStandardServerCluster, TestAssignmentNotInList)
 
     ASSERT_FALSE(basic.IsInList());
 
-    // copy constructor: not in list
-    {
-        FakeStandardServerCluster other(basic);
-        ASSERT_FALSE(other.IsInList());
-    }
-
-    // assignment : not in list
-    {
-        FakeStandardServerCluster other(2);
-        ASSERT_FALSE(other.IsInList());
-
-        other = basic;
-        ASSERT_FALSE(other.IsInList());
-    }
-
     // move constructor
     {
         FakeStandardServerCluster movable(2);
@@ -118,24 +103,6 @@ TEST(TestStandardServerCluster, TestAssignmentInList)
     basic.SetNextListItem(nullptr);
     ASSERT_TRUE(basic.IsInList());
 
-    // copy constructor: not in list
-    {
-        FakeStandardServerCluster other(basic);
-        ASSERT_TRUE(other.IsInList());
-        other.SetNotInList(); // ensure destruction is ok
-    }
-
-    // assignment : not in list
-    {
-        FakeStandardServerCluster other(2);
-        ASSERT_FALSE(other.IsInList());
-
-        other = basic;
-        ASSERT_TRUE(other.IsInList());
-
-        other.SetNotInList(); // ensure destruction is ok
-    }
-
     // move constructor
     {
         FakeStandardServerCluster movable(2);