diff --git a/frameworks/kits/ability/ability_runtime/test/mock/AMS/mock_serviceability_manager_service.cpp b/frameworks/kits/ability/ability_runtime/test/mock/AMS/mock_serviceability_manager_service.cpp index 89d93be8c6ac445c2ab9bdc29128ceda793ecc7a..c688eafdc5e621710d44a6823af55e9770e6a7ba 100644 --- a/frameworks/kits/ability/ability_runtime/test/mock/AMS/mock_serviceability_manager_service.cpp +++ b/frameworks/kits/ability/ability_runtime/test/mock/AMS/mock_serviceability_manager_service.cpp @@ -143,11 +143,6 @@ int MockServiceAbilityManagerService::RemoveMission(int id) return 0; } -int MockServiceAbilityManagerService::RemoveStack(int id) -{ - return 0; -} - int MockServiceAbilityManagerService::MoveMissionToEnd(const sptr &token, const bool nonFirst) { return 0; diff --git a/frameworks/kits/ability/ability_runtime/test/mock/AMS/mock_serviceability_manager_service.h b/frameworks/kits/ability/ability_runtime/test/mock/AMS/mock_serviceability_manager_service.h index 79805a4cc33f521a105eaa738c0cf09237891cbf..1a62e61740536f4933e6557e04c7fc7c171e1cc7 100644 --- a/frameworks/kits/ability/ability_runtime/test/mock/AMS/mock_serviceability_manager_service.h +++ b/frameworks/kits/ability/ability_runtime/test/mock/AMS/mock_serviceability_manager_service.h @@ -149,7 +149,6 @@ public: int RemoveMission(int id) override; - int RemoveStack(int id) override; int PowerOff() override; int PowerOn() override; int LockMission(int missionId) override diff --git a/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_client_interface1.cpp b/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_client_interface1.cpp index 9cd20832d354ec60e57e339b5e9a9e88a27b4d80..e934cf1b242a388bc4fd11f6308baa955208452f 100644 --- a/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_client_interface1.cpp +++ b/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_client_interface1.cpp @@ -180,11 +180,6 @@ ErrCode AbilityManagerClient::RemoveMissions(std::vector missionId) return ret; } -ErrCode AbilityManagerClient::RemoveStack(int id) -{ - return ERR_OK; -} - ErrCode AbilityManagerClient::KillProcess(const std::string &bundleName) { return ERR_OK; diff --git a/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_service.cpp b/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_service.cpp index d40a91e86cd9e518b776c90202d60aff8c80f165..6bdf4f912f778b2986a7b67bace9df4d24e54ec6 100644 --- a/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_service.cpp +++ b/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_service.cpp @@ -156,11 +156,6 @@ int MockAbilityManagerService::RemoveMission(int id) return 0; } -int MockAbilityManagerService::RemoveStack(int id) -{ - return 0; -} - int MockAbilityManagerService::MoveMissionToTop(int32_t missionId) { return 0; diff --git a/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_service.h b/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_service.h index b52b1b1867e26ec199236cc41342f0a7b9f91f1a..9aadc05adacabfd51a19df731cad35309e1a2715 100644 --- a/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_service.h +++ b/frameworks/kits/ability/native/test/mock/include/mock_ability_manager_service.h @@ -171,7 +171,6 @@ public: int RemoveMission(int id) override; - int RemoveStack(int id) override; sptr AcquireDataAbility( const Uri &uri, bool tryBind, const sptr &callerToken) override { diff --git a/frameworks/kits/appkit/native/test/mock/include/mock_ability_manager_client_interface1.cpp b/frameworks/kits/appkit/native/test/mock/include/mock_ability_manager_client_interface1.cpp index 2a86fb9e7c0dce93cfde80581fa4891cfee4cc73..ae73ffc6c9caccb510d7ffb420cb39caa5db00de 100644 --- a/frameworks/kits/appkit/native/test/mock/include/mock_ability_manager_client_interface1.cpp +++ b/frameworks/kits/appkit/native/test/mock/include/mock_ability_manager_client_interface1.cpp @@ -175,11 +175,6 @@ ErrCode AbilityManagerClient::RemoveMissions(std::vector missionId) return ret; } -ErrCode AbilityManagerClient::RemoveStack(int id) -{ - return ERR_OK; -} - ErrCode AbilityManagerClient::KillProcess(const std::string &bundleName) { return ERR_OK; diff --git a/frameworks/kits/appkit/test/Mock/include/mock_ability_mgr_service.h b/frameworks/kits/appkit/test/Mock/include/mock_ability_mgr_service.h index d6588ed87ac47fc900799a17418beb8ffb788c7c..954b1140787d610bacbfd74c87059791db16498d 100644 --- a/frameworks/kits/appkit/test/Mock/include/mock_ability_mgr_service.h +++ b/frameworks/kits/appkit/test/Mock/include/mock_ability_mgr_service.h @@ -55,7 +55,6 @@ public: MOCK_METHOD3(GetRecentMissions, int(const int32_t, const int32_t, std::vector &)); MOCK_METHOD2(GetMissionSnapshot, int(const int32_t, MissionPixelMap &)); MOCK_METHOD1(RemoveMission, int(int)); - MOCK_METHOD1(RemoveStack, int(int)); MOCK_METHOD4(OnRemoteRequest, int(uint32_t code, MessageParcel &data, MessageParcel &reply, MessageOption &option)); MOCK_METHOD1(MoveMissionToTop, int(int32_t)); diff --git a/frameworks/kits/test/mock/AMS/mock_ability_manager_service.cpp b/frameworks/kits/test/mock/AMS/mock_ability_manager_service.cpp index 38e407f46f1d88df5e9a8e260c84bd2d614e09cf..9d78ed9e276402e6e4e551c4d5cfd060f325bc5f 100644 --- a/frameworks/kits/test/mock/AMS/mock_ability_manager_service.cpp +++ b/frameworks/kits/test/mock/AMS/mock_ability_manager_service.cpp @@ -190,11 +190,6 @@ int MockAbilityManagerService::RemoveMission(int id) return 0; } -int MockAbilityManagerService::RemoveStack(int id) -{ - return 0; -} - sptr MockAbilityManagerService::AcquireDataAbility( const Uri &uri, bool tryBind, const sptr &callerToken) { diff --git a/frameworks/kits/test/mock/AMS/mock_ability_manager_service.h b/frameworks/kits/test/mock/AMS/mock_ability_manager_service.h index f6b37411010254e1f9f5308dd9f446819323bdae..3c21ceecdabdbb10773fee85fae9f7d8609ee34a 100644 --- a/frameworks/kits/test/mock/AMS/mock_ability_manager_service.h +++ b/frameworks/kits/test/mock/AMS/mock_ability_manager_service.h @@ -154,7 +154,6 @@ public: int RemoveMission(int id) override; - int RemoveStack(int id) override; int PowerOff() override; int PowerOn() override; int LockMission(int missionId) override diff --git a/frameworks/kits/test/mock/AMS/mock_serviceability_manager_service.cpp b/frameworks/kits/test/mock/AMS/mock_serviceability_manager_service.cpp index d45b1c8c00316d76ded124994af2857e29fb7000..e5af44a41882ce01502ffd2635d3b3b5a5674b6c 100644 --- a/frameworks/kits/test/mock/AMS/mock_serviceability_manager_service.cpp +++ b/frameworks/kits/test/mock/AMS/mock_serviceability_manager_service.cpp @@ -181,11 +181,6 @@ int MockServiceAbilityManagerService::RemoveMission(int id) return 0; } -int MockServiceAbilityManagerService::RemoveStack(int id) -{ - return 0; -} - int MockServiceAbilityManagerService::MoveMissionToEnd(const sptr &token, const bool nonFirst) { return 0; diff --git a/frameworks/kits/test/mock/AMS/mock_serviceability_manager_service.h b/frameworks/kits/test/mock/AMS/mock_serviceability_manager_service.h index 438c3e677a61ad1d71b513e4b97832f0aec7c2bc..ad92b9a1efebbe61d9d9cc72960327c5bdfbbfd4 100644 --- a/frameworks/kits/test/mock/AMS/mock_serviceability_manager_service.h +++ b/frameworks/kits/test/mock/AMS/mock_serviceability_manager_service.h @@ -148,7 +148,6 @@ public: int RemoveMission(int id) override; - int RemoveStack(int id) override; int PowerOff() override; int PowerOn() override; int LockMission(int missionId) override diff --git a/interfaces/innerkits/ability_manager/include/ability_manager_client.h b/interfaces/innerkits/ability_manager/include/ability_manager_client.h index ab936d74dcfa40205264435f0b6764403be3b81d..ba964c84fcdb788428e0beeedbf84bd6e9cd8225 100644 --- a/interfaces/innerkits/ability_manager/include/ability_manager_client.h +++ b/interfaces/innerkits/ability_manager/include/ability_manager_client.h @@ -318,14 +318,6 @@ public: */ ErrCode RemoveMissions(std::vector missionId); - /** - * Remove the specified mission stack by stack id - * - * @param id. - * @return Returns ERR_OK on success, others on failure. - */ - ErrCode RemoveStack(int id); - /** * Kill the process immediately. * diff --git a/interfaces/innerkits/ability_manager/include/ability_manager_interface.h b/interfaces/innerkits/ability_manager/include/ability_manager_interface.h index 191d3d96360689da131a133a73d62d80ff0f2e5f..63ce3566ef5f105a12da6952873f915dcc47748d 100644 --- a/interfaces/innerkits/ability_manager/include/ability_manager_interface.h +++ b/interfaces/innerkits/ability_manager/include/ability_manager_interface.h @@ -357,14 +357,6 @@ public: */ virtual int RemoveMission(int id) = 0; - /** - * Remove the specified mission stack by stack id - * - * @param id. - * @return Returns ERR_OK on success, others on failure. - */ - virtual int RemoveStack(int id) = 0; - /** * Kill the process immediately. * diff --git a/interfaces/kits/napi/aafwk/abilityManager/napi_ability_manager.cpp b/interfaces/kits/napi/aafwk/abilityManager/napi_ability_manager.cpp index c40ee7839f8ca4809ae197c60e4ceca4a8f8dc64..41126989dd1ce259356f9b8d259c0d66d4479fb3 100644 --- a/interfaces/kits/napi/aafwk/abilityManager/napi_ability_manager.cpp +++ b/interfaces/kits/napi/aafwk/abilityManager/napi_ability_manager.cpp @@ -1361,124 +1361,6 @@ napi_value NAPI_RemoveMissions(napi_env env, napi_callback_info info) return ((callBackMode) ? (nullptr) : (ret)); } -auto NAPI_ClearMissionsAsyncExecuteCallback = [](napi_env env, void *data) { - HILOG_INFO("clearMissions called(CallBack Mode)..."); - AsyncClearMissionsCallbackInfo *async_callback_info = (AsyncClearMissionsCallbackInfo *)data; - async_callback_info->result = GetAbilityManagerInstance()->RemoveStack(NapiAbilityMgr::DEFAULT_STACK_ID); -}; - -auto NAPI_ClearMissionsAsyncCompleteCallback = [](napi_env env, napi_status status, void *data) { - HILOG_INFO("clearMissions compeleted(CallBack Mode)..."); - AsyncClearMissionsCallbackInfo *async_callback_info = (AsyncClearMissionsCallbackInfo *)data; - napi_value result[NUMBER_OF_PARAMETERS_TWO] = {0}; - napi_value callback = nullptr; - napi_value undefined = nullptr; - napi_value callResult = nullptr; - - result[0] = GetCallbackErrorValue(async_callback_info->env, NapiAbilityMgr::BUSINESS_ERROR_CODE_OK); - napi_create_int32(async_callback_info->env, async_callback_info->result, &result[1]); - napi_get_undefined(env, &undefined); - - napi_get_reference_value(env, async_callback_info->callback[0], &callback); - napi_call_function(env, undefined, callback, NUMBER_OF_PARAMETERS_TWO, &result[0], &callResult); - - if (async_callback_info->callback[0] != nullptr) { - napi_delete_reference(env, async_callback_info->callback[0]); - } - - napi_delete_async_work(env, async_callback_info->asyncWork); - delete async_callback_info; -}; - -auto NAPI_ClearMissionsPromiseCompleteCallback = [](napi_env env, napi_status status, void *data) { - HILOG_INFO("clearMissions compeleted(Promise Mode)..."); - AsyncClearMissionsCallbackInfo *async_callback_info = (AsyncClearMissionsCallbackInfo *)data; - napi_value result = nullptr; - napi_create_int32(async_callback_info->env, async_callback_info->result, &result); - napi_resolve_deferred(async_callback_info->env, async_callback_info->deferred, result); - napi_delete_async_work(env, async_callback_info->asyncWork); - delete async_callback_info; -}; - -napi_value NAPI_ClearMissionsWrap( - napi_env env, napi_callback_info info, bool callBackMode, AsyncClearMissionsCallbackInfo &async_callback_info) -{ - HILOG_INFO("NAPI_ClearMissions called..."); - if (callBackMode) { - napi_value resourceName = nullptr; - napi_create_string_latin1(env, "NAPI_ClearMissionsCallBack", NAPI_AUTO_LENGTH, &resourceName); - - napi_create_async_work(env, - nullptr, - resourceName, - NAPI_ClearMissionsAsyncExecuteCallback, - NAPI_ClearMissionsAsyncCompleteCallback, - (void *)&async_callback_info, - &async_callback_info.asyncWork); - - NAPI_CALL(env, napi_queue_async_work(env, async_callback_info.asyncWork)); - // create return - napi_value ret = 0; - NAPI_CALL(env, napi_create_int32(env, 0, &ret)); - return ret; - } else { - napi_value resourceName = nullptr; - napi_create_string_latin1(env, "NAPI_ClearMissionsPromise", NAPI_AUTO_LENGTH, &resourceName); - - napi_deferred deferred; - napi_value promise; - NAPI_CALL(env, napi_create_promise(env, &deferred, &promise)); - async_callback_info.deferred = deferred; - - napi_create_async_work(env, - nullptr, - resourceName, - NAPI_ClearMissionsAsyncExecuteCallback, - NAPI_ClearMissionsPromiseCompleteCallback, - (void *)&async_callback_info, - &async_callback_info.asyncWork); - napi_queue_async_work(env, async_callback_info.asyncWork); - return promise; - } -} - -napi_value NAPI_ClearMissions(napi_env env, napi_callback_info info) -{ - size_t argc = 1; - napi_value argv[1] = {}; - NAPI_CALL(env, napi_get_cb_info(env, info, &argc, argv, NULL, NULL)); - HILOG_INFO("argc = [%{public}zu]", argc); - - bool callBackMode = false; - if (argc >= 1) { - napi_valuetype valuetype = napi_valuetype::napi_null; - NAPI_CALL(env, napi_typeof(env, argv[0], &valuetype)); - NAPI_ASSERT(env, valuetype == napi_function, "Wrong argument type. Function expected."); - callBackMode = true; - } - - AsyncClearMissionsCallbackInfo *async_callback_info = new (std::nothrow) AsyncClearMissionsCallbackInfo { - .env = env, - .asyncWork = nullptr, - .deferred = nullptr, - }; - if (async_callback_info == nullptr) { - return nullptr; - } - - if (callBackMode) { - napi_create_reference(env, argv[0], 1, &async_callback_info->callback[0]); - } - - napi_value ret = NAPI_ClearMissionsWrap(env, info, callBackMode, *async_callback_info); - if (ret == nullptr) { - delete async_callback_info; - async_callback_info = nullptr; - } - - return ((callBackMode) ? (nullptr) : (ret)); -} - auto NAPI_MoveMissionToTopAsyncExecuteCallback = [](napi_env env, void *data) { HILOG_INFO("moveMissionToTop called(CallBack Mode)..."); AsyncMoveMissionToTopCallbackInfo *async_callback_info = (AsyncMoveMissionToTopCallbackInfo *)data; diff --git a/interfaces/kits/napi/aafwk/abilityManager/napi_ability_manager.h b/interfaces/kits/napi/aafwk/abilityManager/napi_ability_manager.h index fb6d259d6ca5176017b3bc37f3e1bcd2a9fe4950..2cefa6198ef3ac60c40ac9e3386b52887b8ca196 100644 --- a/interfaces/kits/napi/aafwk/abilityManager/napi_ability_manager.h +++ b/interfaces/kits/napi/aafwk/abilityManager/napi_ability_manager.h @@ -161,7 +161,6 @@ napi_value NAPI_QueryRecentAbilityMissionInfos(napi_env env, napi_callback_info napi_value NAPI_GetPreviousAbilityMissionInfos(napi_env env, napi_callback_info info); napi_value NAPI_RemoveMission(napi_env env, napi_callback_info info); napi_value NAPI_RemoveMissions(napi_env env, napi_callback_info info); -napi_value NAPI_ClearMissions(napi_env env, napi_callback_info info); napi_value NAPI_MoveMissionToTop(napi_env env, napi_callback_info info); napi_value NAPI_KillProcessesByBundleName(napi_env env, napi_callback_info info); napi_value NAPI_ClearUpApplicationData(napi_env env, napi_callback_info info); diff --git a/interfaces/kits/napi/aafwk/abilityManager/native_module.cpp b/interfaces/kits/napi/aafwk/abilityManager/native_module.cpp index 088ba3be0189da17de6f317e3070f8a02591177c..08cb43573e60e3d979caf3f8b96fb4bfb033b0a6 100644 --- a/interfaces/kits/napi/aafwk/abilityManager/native_module.cpp +++ b/interfaces/kits/napi/aafwk/abilityManager/native_module.cpp @@ -49,7 +49,6 @@ static napi_value Init(napi_env env, napi_value exports) DECLARE_NAPI_FUNCTION("removeRecentTask", NAPI_RemoveMission), DECLARE_NAPI_FUNCTION("removeMissions", NAPI_RemoveMissions), DECLARE_NAPI_FUNCTION("deleteMissions", NAPI_RemoveMissions), - DECLARE_NAPI_FUNCTION("clearMissions", NAPI_ClearMissions), DECLARE_NAPI_FUNCTION("moveMissionToTop", NAPI_MoveMissionToTop), DECLARE_NAPI_FUNCTION("killProcessesByBundleName", NAPI_KillProcessesByBundleName), DECLARE_NAPI_FUNCTION("clearUpApplicationData", NAPI_ClearUpApplicationData), diff --git a/services/abilitymgr/include/ability_manager_proxy.h b/services/abilitymgr/include/ability_manager_proxy.h index 634ac10a38827266778aeb662184b605a2719774..75c42dc224351e87657a96c7f5772a5f38b2a300 100644 --- a/services/abilitymgr/include/ability_manager_proxy.h +++ b/services/abilitymgr/include/ability_manager_proxy.h @@ -320,14 +320,6 @@ public: */ virtual int RemoveMission(int id) override; - /** - * Remove the specified mission stack by stack id - * - * @param id. - * @return Returns ERR_OK on success, others on failure. - */ - virtual int RemoveStack(int id) override; - /** * Kill the process immediately. * diff --git a/services/abilitymgr/include/ability_manager_service.h b/services/abilitymgr/include/ability_manager_service.h index 6899170bc2b9615a1964399abb6004ab5db8276d..5405f84e2035b5868290b4f535e94721c1265613 100644 --- a/services/abilitymgr/include/ability_manager_service.h +++ b/services/abilitymgr/include/ability_manager_service.h @@ -447,14 +447,6 @@ public: */ virtual int RemoveMission(int id) override; - /** - * Remove the specified mission stack by stack id - * - * @param id. - * @return Returns ERR_OK on success, others on failure. - */ - virtual int RemoveStack(int id) override; - /** * Kill the process immediately. * diff --git a/services/abilitymgr/include/ability_manager_stub.h b/services/abilitymgr/include/ability_manager_stub.h index 76bc7812ecf3c057c1d02c0edbce9ee7341fb61d..d7e748889473a99ef7ce5b8c000dbf7c3500b122 100644 --- a/services/abilitymgr/include/ability_manager_stub.h +++ b/services/abilitymgr/include/ability_manager_stub.h @@ -69,7 +69,6 @@ private: int GetAllStackInfoInner(MessageParcel &data, MessageParcel &reply); int GetRecentMissionsInner(MessageParcel &data, MessageParcel &reply); int RemoveMissionInner(MessageParcel &data, MessageParcel &reply); - int RemoveStackInner(MessageParcel &data, MessageParcel &reply); int ScheduleCommandAbilityDoneInner(MessageParcel &data, MessageParcel &reply); int GetMissionSnapshotInner(MessageParcel &data, MessageParcel &reply); int AcquireDataAbilityInner(MessageParcel &data, MessageParcel &reply); diff --git a/services/abilitymgr/include/ability_stack_manager.h b/services/abilitymgr/include/ability_stack_manager.h index 32553b3b2ef299b49a8e7fe78ed1fc149a84ca6d..0514f144c41c113656a224acb9860d8b9b635d5e 100644 --- a/services/abilitymgr/include/ability_stack_manager.h +++ b/services/abilitymgr/include/ability_stack_manager.h @@ -244,14 +244,6 @@ public: */ int RemoveMissionById(int missionId); - /** - * Remove the specified mission stack by stack id - * - * @param id. - * @return Returns ERR_OK on success, others on failure. - */ - int RemoveStack(int stackId); - /** * move the mission stack to the top. * diff --git a/services/abilitymgr/src/ability_manager_client.cpp b/services/abilitymgr/src/ability_manager_client.cpp index b3cabdb149b490ef0d184171f6c17a21a0d72c12..f6931c026121d287b98a532dd3f80ed34dc20ffb 100644 --- a/services/abilitymgr/src/ability_manager_client.cpp +++ b/services/abilitymgr/src/ability_manager_client.cpp @@ -343,13 +343,6 @@ ErrCode AbilityManagerClient::RemoveMissions(std::vector missionId) return error; } - -ErrCode AbilityManagerClient::RemoveStack(int id) -{ - CHECK_REMOTE_OBJECT_AND_RETURN(remoteObject_, ABILITY_SERVICE_NOT_CONNECTED); - sptr abms = iface_cast(remoteObject_); - return abms->RemoveStack(id); -} #endif ErrCode AbilityManagerClient::KillProcess(const std::string &bundleName) diff --git a/services/abilitymgr/src/ability_manager_proxy.cpp b/services/abilitymgr/src/ability_manager_proxy.cpp index 12120c09917b5646556f4cbf55f4b8cdff47520c..c651a2347bb3931d7a5d4011e94ad071abfebc99 100644 --- a/services/abilitymgr/src/ability_manager_proxy.cpp +++ b/services/abilitymgr/src/ability_manager_proxy.cpp @@ -842,28 +842,6 @@ int AbilityManagerProxy::RemoveMission(int id) return reply.ReadInt32(); } -int AbilityManagerProxy::RemoveStack(int id) -{ - int error; - MessageParcel data; - MessageParcel reply; - MessageOption option; - - if (!WriteInterfaceToken(data)) { - return INNER_ERR; - } - if (!data.WriteInt32(id)) { - HILOG_ERROR("id write failed."); - return ERR_INVALID_VALUE; - } - error = Remote()->SendRequest(IAbilityManager::REMOVE_STACK, data, reply, option); - if (error != NO_ERROR) { - HILOG_ERROR("Send request error: %{public}d", error); - return error; - } - return reply.ReadInt32(); -} - int AbilityManagerProxy::KillProcess(const std::string &bundleName) { MessageParcel data; diff --git a/services/abilitymgr/src/ability_manager_service.cpp b/services/abilitymgr/src/ability_manager_service.cpp index d82a86fe87866fd17b57976513a4a3b0abd79cf3..ca074604404e04a019345f686866018645b6f4ea 100644 --- a/services/abilitymgr/src/ability_manager_service.cpp +++ b/services/abilitymgr/src/ability_manager_service.cpp @@ -922,16 +922,6 @@ int AbilityManagerService::RemoveMission(int id) } return currentStackManager_->RemoveMissionById(id); } - -int AbilityManagerService::RemoveStack(int id) -{ - HILOG_INFO("Remove stack."); - if (id < 0) { - HILOG_ERROR("Stack id is invalid."); - return ERR_INVALID_VALUE; - } - return currentStackManager_->RemoveStack(id); -} #endif int AbilityManagerService::ConnectAbility( diff --git a/services/abilitymgr/src/ability_manager_stub.cpp b/services/abilitymgr/src/ability_manager_stub.cpp index 9ad749d60d25045b73ddb372c13eb4734a6ea45d..bd27fe413700ca674dfd6abcd8076e059ad8f68a 100644 --- a/services/abilitymgr/src/ability_manager_stub.cpp +++ b/services/abilitymgr/src/ability_manager_stub.cpp @@ -53,7 +53,6 @@ void AbilityManagerStub::FirstStepInit() requestFuncMap_[LIST_STACK_INFO] = &AbilityManagerStub::GetAllStackInfoInner; requestFuncMap_[GET_RECENT_MISSION] = &AbilityManagerStub::GetRecentMissionsInner; requestFuncMap_[REMOVE_MISSION] = &AbilityManagerStub::RemoveMissionInner; - requestFuncMap_[REMOVE_STACK] = &AbilityManagerStub::RemoveStackInner; requestFuncMap_[COMMAND_ABILITY_DONE] = &AbilityManagerStub::ScheduleCommandAbilityDoneInner; requestFuncMap_[GET_MISSION_SNAPSHOT] = &AbilityManagerStub::GetMissionSnapshotInner; requestFuncMap_[ACQUIRE_DATA_ABILITY] = &AbilityManagerStub::AcquireDataAbilityInner; @@ -313,17 +312,6 @@ int AbilityManagerStub::RemoveMissionInner(MessageParcel &data, MessageParcel &r return NO_ERROR; } -int AbilityManagerStub::RemoveStackInner(MessageParcel &data, MessageParcel &reply) -{ - int id = data.ReadInt32(); - int32_t result = RemoveStack(id); - if (!reply.WriteInt32(result)) { - HILOG_ERROR("remove stack error"); - return ERR_INVALID_VALUE; - } - return NO_ERROR; -} - int AbilityManagerStub::ScheduleCommandAbilityDoneInner(MessageParcel &data, MessageParcel &reply) { auto token = data.ReadParcelable(); diff --git a/services/abilitymgr/src/ability_stack_manager.cpp b/services/abilitymgr/src/ability_stack_manager.cpp index b99cdbfb6a167de1bb44024d2d4c289b93a5e349..bfc41f2926545b1b2bcb75c66ec78d13bd289e2f 100644 --- a/services/abilitymgr/src/ability_stack_manager.cpp +++ b/services/abilitymgr/src/ability_stack_manager.cpp @@ -765,23 +765,6 @@ int AbilityStackManager::RemoveMissionByIdLocked(int missionId) return TerminateAbilityLocked(terminateLists); } -int AbilityStackManager::RemoveStack(int stackId) -{ - HILOG_DEBUG("stackId : %{public}d", stackId); - std::lock_guard guard(stackLock_); - if (stackId < MIN_MISSION_STACK_ID) { - HILOG_ERROR("stackId:%{public}d is invalid.", stackId); - return ERR_INVALID_VALUE; - } - - if (lockMissionContainer_ && lockMissionContainer_->IsLockedMissionState()) { - HILOG_ERROR("current is lock mission state, refusing to operate other mission."); - return ERR_INVALID_VALUE; - } - - return RemoveStackLocked(stackId); -} - int AbilityStackManager::RemoveStackLocked(int stackId) { HILOG_DEBUG("stackId : %{public}d", stackId); diff --git a/services/abilitymgr/test/unittest/phone/ability_manager_client_test/ability_manager_stub_mock_test.h b/services/abilitymgr/test/unittest/phone/ability_manager_client_test/ability_manager_stub_mock_test.h index f064283b3abda61a4ea1f8339c523e844ada0428..d0d6822a9a05ad6d2883febe2c9af528f7f52837 100644 --- a/services/abilitymgr/test/unittest/phone/ability_manager_client_test/ability_manager_stub_mock_test.h +++ b/services/abilitymgr/test/unittest/phone/ability_manager_client_test/ability_manager_stub_mock_test.h @@ -202,11 +202,6 @@ public: return 0; } - virtual int RemoveStack(int id) - { - return 0; - } - virtual int KillProcess(const std::string &bundleName) { return 0; diff --git a/services/abilitymgr/test/unittest/phone/ability_manager_proxy_test/ability_manager_proxy_test.cpp b/services/abilitymgr/test/unittest/phone/ability_manager_proxy_test/ability_manager_proxy_test.cpp index 73d01f903856778d480eb355cd8b4021271ac928..4510d2a1716b877c1c185a1018584dd8ddf522c3 100644 --- a/services/abilitymgr/test/unittest/phone/ability_manager_proxy_test/ability_manager_proxy_test.cpp +++ b/services/abilitymgr/test/unittest/phone/ability_manager_proxy_test/ability_manager_proxy_test.cpp @@ -532,24 +532,6 @@ HWTEST_F(AbilityManagerProxyTest, AbilityManagerProxy_0023, TestSize.Level1) EXPECT_EQ(IAbilityManager::REMOVE_MISSION, mock_->code_); } -/* - * Feature: AbilityManagerService - * Function: RemoveStack - * SubFunction: NA - * FunctionPoints: AbilityManagerService RemoveStack - * EnvConditions: NA - * CaseDescription: Verify the normal conditions of RemoveStack - */ -HWTEST_F(AbilityManagerProxyTest, AbilityManagerProxy_0024, TestSize.Level1) -{ - EXPECT_CALL(*mock_, SendRequest(_, _, _, _)) - .Times(1) - .WillOnce(Invoke(mock_.GetRefPtr(), &AbilityManagerStubMock::InvokeSendRequest)); - proxy_->RemoveStack(1); - - EXPECT_EQ(IAbilityManager::REMOVE_STACK, mock_->code_); -} - /* * Feature: AbilityManagerService * Function: MoveMissionToTop diff --git a/services/abilitymgr/test/unittest/phone/ability_manager_proxy_test/ability_manager_stub_mock.h b/services/abilitymgr/test/unittest/phone/ability_manager_proxy_test/ability_manager_stub_mock.h index 11cb38f66b76e0635a88a8040461bc2711c07368..bc045ed4edd82f6025ea1daeb1593decd4424036 100644 --- a/services/abilitymgr/test/unittest/phone/ability_manager_proxy_test/ability_manager_stub_mock.h +++ b/services/abilitymgr/test/unittest/phone/ability_manager_proxy_test/ability_manager_stub_mock.h @@ -202,11 +202,6 @@ public: return 0; } - virtual int RemoveStack(int id) - { - return 0; - } - virtual int KillProcess(const std::string &bundleName) { return 0; diff --git a/services/abilitymgr/test/unittest/phone/ability_manager_service_test/ability_manager_service_test.cpp b/services/abilitymgr/test/unittest/phone/ability_manager_service_test/ability_manager_service_test.cpp index ae0b36b1d0f7c6b45bbd18344a2324b23abcad22..4eebf329c30b7b622f4c9bde458056edb571d6bc 100644 --- a/services/abilitymgr/test/unittest/phone/ability_manager_service_test/ability_manager_service_test.cpp +++ b/services/abilitymgr/test/unittest/phone/ability_manager_service_test/ability_manager_service_test.cpp @@ -797,54 +797,6 @@ HWTEST_F(AbilityManagerServiceTest, Interface_018, TestSize.Level1) EXPECT_EQ(info[0].topAbility.GetDeviceID(), want1.GetElement().GetDeviceID()); } -/* - * Feature: AbilityManagerService - * Function: RemoveStack - * SubFunction: NA - * FunctionPoints: AbilityManagerService RemoveStack - * EnvConditions: NA - * CaseDescription: 1.stackid < 0, RemoveStack failed - * 2.stackid no exist, RemoveStack failed - * 3.don't allow remove luncher mission stack - */ -HWTEST_F(AbilityManagerServiceTest, Interface_019, TestSize.Level1) -{ - EXPECT_EQ(abilityMs_->RemoveStack(-1), OHOS::ERR_INVALID_VALUE); - EXPECT_EQ(abilityMs_->RemoveStack(INT_MAX), REMOVE_STACK_ID_NOT_EXIST); - EXPECT_EQ(abilityMs_->RemoveStack(0), REMOVE_STACK_LAUNCHER_DENIED); -} - -/* - * Feature: AbilityManagerService - * Function: RemoveStack - * SubFunction: NA - * FunctionPoints: AbilityManagerService RemoveStack - * EnvConditions: NA - * CaseDescription: remove default stack success - */ -HWTEST_F(AbilityManagerServiceTest, Interface_020, TestSize.Level1) -{ - Want want; - ElementName element("device", "com.ix.music", "MusicAbility"); - want.SetElement(element); - auto result = StartAbility(want); - WaitUntilTaskFinished(); - EXPECT_EQ(OHOS::ERR_OK, result); - auto stackManage = abilityMs_->GetStackManager(); - stackManage->GetCurrentTopAbility()->SetAbilityState(OHOS::AAFwk::AbilityState::ACTIVE); - - Want want1; - ElementName element1("device", "com.ix.music", "MusicAbility"); - want1.SetElement(element1); - auto result1 = StartAbility(want); - WaitUntilTaskFinished(); - EXPECT_EQ(OHOS::ERR_OK, result1); - - EXPECT_EQ(stackManage->GetCurrentMissionStack()->GetMissionRecordCount(), 1); - EXPECT_EQ(abilityMs_->RemoveStack(1), ERR_OK); - -} - /* * Feature: AbilityManagerService * Function: ConnectAbility diff --git a/services/abilitymgr/test/unittest/phone/ability_manager_stub_test/ability_manager_stub_impl_mock.h b/services/abilitymgr/test/unittest/phone/ability_manager_stub_test/ability_manager_stub_impl_mock.h index 1da93ac68b26ed05ad64451b7ca9d6e392927e62..d7af3387acfb11d6c543eca2f16fad9f3138808b 100644 --- a/services/abilitymgr/test/unittest/phone/ability_manager_stub_test/ability_manager_stub_impl_mock.h +++ b/services/abilitymgr/test/unittest/phone/ability_manager_stub_test/ability_manager_stub_impl_mock.h @@ -215,11 +215,6 @@ public: return 0; } - virtual int RemoveStack(int id) - { - return 0; - } - virtual int KillProcess(const std::string &bundleName) { return 0; diff --git a/services/abilitymgr/test/unittest/phone/ability_manager_test/ability_manager_stub_mock.h b/services/abilitymgr/test/unittest/phone/ability_manager_test/ability_manager_stub_mock.h index ebae85c6f74f9240d50f6c5abc878fcf8f6baffc..99350177f83aa70a91506558155b2f9ca5272095 100644 --- a/services/abilitymgr/test/unittest/phone/ability_manager_test/ability_manager_stub_mock.h +++ b/services/abilitymgr/test/unittest/phone/ability_manager_test/ability_manager_stub_mock.h @@ -62,7 +62,6 @@ public: MOCK_METHOD3(GetRecentMissions, int(const int32_t, const int32_t, std::vector &)); MOCK_METHOD2(GetMissionSnapshot, int(const int32_t, MissionSnapshotInfo &)); MOCK_METHOD1(RemoveMission, int(int)); - MOCK_METHOD1(RemoveStack, int(int)); MOCK_METHOD1(MoveMissionToTop, int(int32_t)); MOCK_METHOD1(KillProcess, int(const std::string &)); MOCK_METHOD2(UninstallApp, int(const std::string &, int32_t)); diff --git a/services/abilitymgr/test/unittest/phone/ability_stack_manager_test/ability_stack_manager_test.cpp b/services/abilitymgr/test/unittest/phone/ability_stack_manager_test/ability_stack_manager_test.cpp index 05d94121bd23b89f6f5e5299e19ed9c0da7dce60..6832bfbb17128302f520de87b5fbb4005e11e0fe 100644 --- a/services/abilitymgr/test/unittest/phone/ability_stack_manager_test/ability_stack_manager_test.cpp +++ b/services/abilitymgr/test/unittest/phone/ability_stack_manager_test/ability_stack_manager_test.cpp @@ -1585,76 +1585,11 @@ HWTEST_F(AbilityStackManagerTest, ability_stack_manager_operating_044, TestSize. /* * Feature: AbilityStackManager - * Function: RemoveStack - * SubFunction: NA - * FunctionPoints: RemoveStack - * EnvConditions: NA - * CaseDescription: Failed to verify RemoveStack - */ -HWTEST_F(AbilityStackManagerTest, ability_stack_manager_operating_045, TestSize.Level1) -{ - stackManager_->Init(); - auto result = stackManager_->StartAbility(launcherAbilityRequest_); - EXPECT_EQ(ERR_OK, result); - auto firstTopAbility = stackManager_->GetCurrentTopAbility(); - firstTopAbility->SetAbilityState(OHOS::AAFwk::ACTIVE); - - result = stackManager_->StartAbility(musicAbilityRequest_); - EXPECT_EQ(ERR_OK, result); - auto secondTopAbility = stackManager_->GetCurrentTopAbility(); - secondTopAbility->SetAbilityState(OHOS::AAFwk::ACTIVE); - - result = stackManager_->RemoveStack(-1); - EXPECT_EQ(ERR_INVALID_VALUE, result); - - result = stackManager_->RemoveStack(10); - EXPECT_EQ(REMOVE_STACK_ID_NOT_EXIST, result); - - result = stackManager_->RemoveStack(0); - EXPECT_EQ(REMOVE_STACK_LAUNCHER_DENIED, result); - - stackManager_->missionStackList_.clear(); - result = stackManager_->RemoveStack(1); - EXPECT_EQ(MISSION_STACK_LIST_IS_EMPTY, result); -} - -/* - * Feature: AbilityStackManager - * Function: RemoveStack - * SubFunction: NA - * FunctionPoints: RemoveStack - * EnvConditions: NA - * CaseDescription: Succeeded to verify RemoveStack - */ -HWTEST_F(AbilityStackManagerTest, ability_stack_manager_operating_046, TestSize.Level1) -{ - stackManager_->Init(); - auto result = stackManager_->StartAbility(launcherAbilityRequest_); - EXPECT_EQ(ERR_OK, result); - auto firstTopAbility = stackManager_->GetCurrentTopAbility(); - firstTopAbility->SetAbilityState(OHOS::AAFwk::ACTIVE); - - result = stackManager_->StartAbility(musicAbilityRequest_); - EXPECT_EQ(ERR_OK, result); - auto secondTopAbility = stackManager_->GetCurrentTopAbility(); - secondTopAbility->SetAbilityState(OHOS::AAFwk::ACTIVE); - - result = stackManager_->StartAbility(launcherAbilityRequest_); - EXPECT_EQ(ERR_OK, result); - auto topAbility = stackManager_->GetCurrentTopAbility(); - topAbility->SetAbilityState(OHOS::AAFwk::ACTIVE); - - result = stackManager_->RemoveStack(1); - EXPECT_EQ(ERR_OK, result); -} - -/* - * Feature: AbilityStackManager - * Function: RemoveStack + * Function: StartAbility * SubFunction: NA - * FunctionPoints: RemoveStack + * FunctionPoints: StartAbility * EnvConditions: NA - * CaseDescription: Succeeded to verify RemoveStack + * CaseDescription: Succeeded to verify StartAbility */ HWTEST_F(AbilityStackManagerTest, ability_stack_manager_operating_047, TestSize.Level1) { @@ -1675,11 +1610,11 @@ HWTEST_F(AbilityStackManagerTest, ability_stack_manager_operating_047, TestSize. /* * Feature: AbilityStackManager - * Function: RemoveStack + * Function: StartAbility * SubFunction: NA - * FunctionPoints: RemoveStack + * FunctionPoints: StartAbility * EnvConditions: NA - * CaseDescription: Succeeded to verify RemoveStack + * CaseDescription: Succeeded to verify StartAbility */ HWTEST_F(AbilityStackManagerTest, ability_stack_manager_operating_048, TestSize.Level1) { @@ -1740,11 +1675,11 @@ HWTEST_F(AbilityStackManagerTest, ability_stack_manager_operating_048, TestSize. /* * Feature: AbilityStackManager - * Function: RemoveStack + * Function: StartAbility * SubFunction: NA - * FunctionPoints: RemoveStack + * FunctionPoints: StartAbility * EnvConditions: NA - * CaseDescription: Succeeded to verify RemoveStack + * CaseDescription: Succeeded to verify StartAbility */ HWTEST_F(AbilityStackManagerTest, ability_stack_manager_operating_049, TestSize.Level1) { diff --git a/services/formmgr/test/mock/include/mock_ability_manager.h b/services/formmgr/test/mock/include/mock_ability_manager.h index 5a773d2479e4ff640ae7cd1c6c291760aed84ce5..e3f85d151c6c740e63b495bd8ae33dd6884c4d26 100644 --- a/services/formmgr/test/mock/include/mock_ability_manager.h +++ b/services/formmgr/test/mock/include/mock_ability_manager.h @@ -223,10 +223,6 @@ public: { return 0; } - virtual int RemoveStack(int id) - { - return 0; - } virtual int KillProcess(const std::string &bundleName) { return 0; @@ -606,10 +602,6 @@ public: { return 0; } - virtual int RemoveStack(int id) - { - return 0; - } virtual int KillProcess(const std::string &bundleName) { return 0; diff --git a/services/test/mock/include/appmgr/mock_ability_mgr_host.h b/services/test/mock/include/appmgr/mock_ability_mgr_host.h index 8101cdcc27a61aed4e78aad6f0929554e6105a2a..5a080637db4b4253831e6572b0837dd0603580d0 100644 --- a/services/test/mock/include/appmgr/mock_ability_mgr_host.h +++ b/services/test/mock/include/appmgr/mock_ability_mgr_host.h @@ -142,10 +142,6 @@ public: { return 0; } - virtual int RemoveStack(int id) override - { - return 0; - } virtual int KillProcess(const std::string &bundleName) override { return 0; diff --git a/services/test/mock/include/mock_ability_manager_proxy.h b/services/test/mock/include/mock_ability_manager_proxy.h index 869639d68d49c448e6bed9b173cd8d1c7925d01b..8a38505adca4d7496b66deafe361e0a6272aad88 100644 --- a/services/test/mock/include/mock_ability_manager_proxy.h +++ b/services/test/mock/include/mock_ability_manager_proxy.h @@ -53,7 +53,6 @@ public: MOCK_METHOD3(GetRecentMissions, int(const int32_t, const int32_t, std::vector &)); MOCK_METHOD2(GetMissionSnapshot, int(const int32_t, MissionPixelMap &)); MOCK_METHOD1(RemoveMission, int(int)); - MOCK_METHOD1(RemoveStack, int(int)); MOCK_METHOD1(MoveMissionToTop, int(int32_t)); MOCK_METHOD1(GetMissionIdByToken, int32_t(const sptr &token)); MOCK_METHOD1(KillProcess, int(const std::string &)); diff --git a/services/test/mock/include/mock_ability_mgr_service.h b/services/test/mock/include/mock_ability_mgr_service.h index c4609ad5c5ab8ba1b5f055b19d6e342f29ccf67c..fd35fee42cf845f3642ffdd7780913aecefa422b 100644 --- a/services/test/mock/include/mock_ability_mgr_service.h +++ b/services/test/mock/include/mock_ability_mgr_service.h @@ -61,7 +61,6 @@ public: MOCK_METHOD3(GetRecentMissions, int(const int32_t, const int32_t, std::vector &)); MOCK_METHOD2(GetMissionSnapshot, int(const int32_t, MissionPixelMap &)); MOCK_METHOD1(RemoveMission, int(int)); - MOCK_METHOD1(RemoveStack, int(int)); MOCK_METHOD1(MoveMissionToTop, int(int32_t)); MOCK_METHOD1(GetMissionIdByToken, int32_t(const sptr &token)); MOCK_METHOD1(KillProcess, int(const std::string &)); diff --git a/services/test/moduletest/ability_stack_test/ability_stack_module_test.cpp b/services/test/moduletest/ability_stack_test/ability_stack_module_test.cpp index 268565fdf0a829b865f737f98a03b9c812a9df7c..47e4f0bbf1ee1e1ca1479d4fb9431133015fb132 100644 --- a/services/test/moduletest/ability_stack_test/ability_stack_module_test.cpp +++ b/services/test/moduletest/ability_stack_test/ability_stack_module_test.cpp @@ -3487,60 +3487,6 @@ HWTEST_F(AbilityStackModuleTest, ability_stack_test_068, TestSize.Level1) EXPECT_EQ(stackManager_->GetCurrentTopAbility(), topAbilityRecordMusic); } -/* - * Feature: AaFwk - * Function:RemoveStack - * SubFunction:Remove Stack - * FunctionPoints: - * EnvConditions: NA - * CaseDescription:Remove Stack - */ -HWTEST_F(AbilityStackModuleTest, ability_stack_test_069, TestSize.Level1) -{ - stackManager_->isMultiWinMoving_ = false; - // start launcher - auto worldAbilityRequest = GenerateAbilityRequest("device", "WorldAbility", "world", "com.ix.hiworld"); - auto ref = stackManager_->StartAbility(worldAbilityRequest); - EXPECT_EQ(ERR_OK, ref); - auto abilityRecordWorld = stackManager_->GetCurrentTopAbility(); - EXPECT_TRUE(abilityRecordWorld); - abilityRecordWorld->SetAbilityState(OHOS::AAFwk::ACTIVE); - - auto musicTopAbilityRequest = GenerateAbilityRequest("device", "MusicTopAbility", "musicTop", "com.ix.hiTopMusic"); - ref = stackManager_->StartAbility(musicTopAbilityRequest); - EXPECT_EQ(ERR_OK, ref); - auto topAbilityRecordMusic = stackManager_->GetCurrentTopAbility(); - EXPECT_TRUE(topAbilityRecordMusic); - topAbilityRecordMusic->SetAbilityState(OHOS::AAFwk::ACTIVE); - - std::shared_ptr targetStack = stackManager_->GetTargetMissionStack(musicTopAbilityRequest); - EXPECT_EQ(stackManager_->currentMissionStack_, targetStack); - - // start split ability - auto radioAbilityRequest = GenerateAbilityRequest("device", "RadioAbility", "radio", "com.ix.hiRadio"); - auto abilityStartSetting = AbilityStartSetting::GetEmptySetting(); - EXPECT_TRUE(abilityStartSetting); - abilityStartSetting->AddProperty(AbilityStartSetting::WINDOW_MODE_KEY, - std::to_string(AbilityWindowConfiguration::MULTI_WINDOW_DISPLAY_SECONDARY)); - radioAbilityRequest.startSetting = abilityStartSetting; - ref = stackManager_->StartAbility(radioAbilityRequest); - EXPECT_EQ(ERR_OK, ref); - - auto topAbilityRecordRadio = stackManager_->GetCurrentTopAbility(); - EXPECT_TRUE(topAbilityRecordRadio); - topAbilityRecordRadio->SetAbilityState(OHOS::AAFwk::ACTIVE); - auto topMissionRecordRadio = stackManager_->GetTopMissionRecord(); - EXPECT_TRUE(topMissionRecordRadio); - -#ifdef SUPPORT_GRAPHICS - EXPECT_EQ(stackManager_->curSysWindowMode_, SystemWindowMode::SPLITSCREEN_WINDOW_MODE); -#endif - - auto stackId = topMissionRecordRadio->GetMissionStack()->GetMissionStackId(); - stackManager_->RemoveStack(stackId); - EXPECT_EQ(stackManager_->GetCurrentTopAbility(), abilityRecordWorld); -} - /* * Feature: AaFwk * Function:OnTimeOut diff --git a/services/test/moduletest/ipc_ability_mgr_test/ipc_ability_mgr_module_test.cpp b/services/test/moduletest/ipc_ability_mgr_test/ipc_ability_mgr_module_test.cpp index fde03e58f83ff571eaeed57a5050f29b9a513ed5..48f97ade8b32f7952cdcaeffd2513cda68ff7ef3 100644 --- a/services/test/moduletest/ipc_ability_mgr_test/ipc_ability_mgr_module_test.cpp +++ b/services/test/moduletest/ipc_ability_mgr_test/ipc_ability_mgr_module_test.cpp @@ -517,32 +517,6 @@ HWTEST_F(IpcAbilityMgrModuleTest, AbilityMgrService_IPC_016, TestSize.Level1) GTEST_LOG_(INFO) << "IpcAbilityMgrModuleTest AbilityMgrService_IPC_016 end"; } -/* - * Feature: AAFwk - * Function: AbilityManagerService - * SubFunction: IPC of client and server - * FunctionPoints: RemoveStack - * EnvConditions: NA - * CaseDescription: verify RemoveStack IPC between client and server. - */ -HWTEST_F(IpcAbilityMgrModuleTest, AbilityMgrService_IPC_017, TestSize.Level1) -{ - GTEST_LOG_(INFO) << "IpcAbilityMgrModuleTest AbilityMgrService_IPC_017 start"; - - for (int i = 0; i < COUNT; i++) { - sptr mockAbilityMgr(new MockAbilityMgrService()); - sptr abilityMgrClient = iface_cast(mockAbilityMgr); - - EXPECT_CALL(*mockAbilityMgr, RemoveStack(_)) - .Times(1) - .WillOnce(InvokeWithoutArgs(mockAbilityMgr.GetRefPtr(), &MockAbilityMgrService::Post)); - abilityMgrClient->RemoveStack(1); - mockAbilityMgr->Wait(); - } - - GTEST_LOG_(INFO) << "IpcAbilityMgrModuleTest AbilityMgrService_IPC_017 end"; -} - /* * Feature: AAFwk * Function: AbilityManagerService diff --git a/test/systemtest/common/ams/ams_app_process_manage_test/ams_app_process_manage_test.cpp b/test/systemtest/common/ams/ams_app_process_manage_test/ams_app_process_manage_test.cpp index c1b1e6c18155eaf1b74b52b067f288136a5aa447..6245935d39fe24a3dc342ee19f7d2ba4c446e710 100644 --- a/test/systemtest/common/ams/ams_app_process_manage_test/ams_app_process_manage_test.cpp +++ b/test/systemtest/common/ams/ams_app_process_manage_test/ams_app_process_manage_test.cpp @@ -126,7 +126,6 @@ void AmsAppProcessManageTest::SetUp(void) void AmsAppProcessManageTest::TearDown(void) { GTEST_LOG_(INFO) << "void AmsAppProcessManageTest::TearDown(void)"; - STAbilityUtil::RemoveStack(1, abilityMs_, WAIT_TIME, WAIT_LAUNCHER_OK); std::vector vecBundleName; for (const auto &suffix : bundleNameSuffix) { vecBundleName.push_back(bundleNameBase + suffix); diff --git a/test/systemtest/common/ams/ams_kit_test/acts_ams_kit_test.cpp b/test/systemtest/common/ams/ams_kit_test/acts_ams_kit_test.cpp index abd73ea412c04147c77c57cb3ba4c2d83bf8abbc..43423fcb8d1c7684226589bd651ace7d5db313eb 100644 --- a/test/systemtest/common/ams/ams_kit_test/acts_ams_kit_test.cpp +++ b/test/systemtest/common/ams/ams_kit_test/acts_ams_kit_test.cpp @@ -148,7 +148,6 @@ void ActsAmsKitTest::SetUp(void) void ActsAmsKitTest::TearDown(void) { - STAbilityUtil::RemoveStack(1, abilityMs, WAIT_TIME, WAIT_LAUNCHER_OK); CleanMsg(); } @@ -2023,59 +2022,3 @@ HWTEST_F(ActsAmsKitTest, AMS_Page_Application_2600, Function | MediumTest | Leve } } } - -/** - * @tc.number : AMS_Page_Application_2700 - * @tc.name : OHOSApplicationApi::Lifecycle callback function - * @tc.desc : 1.Control the life cycle of the ability - * 2.Application registration capability life cycle callback - * 3.The callback function corresponding to the registered class will be called - */ -HWTEST_F(ActsAmsKitTest, AMS_Page_Application_2700, Function | MediumTest | Level1) -{ - for (int i = 1; i <= stLevel_.AMSLevel; i++) { - MAP_STR_STR params; - params["targetBundle"] = bundleName2; - params["targetAbility"] = thirdAbilityName; - Want want = STAbilityUtil::MakeWant("device", sixthAbilityName, bundleName2, params); - STAbilityUtil::StartAbility(want, abilityMs); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, sixthAbilityName + g_onAbilityActive, 0), 0); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, sixthAbilityName + g_onAbilityInactive, 0), 0); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, thirdAbilityName + g_onAbilityStart, 0), 0); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, thirdAbilityName + g_onAbilityActive, 0), 0); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, sixthAbilityName + g_onAbilityBackground, 0), 0); - CleanMsg(); - STAbilityUtil::RemoveStack(1, abilityMs, WAIT_TIME, WAIT_LAUNCHER_OK); - } -} - -/** - * @tc.number : AMS_Page_Application_2800 - * @tc.name : OHOSApplicationApi::Lifecycle callback function - * @tc.desc : 1.Control the life cycle of the ability - * 2.Application registration capability life cycle callback - * 3.The callback function corresponding to the registered class will be called - */ -HWTEST_F(ActsAmsKitTest, AMS_Page_Application_2800, Function | MediumTest | Level1) -{ - for (int i = 1; i <= stLevel_.AMSLevel; i++) { - MAP_STR_STR params; - int onStopWantCount = 1; - params["targetBundle"] = bundleName3; - params["targetAbility"] = abilityNameBase; - Want want = STAbilityUtil::MakeWant("device", sixthAbilityName, bundleName2, params); - STAbilityUtil::StartAbility(want, abilityMs); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, sixthAbilityName + g_onAbilityActive, 0), 0); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, sixthAbilityName + g_onAbilityInactive, 0), 0); - EXPECT_EQ( - STAbilityUtil::WaitCompleted(abilityEvent, abilityNameBase + g_abilityStateOnActive, onStopWantCount), 0); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, sixthAbilityName + g_onAbilityBackground, 0), 0); - STAbilityUtil::StopAbility(terminatePageAbility, 0, abilityNameBase); - EXPECT_EQ(STAbilityUtil::WaitCompleted(abilityEvent, sixthAbilityName + g_onAbilityActive, 0), 0); - EXPECT_EQ( - STAbilityUtil::WaitCompleted(abilityEvent, abilityNameBase + g_abilityStateOnStop, onStopWantCount), 0); - - CleanMsg(); - STAbilityUtil::RemoveStack(1, abilityMs, WAIT_TIME, WAIT_LAUNCHER_OK); - } -} \ No newline at end of file diff --git a/test/systemtest/common/ams/ams_page_ability_test/ams_page_ability_test.cpp b/test/systemtest/common/ams/ams_page_ability_test/ams_page_ability_test.cpp index 00988b712d761befd296cd70c6be05077387eafc..f7e2f859586e8b8df8534ea4ff78e5d25521f359 100644 --- a/test/systemtest/common/ams/ams_page_ability_test/ams_page_ability_test.cpp +++ b/test/systemtest/common/ams/ams_page_ability_test/ams_page_ability_test.cpp @@ -177,7 +177,6 @@ void AmsPageAbilityTest::SetUp(void) void AmsPageAbilityTest::TearDown(void) { GTEST_LOG_(INFO) << "void AmsPageAbilityTest::TearDown(void)"; - STAbilityUtil::RemoveStack(1, abilityMs_, WAIT_TIME, WAIT_LAUNCHER_OK); std::vector vecBundleName; for (const auto &suffix : bundleNameSuffix) { vecBundleName.push_back(bundleNameBase + suffix); diff --git a/test/systemtest/common/ams/ams_power_test/ams_power_test.cpp b/test/systemtest/common/ams/ams_power_test/ams_power_test.cpp index c3448baf7114c82d688835783fa0b7986b58565e..66b991c7daa1f41bc55b58c4829a33b1240c1793 100644 --- a/test/systemtest/common/ams/ams_power_test/ams_power_test.cpp +++ b/test/systemtest/common/ams/ams_power_test/ams_power_test.cpp @@ -159,7 +159,6 @@ void AmsPowerTest::SetUp(void) void AmsPowerTest::TearDown(void) { GTEST_LOG_(INFO) << "void AmsPowerTest::TearDown(void)"; - STAbilityUtil::RemoveStack(1, abilityMs_, WAIT_TIME, WAIT_LAUNCHER_OK); std::vector vecBundleName; for (const auto &suffix : bundleNameSuffix) { vecBundleName.push_back(bundleNameBase + suffix); diff --git a/test/systemtest/common/ams/tool/include/st_ability_util.h b/test/systemtest/common/ams/tool/include/st_ability_util.h index f76e49f317abc0e790c81740d5f687f74c962de7..6c1647af4e0d5c486afdcc88a09c019086da69f3 100644 --- a/test/systemtest/common/ams/tool/include/st_ability_util.h +++ b/test/systemtest/common/ams/tool/include/st_ability_util.h @@ -272,17 +272,6 @@ public: */ static ErrCode StopServiceAbility(const AAFwk::Want &want, unsigned int usec = 0); - /** - * Remove the specified mission stack by stack id - * - * @param id. - * @param {sptr} abilityMs : Ability Manager Service ptr - * @param {time_t} delay : Waiting time for ability to GetAppProcessInfo (milliseconds) - * @return Returns ERR_OK on success, others on failure. - */ - static int RemoveStack(int id, sptr &abilityMs, const time_t &backHmoeDelay = 0, - const time_t &removeDelay = 0); - /** * * @param {string} bundleFilePath : Hap File Name diff --git a/test/systemtest/common/ams/tool/src/st_ability_util.cpp b/test/systemtest/common/ams/tool/src/st_ability_util.cpp index 06c7694f6e585d52bda984a0528084ae3b435342..f8596103f67b2cfbab60121c32b3557bb22cdd9f 100644 --- a/test/systemtest/common/ams/tool/src/st_ability_util.cpp +++ b/test/systemtest/common/ams/tool/src/st_ability_util.cpp @@ -368,22 +368,6 @@ void STAbilityUtil::PullOperatorFromVector(StOperator &ParentOperator, std::vect } } -int STAbilityUtil::RemoveStack( - int id, sptr &abilityMs, const time_t &backHmoeDelay, const time_t &removeDelay) -{ - Want wantEntity; - wantEntity.AddEntity(Want::FLAG_HOME_INTENT_FROM_SYSTEM); - StartAbility(wantEntity, abilityMs); - std::this_thread::sleep_for(std::chrono::milliseconds(backHmoeDelay)); - if (!abilityMs) { - HILOG_INFO("RemoveStack abilityMs nullptr"); - return OHOS::ERR_INVALID_VALUE; - } - int result = abilityMs->RemoveStack(id); - std::this_thread::sleep_for(std::chrono::milliseconds(removeDelay)); - return result; -} - InstallToolStatusReceiver::InstallToolStatusReceiver() { std::cout << "create status receiver instance" << std::endl; diff --git a/test/systemtest/common/fms/common/include/system_test_form_util.h b/test/systemtest/common/fms/common/include/system_test_form_util.h index 2e4f801581eb130a7a32053550d293d204accfe5..3713e8010976c769577487b000be0bbb8fde8cac 100644 --- a/test/systemtest/common/fms/common/include/system_test_form_util.h +++ b/test/systemtest/common/fms/common/include/system_test_form_util.h @@ -258,17 +258,6 @@ public: */ static ErrCode StopServiceAbility(const AAFwk::Want &want, unsigned int usec = 0); - /** - * Remove the specified mission stack by stack id - * - * @param id. - * @param {sptr} abilityMs : Ability Manager Service ptr - * @param {time_t} delay : Waiting time for ability to GetAppProcessInfo (milliseconds) - * @return Returns ERR_OK on success, others on failure. - */ - static int RemoveStack(int id, sptr &abilityMs, const time_t &backHmoeDelay = 0, - const time_t &removeDelay = 0); - /** * * @param {string} bundleFilePath : Hap File Name diff --git a/test/systemtest/common/fms/common/src/system_test_form_util.cpp b/test/systemtest/common/fms/common/src/system_test_form_util.cpp index dd641b8f4d44e171c18700c83a8dd614544c0fc0..6b319666f55788f590dc4fc71f783cf480e7ee57 100755 --- a/test/systemtest/common/fms/common/src/system_test_form_util.cpp +++ b/test/systemtest/common/fms/common/src/system_test_form_util.cpp @@ -315,22 +315,6 @@ void SystemTestFormUtil::CleanMsg(FormEvent &event) return event.Clean(); } -int SystemTestFormUtil::RemoveStack( - int id, sptr &abilityMs, const time_t &backHmoeDelay, const time_t &removeDelay) -{ - Want wantEntity; - wantEntity.AddEntity(Want::FLAG_HOME_INTENT_FROM_SYSTEM); - StartAbility(wantEntity, abilityMs); - std::this_thread::sleep_for(std::chrono::milliseconds(backHmoeDelay)); - if (!abilityMs) { - HILOG_INFO("RemoveStack abilityMs nullptr"); - return OHOS::ERR_INVALID_VALUE; - } - int result = abilityMs->RemoveStack(id); - std::this_thread::sleep_for(std::chrono::milliseconds(removeDelay)); - return result; -} - const std::string MSG_SUCCESS = "[SUCCESS]"; void SystemTestFormUtil::Install(const std::string &bundleFilePath, const InstallFlag installFlag) { diff --git a/tools/test/mock/mock_ability_manager_stub.h b/tools/test/mock/mock_ability_manager_stub.h index e900d080e2862c2c3650edff5669f37d5c2a382c..1e7275d9eb7f7b3c37aa3be88fafe84ee22641f4 100644 --- a/tools/test/mock/mock_ability_manager_stub.h +++ b/tools/test/mock/mock_ability_manager_stub.h @@ -83,7 +83,6 @@ public: MOCK_METHOD2(GetMissionSnapshot, int(const int32_t missionId, MissionPixelMap &missionPixelMap)); MOCK_METHOD1(MoveMissionToTop, int(int32_t missionId)); MOCK_METHOD1(RemoveMission, int(int id)); - MOCK_METHOD1(RemoveStack, int(int id)); MOCK_METHOD1(KillProcess, int(const std::string &bundleName)); MOCK_METHOD2(UninstallApp, int(const std::string &bundleName, int32_t uid));