Drop TemporaryTestUser copy constructor

Change-Id: Ic0dddc554c809d7d4d46f49cfe51d42a4793b359

diff --git a/src/common/temp_test_user.h b/src/common/temp_test_user.h
index 3a3b6d1661c6bc7d61c15f22c6c642253af2caac..291d5068e73fd3d15bd229e7799cb4b1e7285af9 100644 (file)
--- a/src/common/temp_test_user.h
+++ b/src/common/temp_test_user.h
@@ -34,8 +34,7 @@ public:
             m_offline(offline),
             m_creatorPid(getpid())
             {};
-    // TODO - copy constructor should be deleted
-    TemporaryTestUser(const TemporaryTestUser &) = default;
+    TemporaryTestUser(const TemporaryTestUser &) = delete;
     TemporaryTestUser(TemporaryTestUser &&other);
     TemporaryTestUser& operator=(const TemporaryTestUser &) = delete;
     ~TemporaryTestUser();

diff --git a/src/security-manager-tests/test_cases.cpp b/src/security-manager-tests/test_cases.cpp
index f44dd9a2692e6ff4fd4384699f0ba7030b5dc682..204ed47221883858f622b3b63b1d49c3692e7ae2 100644 (file)
--- a/src/security-manager-tests/test_cases.cpp
+++ b/src/security-manager-tests/test_cases.cpp
@@ -470,9 +470,9 @@ RUNNER_TEST(security_manager_09_app_install_constraint_check)
         }
     };
 
-    std::vector<TemporaryTestUser> users = {
-            {"sm_test_09_user_name_0", GUM_USERTYPE_NORMAL, false},
-            {"sm_test_09_user_name_1", GUM_USERTYPE_NORMAL, false}
+    std::array<TemporaryTestUser, 2> users{
+        TemporaryTestUser{"sm_test_09_user_name_0", GUM_USERTYPE_NORMAL, false},
+        TemporaryTestUser{"sm_test_09_user_name_1", GUM_USERTYPE_NORMAL, false}
     };
 
     for(auto& gu : users)