diff --git a/app/server/appsmith-server/src/test/java/com/appsmith/server/services/UserServiceWithDisabledSignupTest.java b/app/server/appsmith-server/src/test/java/com/appsmith/server/services/UserServiceWithDisabledSignupTest.java index 9e8165255a..5953b5b7c1 100644 --- a/app/server/appsmith-server/src/test/java/com/appsmith/server/services/UserServiceWithDisabledSignupTest.java +++ b/app/server/appsmith-server/src/test/java/com/appsmith/server/services/UserServiceWithDisabledSignupTest.java @@ -1,5 +1,6 @@ package com.appsmith.server.services; +import com.appsmith.server.configurations.CommonConfig; import com.appsmith.server.configurations.WithMockAppsmithUser; import com.appsmith.server.domains.LoginSource; import com.appsmith.server.domains.User; @@ -12,9 +13,10 @@ import lombok.extern.slf4j.Slf4j; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.Mockito; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.boot.test.context.SpringBootTest; -import org.springframework.test.annotation.DirtiesContext; +import org.springframework.boot.test.mock.mockito.SpyBean; import org.springframework.test.context.junit.jupiter.SpringExtension; import reactor.core.publisher.Mono; import reactor.test.StepVerifier; @@ -26,8 +28,7 @@ import static org.assertj.core.api.Assertions.assertThat; @Slf4j @ExtendWith(SpringExtension.class) -@SpringBootTest(properties = {"signup.disabled = true", "admin.emails = dummy_admin@appsmith.com,dummy2@appsmith.com"}) -@DirtiesContext +@SpringBootTest public class UserServiceWithDisabledSignupTest { @Autowired @@ -48,11 +49,17 @@ public class UserServiceWithDisabledSignupTest { @Autowired PermissionGroupRepository permissionGroupRepository; + @SpyBean + CommonConfig commonConfig; + Mono userMono; @BeforeEach public void setup() { userMono = userService.findByEmail("usertest@usertest.com"); + Mockito.when(commonConfig.isSignupDisabled()).thenReturn(Boolean.TRUE); + Mockito.when(commonConfig.getAdminEmails()) + .thenReturn(Set.of("dummy_admin@appsmith.com", "dummy2@appsmith.com")); } @Test diff --git a/app/server/appsmith-server/src/test/java/com/appsmith/server/services/ce/ApplicationServiceCETest.java b/app/server/appsmith-server/src/test/java/com/appsmith/server/services/ce/ApplicationServiceCETest.java index ac6a985efd..c24ccae98c 100644 --- a/app/server/appsmith-server/src/test/java/com/appsmith/server/services/ce/ApplicationServiceCETest.java +++ b/app/server/appsmith-server/src/test/java/com/appsmith/server/services/ce/ApplicationServiceCETest.java @@ -34,6 +34,7 @@ import com.appsmith.server.domains.User; import com.appsmith.server.domains.Workspace; import com.appsmith.server.dtos.ActionCollectionDTO; import com.appsmith.server.dtos.ApplicationAccessDTO; +import com.appsmith.server.dtos.ApplicationJson; import com.appsmith.server.dtos.ApplicationPagesDTO; import com.appsmith.server.dtos.PageDTO; import com.appsmith.server.dtos.UserHomepageDTO; @@ -287,28 +288,32 @@ public class ApplicationServiceCETest { .getDefaultEnvironmentId(workspaceId, environmentPermission.getExecutePermission()) .block(); - gitConnectedApp = new Application(); - gitConnectedApp.setWorkspaceId(workspaceId); + Application gitConnectedApp1 = new Application(); + gitConnectedApp1.setWorkspaceId(workspaceId); GitApplicationMetadata gitData = new GitApplicationMetadata(); gitData.setBranchName("testBranch"); gitData.setDefaultBranchName("testBranch"); gitData.setRepoName("testRepo"); gitData.setRemoteUrl("git@test.com:user/testRepo.git"); gitData.setRepoName("testRepo"); - gitConnectedApp.setGitApplicationMetadata(gitData); + gitConnectedApp1.setGitApplicationMetadata(gitData); // This will be altered in update app by branch test - gitConnectedApp.setName("gitConnectedApp"); - gitConnectedApp = applicationPageService - .createApplication(gitConnectedApp) + gitConnectedApp1.setName("gitConnectedApp"); + Application newGitConnectedApp = applicationPageService + .createApplication(gitConnectedApp1) .flatMap(application -> { application.getGitApplicationMetadata().setDefaultApplicationId(application.getId()); return applicationService.save(application); }) - // Assign the branchName to all the resources connected to the application - .flatMap(application -> - exportApplicationService.exportApplicationById(application.getId(), gitData.getBranchName())) - .flatMap(applicationJson -> importApplicationService.importApplicationInWorkspaceFromGit( - workspaceId, applicationJson, gitConnectedApp.getId(), gitData.getBranchName())) + .block(); + + // Assign the branchName to all the resources connected to the application + ApplicationJson gitConnectedApplicationJson = exportApplicationService + .exportApplicationById(newGitConnectedApp.getId(), gitData.getBranchName()) + .block(); + gitConnectedApp = importApplicationService + .importApplicationInWorkspaceFromGit( + workspaceId, gitConnectedApplicationJson, newGitConnectedApp.getId(), gitData.getBranchName()) .block(); testPlugin = pluginService.findByPackageName("restapi-plugin").block();