From db81565db68b47adf5fddc09dcfad44132432c07 Mon Sep 17 00:00:00 2001 From: Nilesh Sarupriya <20905988+nsarupr@users.noreply.github.com> Date: Wed, 6 Nov 2024 18:27:10 +0530 Subject: [PATCH 1/2] use spybean for SessionUserService --- .../server/solutions/ce/ActionExecutionSolutionCEImplTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/server/appsmith-server/src/test/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImplTest.java b/app/server/appsmith-server/src/test/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImplTest.java index c6c1eb95610a..603142af9435 100644 --- a/app/server/appsmith-server/src/test/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImplTest.java +++ b/app/server/appsmith-server/src/test/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImplTest.java @@ -109,7 +109,7 @@ class ActionExecutionSolutionCEImplTest { @MockBean ApplicationService applicationService; - @MockBean + @SpyBean SessionUserService sessionUserService; @MockBean From 87131cbe02fa4e13ef4a88288ffa64a27846c9fc Mon Sep 17 00:00:00 2001 From: Nilesh Sarupriya <20905988+nsarupr@users.noreply.github.com> Date: Thu, 7 Nov 2024 11:46:04 +0530 Subject: [PATCH 2/2] remove unused method populateExecuteActionDTOWithUserId --- .../solutions/ce/ActionExecutionSolutionCEImpl.java | 9 --------- 1 file changed, 9 deletions(-) diff --git a/app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImpl.java b/app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImpl.java index c8db695056cf..38c8664141d0 100644 --- a/app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImpl.java +++ b/app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImpl.java @@ -265,15 +265,6 @@ private Mono populateExecuteActionDTO(ExecuteActionDTO execute })); } - /** - * Populates the requestParams with logged in userId. - * If the user is not logged in, set the parameter as anonymousUserId - * - */ - protected Mono populateExecuteActionDTOWithUserId(ExecuteActionDTO executeActionDTO) { - return Mono.just(executeActionDTO); - } - /** * Executes the action(queries) by creating executeActionDTO and sending it to the plugin for further execution *