feat(backend): fix unittest bug

thoughtworks · Feb 17, 2023 · 2182a17 · 2182a17
1 parent 163e4f3
commit 2182a17
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/backend/src/main/kotlin/metrik/project/rest/applicationservice/ProjectApplicationService.kt b/backend/src/main/kotlin/metrik/project/rest/applicationservice/ProjectApplicationService.kt
@@ -46,6 +46,7 @@ class ProjectApplicationService {
 
     fun updateProjectName(projectId: String, updateProjectRequest: UpdateProjectRequest): ProjectResponse {
         val projectName = updateProjectRequest.projectName
+        verifyProjectNameNotDuplicate(projectName)
         val project = projectRepository.findById(projectId)
         project.name = projectName
         return ProjectResponse(projectRepository.save(project))

diff --git a/...d/src/test/kotlin/metrik/project/rest/applicationservice/ProjectApplicationServiceTest.kt b/...d/src/test/kotlin/metrik/project/rest/applicationservice/ProjectApplicationServiceTest.kt
@@ -105,10 +105,10 @@ internal class ProjectApplicationServiceTest {
 
     @Test
     fun `update project will fail if project name already exist`() {
-        val newProjectName = "new project name"
+        val newProjectName = UpdateProjectRequest("new project name")
         every { projectRepository.findById(projectId) } returns expectedProject
         every { projectRepository.save(expectedProject) } returns expectedProject
-        every { projectRepository.existWithGivenName(newProjectName) } returns true
+        every { projectRepository.existWithGivenName(newProjectName.projectName) } returns true
 
         val exception = assertThrows<ProjectNameDuplicateException> {
             projectApplicationService.updateProjectName(projectId, newProjectName)