Fix smoke test

Check gid of cache dir is system_share.

Change-Id: Ia09306b6bd71ea83a2f8744f7ab44bbe00424d4a
Signed-off-by: Sangyoon Jang <s89.jang@samsung.com>

diff --git a/src/unit_tests/smoke_test.cc b/src/unit_tests/smoke_test.cc
index 4077fa469c8948076b6c267f68583d65bdcf58cd..ca7e56c2014dee0425ac4047fee1690dab4b430f 100644 (file)
--- a/src/unit_tests/smoke_test.cc
+++ b/src/unit_tests/smoke_test.cc
@@ -164,7 +164,7 @@ bool ValidateFileContentInPackage(const std::string& pkgid,
   return content == expected;
 }
 
-void ValidatePackageRWFS(const std::string& pkgid, uid_t uid, gid_t gid) {
+void ValidatePackageRWFS(const std::string& pkgid, uid_t uid) {
   bf::path root_path = ci::GetRootAppPath(false, uid);
   bf::path package_path = root_path / pkgid;
   bf::path data_path = package_path / "data";
@@ -176,7 +176,7 @@ void ValidatePackageRWFS(const std::string& pkgid, uid_t uid, gid_t gid) {
 
   struct stat stats;
   stat(data_path.c_str(), &stats);
-  // gid of data, and shared/data should be system_share
+  // gid of RW dirs should be system_share
   boost::optional<gid_t> system_share =
       ci::GetGidByGroupName(kSystemShareGroupName);
   ASSERT_EQ(uid, stats.st_uid) << "Invalid gid: " << data_path;
@@ -190,7 +190,7 @@ void ValidatePackageRWFS(const std::string& pkgid, uid_t uid, gid_t gid) {
 
   stat(cache_path.c_str(), &stats);
   ASSERT_EQ(uid, stats.st_uid) << "Invalid gid: " << cache_path;
-  ASSERT_EQ(gid, stats.st_gid) << "Invalid gid: " << cache_path;
+  ASSERT_EQ(*system_share, stats.st_gid) << "Invalid gid: " << cache_path;
 }
 
 void ValidatePackageFS(const std::string& pkgid, const std::string& appid,
@@ -251,9 +251,9 @@ void ValidatePackage(const std::string& pkgid, const std::string& appid) {
   if (kTestUserId == kGlobalUserUid) {
     ci::UserList list = ci::GetUserList();
     for (auto& l : list)
-      ValidatePackageRWFS(pkgid, std::get<0>(l), std::get<1>(l));
+      ValidatePackageRWFS(pkgid, std::get<0>(l));
   } else {
-    ValidatePackageRWFS(pkgid, kTestUserId, kTestGroupId);
+    ValidatePackageRWFS(pkgid, kTestUserId);
   }
 }