STEP_NAME marco

Adding marco to print name of step that failed
in AppInstaller class

Following must be submitted together:
 - https://review.tizen.org/gerrit/72827
 - https://review.tizen.org/gerrit/72828
 - https://review.tizen.org/gerrit/72829

Change-Id: I96af51d6c924669ace683338097c9b8c3d3b40eb

diff --git a/src/tpk/step/configuration/step_parse_preload.h b/src/tpk/step/configuration/step_parse_preload.h
index d0e02b5..51fe4a3 100644 (file)
--- a/src/tpk/step/configuration/step_parse_preload.h
+++ b/src/tpk/step/configuration/step_parse_preload.h
@@ -22,7 +22,7 @@ class StepParsePreload : public common_installer::Step {
   Status undo() override { return Status::OK; }
   Status precheck() override { return Status::OK; }
 
-  SCOPE_LOG_TAG(ParsePreload)
+  STEP_NAME(ParsePreload)
 };
 
 }  // namespace configuration

diff --git a/src/tpk/step/filesystem/step_check_pkg_directory_path.h b/src/tpk/step/filesystem/step_check_pkg_directory_path.h
index 98453ef..39b53e9 100644 (file)
--- a/src/tpk/step/filesystem/step_check_pkg_directory_path.h
+++ b/src/tpk/step/filesystem/step_check_pkg_directory_path.h
@@ -22,7 +22,7 @@ class StepCheckPkgDirPath : public common_installer::Step {
   Status undo() override { return Status::OK; }
   Status precheck() override { return Status::OK; }
 
-  SCOPE_LOG_TAG(CheckPkgDirPath)
+  STEP_NAME(CheckPkgDirPath)
 };
 
 }  // namespace filesystem

diff --git a/src/tpk/step/filesystem/step_create_symbolic_link.h b/src/tpk/step/filesystem/step_create_symbolic_link.h
index 8b9a6ac..5a7f57c 100644 (file)
--- a/src/tpk/step/filesystem/step_create_symbolic_link.h
+++ b/src/tpk/step/filesystem/step_create_symbolic_link.h
@@ -18,7 +18,7 @@ class StepCreateSymbolicLink : public common_installer::Step {
   Status undo() override { return Status::OK; }
   Status precheck() override;
 
-  SCOPE_LOG_TAG(SymbolicLink)
+  STEP_NAME(SymbolicLink)
 };
 
 }  // namespace filesystem

diff --git a/src/tpk/step/filesystem/step_grant_permission.h b/src/tpk/step/filesystem/step_grant_permission.h
index b6776cd..c387437 100644 (file)
--- a/src/tpk/step/filesystem/step_grant_permission.h
+++ b/src/tpk/step/filesystem/step_grant_permission.h
@@ -27,7 +27,7 @@ class StepTpkGrantPermission : public common_installer::Step {
   Status clean() override { return Status::OK; }
   Status precheck() override;
 
-  SCOPE_LOG_TAG(GrantPermission)
+  STEP_NAME(GrantPermission)
 };
 
 }  // namespace filesystem

diff --git a/src/tpk/step/filesystem/step_tpk_patch_icons.h b/src/tpk/step/filesystem/step_tpk_patch_icons.h
index a3806b1..1458cbf 100644 (file)
--- a/src/tpk/step/filesystem/step_tpk_patch_icons.h
+++ b/src/tpk/step/filesystem/step_tpk_patch_icons.h
@@ -34,7 +34,7 @@ class StepTpkPatchIcons : public common_installer::Step {
       application_x* app, icon_x* icon);
   Status FixIconLocation(const boost::filesystem::path& icon_text);
 
-  SCOPE_LOG_TAG(TpkPatchIcons)
+  STEP_NAME(TpkPatchIcons)
 };
 
 }  // namespace filesystem

