From fb3c88de168ba53a7c4264dad74443005e80b9ba Mon Sep 17 00:00:00 2001 From: chensi10 Date: Thu, 9 Sep 2021 20:42:47 +0800 Subject: [PATCH] fix appspawn codex Signed-off-by: chensi10 --- README.md | 37 ++++++++ include/appspawn_server.h | 2 - src/appspawn_server.cpp | 18 ++-- src/socket/appspawn_socket.cpp | 2 +- test/mock/include/main_thread.h | 4 +- test/mock/include/mock_client_socket.h | 2 - test/mock/include/mock_server_socket.h | 1 - test/mock/src/main_thread.cpp | 1 - test/mock/src/server_socket.cpp | 1 - test/moduletest/BUILD.gn | 2 +- ..._mst_test.cpp => appspawn_module_test.cpp} | 87 ++++++++++--------- .../app_spawn_server_mock_test.cpp | 10 +-- .../app_spawn_server_override_test.cpp | 14 ++- 13 files changed, 101 insertions(+), 80 deletions(-) create mode 100644 README.md rename test/moduletest/{appspawn_mst_test.cpp => appspawn_module_test.cpp} (89%) diff --git a/README.md b/README.md new file mode 100644 index 00000000..3f36b4e2 --- /dev/null +++ b/README.md @@ -0,0 +1,37 @@ +# Startup + +- [Introduction](#section11660541593) +- [Directory Structure](#section161941989596) +- [Repositories Involved](#section1371113476307) + +## Introduction + +Appspawn is responsible for creating application process and setting process information function. + +## Directory Structure + +``` +base/startup/appspawn_standard +├── include # include directory +├── parameter # system parameters +├── src # source code +│ └── socket # encapsulation of socket basic library +└── test # test code +``` + +## Repositories Involved + +Startup subsystem + +hmf/startup/syspara\_lite + +**hmf/startup/appspawn_standard** + +hmf/startup/appspawn\_lite + +hmf/startup/bootstrap\_lite + +hmf/startup/startup + +hmf/startup/systemrestore + diff --git a/include/appspawn_server.h b/include/appspawn_server.h index f914d45e..fab78ad6 100644 --- a/include/appspawn_server.h +++ b/include/appspawn_server.h @@ -25,7 +25,6 @@ namespace OHOS { namespace AppSpawn { - class AppSpawnServer { public: /** @@ -140,7 +139,6 @@ private: std::function errHandlerHook_ = nullptr; bool isRunning_{}; }; - } // namespace AppSpawn } // namespace OHOS diff --git a/src/appspawn_server.cpp b/src/appspawn_server.cpp index 8dd64868..28e4d119 100644 --- a/src/appspawn_server.cpp +++ b/src/appspawn_server.cpp @@ -32,7 +32,6 @@ namespace OHOS { namespace AppSpawn { - namespace { constexpr int32_t ERR_PIPE_FAIL = -100; constexpr int32_t MAX_LEN_SHORT_NAME = 16; @@ -51,7 +50,7 @@ static constexpr HiLogLabel LABEL = {LOG_CORE, 0, "AppSpawnServer"}; extern "C" { #endif -static void SignalHandler(int) /* signal_number */ +static void SignalHandler([[maybe_unused]] int sig) { pid_t pid; int status; @@ -205,16 +204,13 @@ bool AppSpawnServer::ServerMain(char *longProcName, int64_t longProcNameLen) if (pid == 0) { // special handle bundle name "com.ohos.photos" and "com.ohos.camera" - if ((strcmp(appProperty->processName, BUNDLE_NAME_CAMERA.data()) == 0) || - (strcmp(appProperty->processName, BUNDLE_NAME_PHOTOS.data()) == 0)) { - if (appProperty->gidCount < MAX_GIDS) { - appProperty->gidTable[appProperty->gidCount] = GID_MEDIA; - appProperty->gidCount++; - } else { - HiLog::Info(LABEL, "gidCount out of bounds !"); - } + if (((strcmp(appProperty->processName, BUNDLE_NAME_CAMERA.data()) == 0) || + (strcmp(appProperty->processName, BUNDLE_NAME_PHOTOS.data()) == 0)) && + appProperty->gidCount < MAX_GIDS) { + appProperty->gidTable[appProperty->gidCount] = GID_MEDIA; + appProperty->gidCount++; + HiLog::Info(LABEL, "Special handle gid groups!"); } - return SetAppProcProperty(connectFd, appProperty, longProcName, longProcNameLen, fd); } // parent process diff --git a/src/socket/appspawn_socket.cpp b/src/socket/appspawn_socket.cpp index 2e11106f..1eb2362b 100755 --- a/src/socket/appspawn_socket.cpp +++ b/src/socket/appspawn_socket.cpp @@ -130,7 +130,7 @@ int AppSpawnSocket::WriteSocketMessage(int socketFd, const void *buf, int len) ssize_t written = 0; ssize_t remain = static_cast(len); const uint8_t *offset = reinterpret_cast(buf); - for (ssize_t wLen; remain > 0; offset += wLen, remain -= wLen, written += wLen) { + for (ssize_t wLen = 0; remain > 0; offset += wLen, remain -= wLen, written += wLen) { wLen = write(socketFd, offset, remain); HiLog::Debug(LABEL, "socket fd %d, wLen %zd", socketFd, wLen); if ((wLen <= 0) && (errno != EINTR)) { diff --git a/test/mock/include/main_thread.h b/test/mock/include/main_thread.h index 6c503be0..1ebc490e 100644 --- a/test/mock/include/main_thread.h +++ b/test/mock/include/main_thread.h @@ -18,9 +18,7 @@ namespace OHOS { namespace AppExecFwk { - -class MainThread { - +class MainThread { public: MainThread() = default; virtual ~MainThread() = default; diff --git a/test/mock/include/mock_client_socket.h b/test/mock/include/mock_client_socket.h index 65908756..0a9bb3a1 100644 --- a/test/mock/include/mock_client_socket.h +++ b/test/mock/include/mock_client_socket.h @@ -18,7 +18,6 @@ namespace OHOS { namespace AppSpawn { - class MockClientSocket : public ClientSocket { public: virtual ~MockClientSocket() = default; @@ -31,6 +30,5 @@ public: MOCK_METHOD2(WriteSocketMessage, int32_t(const void *buf, int32_t len)); MOCK_METHOD2(ReadSocketMessage, int32_t(void *buf, int32_t len)); }; - } // namespace AppSpawn } // namespace OHOS diff --git a/test/mock/include/mock_server_socket.h b/test/mock/include/mock_server_socket.h index 5e167841..3cdbf36f 100644 --- a/test/mock/include/mock_server_socket.h +++ b/test/mock/include/mock_server_socket.h @@ -164,6 +164,5 @@ public: private: std::mutex mutex_; }; - } // namespace AppSpawn } // namespace OHOS diff --git a/test/mock/src/main_thread.cpp b/test/mock/src/main_thread.cpp index 8ca7af85..da611e2f 100644 --- a/test/mock/src/main_thread.cpp +++ b/test/mock/src/main_thread.cpp @@ -25,6 +25,5 @@ void MainThread::Start() { HiLog::Info(LABEL, "Start"); } - } // namespace AppExecFwk } // namespace OHOS diff --git a/test/mock/src/server_socket.cpp b/test/mock/src/server_socket.cpp index 4f443907..2ffd30e4 100644 --- a/test/mock/src/server_socket.cpp +++ b/test/mock/src/server_socket.cpp @@ -89,6 +89,5 @@ int ServerSocket::WaitForConnection() HiLog::Info(LABEL, "WaitForConnection"); return 0; } - } // namespace AppSpawn } // namespace OHOS diff --git a/test/moduletest/BUILD.gn b/test/moduletest/BUILD.gn index f2356871..6a8bd078 100644 --- a/test/moduletest/BUILD.gn +++ b/test/moduletest/BUILD.gn @@ -17,7 +17,7 @@ import("//build/test.gni") ohos_moduletest("AppSpawnModuleTest") { module_out_path = "${module_output_path}" - sources = [ "${appspawn_path}/test/moduletest/appspawn_mst_test.cpp" ] + sources = [ "${appspawn_path}/test/moduletest/appspawn_module_test.cpp" ] include_dirs = [ "//utils/native/base/include", diff --git a/test/moduletest/appspawn_mst_test.cpp b/test/moduletest/appspawn_module_test.cpp similarity index 89% rename from test/moduletest/appspawn_mst_test.cpp rename to test/moduletest/appspawn_module_test.cpp index 78a665e7..d6cb5e6c 100644 --- a/test/moduletest/appspawn_mst_test.cpp +++ b/test/moduletest/appspawn_module_test.cpp @@ -13,21 +13,22 @@ * limitations under the License. */ -#include "gtest/gtest.h" -#include "hilog/log.h" -#include -#include #include - -#include "securec.h" +#include +#include "cstdio" #include "app_spawn_client.h" +#include "gtest/gtest.h" +#include "hilog/log.h" +#include "securec.h" using namespace testing::ext; using namespace OHOS; using namespace OHOS::AppExecFwk; using namespace OHOS::HiviewDFX; +namespace OHOS { +namespace AppSpawn { static constexpr HiLogLabel LABEL = {LOG_CORE, 0, "AppSpawnMST"}; namespace { @@ -49,7 +50,6 @@ const char *DELIMITER_NEWLINE = "\n"; char buffer[BUFFER_SIZE]; int32_t newPid = 0; int32_t retryCount = 0; - } // namespace bool checkFileIsExists(const char *filepath) @@ -152,7 +152,7 @@ std::size_t getGids(const int32_t &pid, std::vector &gids) groupsPtr = groupsPtr + GROUPS_POSITION_MOVE; } // Get the row content of Groups - char *saveptr = NULL; + char *saveptr = nullptr; char *line = strtok_r(groupsPtr, DELIMITER_NEWLINE, &saveptr); if (line == nullptr || strlen(line) > BUFFER_SIZE) { HiLog::Error(LABEL, "get Groups line info failed."); @@ -171,7 +171,6 @@ std::size_t getGids(const int32_t &pid, std::vector &gids) bool checkGids(const int32_t &pid, const AppSpawnStartMsg ¶ms) { - // Get Gids std::vector gids; std::size_t gCount = getGids(pid, gids); @@ -184,7 +183,6 @@ bool checkGids(const int32_t &pid, const AppSpawnStartMsg ¶ms) bool checkGidsCount(const int32_t &pid, const AppSpawnStartMsg ¶ms) { - // Get GidsCount std::vector gids; std::size_t gCount = getGids(pid, gids); @@ -224,11 +222,9 @@ bool checkProcName(const int32_t &pid, const AppSpawnStartMsg ¶ms) return CHECK_OK; } HiLog::Error(LABEL, " procName=%{public}s, params.procName=%{public}s.", procName, params.procName.c_str()); - } else { HiLog::Error(LABEL, "Getting procName failed."); } - pclose(fp); return CHECK_ERROR; @@ -323,7 +319,10 @@ void AppSpawnModuleTest::TearDownTestCase() void AppSpawnModuleTest::SetUp() { newPid = 0; - EXPECT_EQ(memset_s(buffer, sizeof(buffer), 0x00, BUFFER_SIZE), EOK); + auto ret = memset_s(buffer, sizeof(buffer), 0x00, BUFFER_SIZE); + if (ret != EOK) { + HiLog::Error(LABEL, "memset_s is failed."); + } } void AppSpawnModuleTest::TearDown() @@ -359,8 +358,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_listen_001, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_listen_002, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_listen_002 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); @@ -384,8 +383,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_listen_002, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_fork_001, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_fork_001 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-fork_001", "soPath"}; @@ -414,8 +413,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_fork_001, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_fork_002, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_fork_002 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-fork_002", "soPath"}; @@ -444,8 +443,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_fork_002, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_001, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setUid_001 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setUid_001", "soPath"}; @@ -476,8 +475,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_001, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_002, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setUid_002 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setUid_002", "soPath"}; @@ -508,8 +507,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_002, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_003, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setUid_003 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setUid_003", "soPath"}; @@ -540,8 +539,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_003, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_004, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setUid_004 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setUid_004", "soPath"}; @@ -572,8 +571,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_004, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_005, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setUid_005 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setUid_005", "soPath"}; @@ -604,8 +603,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_005, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_006, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setUid_006 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setUid_006", "soPath"}; @@ -636,8 +635,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_006, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_007, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setUid_007 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setUid_007", "soPath"}; @@ -668,8 +667,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_007, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_008, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setUid_008 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setUid_008", "soPath"}; @@ -700,8 +699,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setUid_008, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setProcName_001, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setProcName_001 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setProcName_001", "soPath"}; @@ -733,8 +732,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setProcName_001, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setProcName_002, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_setProcName_002 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-setProcName_002", "soPath"}; @@ -764,8 +763,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_setProcName_002, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_recycleProc_001, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_recycleProc_001 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-recycleProc_001", "soPath"}; @@ -798,8 +797,8 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_recycleProc_001, TestSize.Level0) HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_recycleProc_002, TestSize.Level0) { HiLog::Info(LABEL, "AppSpawn_HF_recycleProc_002 start"); - std::shared_ptr appSpawnClient = std::make_shared(); - std::shared_ptr appSpawnSocket = std::make_shared(); + std::unique_ptr appSpawnClient = std::make_unique(); + std::shared_ptr appSpawnSocket = std::make_shared(); appSpawnClient->SetSocket(appSpawnSocket); EXPECT_EQ(ERR_OK, appSpawnClient->OpenConnection()); AppSpawnStartMsg params = {10003, 10004, {10003, 10004}, "processName-recycleProc_002", "soPath"}; @@ -820,3 +819,5 @@ HWTEST_F(AppSpawnModuleTest, AppSpawn_HF_recycleProc_002, TestSize.Level0) EXPECT_EQ(SpawnConnectionState::STATE_NOT_CONNECT, appSpawnClient->QueryConnectionState()); HiLog::Info(LABEL, "AppSpawn_HF_recycleProc_002 end"); } +} // namespace AppSpawn +} // namespace OHOS diff --git a/test/unittest/app_spawn_server_test/app_spawn_server_mock_test.cpp b/test/unittest/app_spawn_server_test/app_spawn_server_mock_test.cpp index 2827cfff..ebe99620 100644 --- a/test/unittest/app_spawn_server_test/app_spawn_server_mock_test.cpp +++ b/test/unittest/app_spawn_server_test/app_spawn_server_mock_test.cpp @@ -40,17 +40,15 @@ public: public: static constexpr int TEST_WAIT_TIME = 50 * 1000; // 50 ms protected: - std::unique_ptr appSpawnServer_; - std::shared_ptr mockServerSocket_; + std::unique_ptr appSpawnServer_ = nullptr; + std::shared_ptr mockServerSocket_ = nullptr; }; void AppSpawnServerMockTest::SetUpTestCase() -{ -} +{} void AppSpawnServerMockTest::TearDownTestCase() -{ -} +{} void AppSpawnServerMockTest::SetUp() { diff --git a/test/unittest/app_spawn_server_test/app_spawn_server_override_test.cpp b/test/unittest/app_spawn_server_test/app_spawn_server_override_test.cpp index f0f4ffc3..7deb9969 100644 --- a/test/unittest/app_spawn_server_test/app_spawn_server_override_test.cpp +++ b/test/unittest/app_spawn_server_test/app_spawn_server_override_test.cpp @@ -38,16 +38,14 @@ public: public: static constexpr int TEST_WAIT_TIME = 50 * 1000; // 50 ms protected: - std::unique_ptr appSpawnServer_; + std::unique_ptr appSpawnServer_ = nullptr; }; void AppSpawnServerOverrideTest::SetUpTestCase() -{ -} +{} void AppSpawnServerOverrideTest::TearDownTestCase() -{ -} +{} void AppSpawnServerOverrideTest::SetUp() { @@ -98,11 +96,11 @@ HWTEST_F(AppSpawnServerOverrideTest, App_Spawn_Server_Override_002, TestSize.Lev { GTEST_LOG_(INFO) << "App_Spawn_Server_Override_002 start"; - char* longProcName = nullptr; + char *longProcName = nullptr; int64_t longProcNameLen = sizeof(longProcName); char processName[16] = "LongNameTest"; int32_t len = sizeof(processName); - + EXPECT_EQ(-EINVAL, appSpawnServer_->SetProcessName(longProcName, longProcNameLen, processName, len)); GTEST_LOG_(INFO) << "App_Spawn_Server_Override_002 end"; @@ -122,7 +120,7 @@ HWTEST_F(AppSpawnServerOverrideTest, App_Spawn_Server_Override_003, TestSize.Lev char longProcName[20] = "longProcName"; int64_t longProcNameLen = sizeof(longProcName); - char* processName = nullptr; + char *processName = nullptr; int32_t len = sizeof(processName); EXPECT_EQ(-EINVAL, appSpawnServer_->SetProcessName(longProcName, longProcNameLen, processName, len)); -- Gitee