Drop TemporaryTestUser copy constructor

Change-Id: Ic0dddc554c809d7d4d46f49cfe51d42a4793b359

diff --git a/src/common/temp_test_user.h b/src/common/temp_test_user.h
index 3a3b6d1..291d506 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 f44dd9a..204ed47 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)