diff --git a/src/tpk/step/filesystem/step_tpk_prepare_package_directory.h b/src/tpk/step/filesystem/step_tpk_prepare_package_directory.h
index 3437666..5979820 100644 (file)
--- a/src/tpk/step/filesystem/step_tpk_prepare_package_directory.h
+++ b/src/tpk/step/filesystem/step_tpk_prepare_package_directory.h
@@ -54,7 +54,7 @@ class StepTpkPreparePackageDirectory : public common_installer::Step {
   Status PrepareLink(const std::string& entry,
                      const boost::filesystem::path& mount_point);
 
-  SCOPE_LOG_TAG(TpkPreparePackageDirectory)
+  STEP_NAME(TpkPreparePackageDirectory)
 };
 
 }  // namespace filesystem

diff --git a/src/tpk/step/filesystem/step_tpk_update_package_directory.h b/src/tpk/step/filesystem/step_tpk_update_package_directory.h
index a0fe215..0d48ce3 100644 (file)
--- a/src/tpk/step/filesystem/step_tpk_update_package_directory.h
+++ b/src/tpk/step/filesystem/step_tpk_update_package_directory.h
@@ -37,7 +37,7 @@ class StepTpkUpdatePackageDirectory : public StepTpkPreparePackageDirectory {
   Status RestoreDirectory(const std::string& entry,
                           const boost::filesystem::path& backup_path);
 
-  SCOPE_LOG_TAG(TpkUpdatePackageDirectory)
+  STEP_NAME(TpkUpdatePackageDirectory)
 };
 
 }  // namespace filesystem

diff --git a/src/tpk/step/pkgmgr/step_convert_xml.h b/src/tpk/step/pkgmgr/step_convert_xml.h
index 12332f6..f734c3a 100644 (file)
--- a/src/tpk/step/pkgmgr/step_convert_xml.h
+++ b/src/tpk/step/pkgmgr/step_convert_xml.h
@@ -28,7 +28,7 @@ class StepConvertXml : public common_installer::Step {
   boost::filesystem::path xml_path_;
   bool ConvertXml(xmlDocPtr doc);
 
-  SCOPE_LOG_TAG(ConvertXml)
+  STEP_NAME(ConvertXml)
 };
 
 }  // namespace pkgmgr

diff --git a/src/tpk/step/pkgmgr/step_manifest_adjustment.h b/src/tpk/step/pkgmgr/step_manifest_adjustment.h
index 9bac6ed..6d3986a 100644 (file)
--- a/src/tpk/step/pkgmgr/step_manifest_adjustment.h
+++ b/src/tpk/step/pkgmgr/step_manifest_adjustment.h
@@ -26,7 +26,7 @@ class StepManifestAdjustment : public common_installer::Step {
  private:
   boost::filesystem::path xml_path_;
 
-  SCOPE_LOG_TAG(StepManifestAdjustment)
+  STEP_NAME(StepManifestAdjustment)
 };
 
 }  // namespace pkgmgr

diff --git a/src/tpk/step/rds/step_tpk_rds_modify.h b/src/tpk/step/rds/step_tpk_rds_modify.h
index 9d8b5a2..cc1b404 100644 (file)
--- a/src/tpk/step/rds/step_tpk_rds_modify.h
+++ b/src/tpk/step/rds/step_tpk_rds_modify.h
@@ -26,6 +26,8 @@ class StepTpkRDSModify : public common_installer::rds::StepRDSModify {
 
  protected:
   std::string GetAppPath() override;
+
+  STEP_NAME(TpkRDSModify)
 };
 
 }  // namespace rds

diff --git a/src/unit_tests/smoke_test_helper.cc b/src/unit_tests/smoke_test_helper.cc
index f803ddd..0edb55b 100644 (file)
--- a/src/unit_tests/smoke_test_helper.cc
+++ b/src/unit_tests/smoke_test_helper.cc
@@ -26,6 +26,8 @@ class StepCrash : public ci::Step {
   ci::Step::Status clean() override { return ci::Step::Status::OK; }
   ci::Step::Status undo() override { return ci::Step::Status::OK; }
   ci::Step::Status precheck() override { return ci::Step::Status::OK; }
+
+  STEP_NAME(Crash)
 };
 
 }  // namespace