From a2d86dee652a4e24a7282cfab28c8f6402e2f491 Mon Sep 17 00:00:00 2001
From: Marcin Niesluchowski <m.niesluchow@samsung.com>
Date: Tue, 3 Feb 2015 17:31:38 +0100
Subject: [PATCH] Tidy up TemporaryTestUser class

Make TemporaryTestUser class more readable and add gid entry.

Change-Id: I7a33aee2efc244a430c1fe98f83340d233480ef6
---
 tests/common/temp_test_user.cpp | 22 ++++++++++++++--------
 tests/common/temp_test_user.h   |  6 ++++--
 2 files changed, 18 insertions(+), 10 deletions(-)

diff --git a/tests/common/temp_test_user.cpp b/tests/common/temp_test_user.cpp
index 672bae4..c15c029 100644
--- a/tests/common/temp_test_user.cpp
+++ b/tests/common/temp_test_user.cpp
@@ -26,21 +26,26 @@
 #include <glib-object.h>
 #include <dpl/test/test_runner.h>
 
-TemporaryTestUser::TemporaryTestUser(std::string userName, GumUserType userType, bool offline) {
-    m_userName =  userName;
-    m_id = 0;
-    m_userType = userType;
+TemporaryTestUser::TemporaryTestUser(std::string userName, GumUserType userType, bool offline)
+    : m_uid(0)
+    , m_gid(0)
+    , m_userName(userName)
+    , m_userType(userType)
+{
     m_guser = gum_user_create_sync (offline);
     RUNNER_ASSERT_MSG(m_guser != nullptr, "Failed to create gumd user object");
     g_object_set(G_OBJECT(m_guser), "usertype", userType, NULL);
     g_object_set(G_OBJECT(m_guser), "username", userName.c_str(), NULL);
     gboolean added = gum_user_add_sync(m_guser);
     RUNNER_ASSERT_MSG(added, "Failed to add user");
-    g_object_get(G_OBJECT(m_guser), "uid", &m_id, NULL);
-    RUNNER_ASSERT_MSG(m_id != 0, "Something strange happened during user creation");
+    g_object_get(G_OBJECT(m_guser), "uid", &m_uid, NULL);
+    RUNNER_ASSERT_MSG(m_uid != 0, "Something strange happened during user creation. uid == 0.");
+    g_object_get(G_OBJECT(m_guser), "gid", &m_gid, NULL);
+    RUNNER_ASSERT_MSG(m_gid != 0, "Something strange happened during user creation. gid == 0.");
 }
 
-void TemporaryTestUser::remove(void) {
+void TemporaryTestUser::remove(void)
+{
     if(m_guser){
         gum_user_delete_sync(m_guser, TRUE);
         g_object_unref(m_guser);
@@ -48,6 +53,7 @@ void TemporaryTestUser::remove(void) {
     }
 }
 
-TemporaryTestUser::~TemporaryTestUser() {
+TemporaryTestUser::~TemporaryTestUser()
+{
     this->remove();
 }
diff --git a/tests/common/temp_test_user.h b/tests/common/temp_test_user.h
index 8f90bf3..f56c30a 100644
--- a/tests/common/temp_test_user.h
+++ b/tests/common/temp_test_user.h
@@ -27,11 +27,13 @@ public:
     TemporaryTestUser(std::string userName, GumUserType userType, bool offline);
     ~TemporaryTestUser();
     void remove(void);
-    uid_t getUid() const {return m_id;}
+    uid_t getUid() const {return m_uid;}
+    uid_t getGid() const {return m_gid;}
     const std::string& getUserName() const {return m_userName;}
     GumUserType getUserType() const {return m_userType;}
 private:
-    uid_t m_id;
+    uid_t m_uid;
+    uid_t m_gid;
     std::string m_userName;
     GumUserType m_userType;
     GumUser *m_guser;
-- 
2.7.4