Skip to content

Commit aff542f

Browse files
committed
travis test 3
1 parent e4fabd9 commit aff542f

File tree

3 files changed

+19
-5
lines changed

3 files changed

+19
-5
lines changed

src/main/java/com/gmail/merikbest2015/ecommerce/controller/RegistrationController.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,8 @@
11
package com.gmail.merikbest2015.ecommerce.controller;
22

3+
import com.gmail.merikbest2015.ecommerce.dto.CaptchaResponseDto;
34
import com.gmail.merikbest2015.ecommerce.dto.RegistrationRequestDto;
4-
import com.gmail.merikbest2015.ecommerce.exception.ApiRequestException;
5-
import com.gmail.merikbest2015.ecommerce.exception.EmailException;
6-
import com.gmail.merikbest2015.ecommerce.exception.InputFieldException;
7-
import com.gmail.merikbest2015.ecommerce.exception.PasswordException;
5+
import com.gmail.merikbest2015.ecommerce.exception.*;
86
import com.gmail.merikbest2015.ecommerce.mapper.UserMapper;
97
import com.gmail.merikbest2015.ecommerce.utils.ControllerUtils;
108
import lombok.RequiredArgsConstructor;
@@ -25,7 +23,7 @@ public class RegistrationController {
2523

2624
@PostMapping
2725
public ResponseEntity<String> registration(@Valid @RequestBody RegistrationRequestDto user, BindingResult bindingResult) {
28-
controllerUtils.captchaValidation(user.getCaptcha());
26+
CaptchaResponseDto captchaResponseDto = controllerUtils.captchaValidation(user.getCaptcha());
2927

3028
if (controllerUtils.isPasswordDifferent(user.getPassword(), user.getPassword2())) {
3129
throw new PasswordException("Passwords do not match.");
@@ -36,6 +34,9 @@ public ResponseEntity<String> registration(@Valid @RequestBody RegistrationReque
3634
if (!userMapper.registerUser(user)) {
3735
throw new EmailException("Email is already used.");
3836
}
37+
if (!captchaResponseDto.isSuccess()) {
38+
throw new CaptchaException("Fill captcha.");
39+
}
3940
return ResponseEntity.ok("User successfully registered.");
4041
}
4142

src/test/java/com/gmail/merikbest2015/ecommerce/controller/RegistrationControllerTest.java

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,17 @@ public void registration_ShouldUserEmailIsExist() throws Exception {
8787
.andExpect(jsonPath("$.emailError").value("Email is already used."));
8888
}
8989

90+
@Test
91+
public void registration_ShouldCaptchaNotFilled() throws Exception {
92+
registrationRequestDto.setCaptcha(null);
93+
94+
mockMvc.perform(post(URL_REGISTRATION_BASIC)
95+
.content(mapper.writeValueAsString(registrationRequestDto))
96+
.contentType(MediaType.APPLICATION_JSON_VALUE))
97+
.andExpect(status().isBadRequest())
98+
.andExpect(jsonPath("$.captchaError").value("Fill captcha."));
99+
}
100+
90101
@Test
91102
public void registration_ShouldInputFieldsAreEmpty() throws Exception {
92103
mockMvc.perform(post(URL_REGISTRATION_BASIC)

src/test/java/com/gmail/merikbest2015/ecommerce/service/Impl/UserServiceImlTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,8 @@ public void loadUserByUsername() {
147147
assertTrue(user.isActive());
148148
}
149149

150+
151+
150152
@Test
151153
public void registerUser() {
152154
User user = new User();

0 commit comments

Comments
 (0)