Merge branch 'master' into master-web
This commit is contained in:
@@ -32,17 +32,19 @@ public final class QuizDtoConverter {
|
||||
.resourceId(quiz.getResourceId())
|
||||
.answerResourceId(getAnswerResourceId(quiz))
|
||||
.title(quiz.getTitle())
|
||||
.category(quiz.getQuizCategory().getTitle())
|
||||
.quizLevelList(convert(quiz.getQuizLevel()))
|
||||
.createdAt(Timestamp.valueOf(quiz.getCreatedAt()).getTime())
|
||||
.build();
|
||||
}
|
||||
|
||||
@Deprecated
|
||||
public static GetQuizResponse convert(final Quiz quiz, final EnumMapperValue enumMapperValue) {
|
||||
return GetQuizResponse.builder()
|
||||
.resourceId(quiz.getResourceId())
|
||||
.answerResourceId(getAnswerResourceId(quiz))
|
||||
.title(quiz.getTitle())
|
||||
.category(enumMapperValue)
|
||||
.category(quiz.getQuizCategory().getTitle())
|
||||
.quizLevelList(convert(quiz.getQuizLevel()))
|
||||
.createdAt(Timestamp.valueOf(quiz.getCreatedAt()).getTime())
|
||||
.build();
|
||||
|
||||
@@ -19,7 +19,7 @@ public class GetQuizResponse {
|
||||
private final String title;
|
||||
|
||||
@JsonInclude(JsonInclude.Include.NON_NULL)
|
||||
private final EnumMapperValue category;
|
||||
private final String category;
|
||||
|
||||
private final List<String> quizLevelList;
|
||||
private final long createdAt;
|
||||
|
||||
@@ -56,7 +56,7 @@ public class QuizService {
|
||||
final Page<Quiz> quizPage = quizRepository.findByQuizCategoryIsAndIsEnableTrue(getQuizRequest.getCategory(), pageRequest);
|
||||
|
||||
final List<GetQuizResponse> quizResponseList = quizPage.getContent().stream()
|
||||
.map(v -> QuizDtoConverter.convert(v, enumMapperFactory.getElement(EnumMapperKey.QUIZ_CATEGORY, v.getQuizCategory())))
|
||||
.map(QuizDtoConverter::convert)
|
||||
.collect(Collectors.toList());
|
||||
|
||||
return GetQuizResponseHolder.builder()
|
||||
@@ -121,7 +121,7 @@ public class QuizService {
|
||||
final Page<Quiz> quizPage = quizRepository.search(condition, pageRequest);
|
||||
|
||||
final List<GetQuizResponse> quizResponseList = quizPage.getContent().stream()
|
||||
.map(v -> QuizDtoConverter.convert(v, enumMapperFactory.getElement(EnumMapperKey.QUIZ_CATEGORY, v.getQuizCategory())))
|
||||
.map(QuizDtoConverter::convert)
|
||||
.collect(Collectors.toList());
|
||||
|
||||
return GetQuizResponseHolder.builder()
|
||||
|
||||
@@ -51,7 +51,7 @@ class QuizControllerTest {
|
||||
final GetQuizResponse quizResponse = GetQuizResponse.builder()
|
||||
.title("quiz")
|
||||
.quizLevelList(Arrays.asList(QuizLevel.JUNIOR.name(), QuizLevel.SENIOR.name()))
|
||||
.category(enumMapperValue(QuizCategory.JAVA))
|
||||
.category(enumMapperValue(QuizCategory.JAVA).getTitle())
|
||||
.build();
|
||||
doReturn(quizResponse).when(quizService).getQuiz(resourceId);
|
||||
|
||||
@@ -67,7 +67,7 @@ class QuizControllerTest {
|
||||
|
||||
assertThat(getQuizResult.getResourceId()).isEqualTo(quizResponse.getResourceId());
|
||||
assertThat(getQuizResult.getTitle()).isEqualTo(quizResponse.getTitle());
|
||||
assertThat(getQuizResult.getCategory().getCode()).isEqualTo(quizResponse.getCategory().getCode());
|
||||
assertThat(getQuizResult.getCategory()).isEqualTo(quizResponse.getCategory());
|
||||
assertThat(getQuizResult.getCreatedAt()).isEqualTo(quizResponse.getCreatedAt());
|
||||
}
|
||||
|
||||
@@ -83,7 +83,7 @@ class QuizControllerTest {
|
||||
final GetQuizResponse quizResponse = GetQuizResponse.builder()
|
||||
.title("quiz")
|
||||
.quizLevelList(Arrays.asList(QuizLevel.JUNIOR.name(), QuizLevel.SENIOR.name()))
|
||||
.category(enumMapperValue(QuizCategory.JAVA))
|
||||
.category(enumMapperValue(QuizCategory.JAVA).getTitle())
|
||||
.build();
|
||||
|
||||
// when
|
||||
@@ -108,7 +108,7 @@ class QuizControllerTest {
|
||||
final GetQuizResponse quizResponse = GetQuizResponse.builder()
|
||||
.title("quiz")
|
||||
.quizLevelList(Arrays.asList(QuizLevel.JUNIOR.name(), QuizLevel.SENIOR.name()))
|
||||
.category(enumMapperValue(QuizCategory.JAVA))
|
||||
.category(enumMapperValue(QuizCategory.JAVA).getTitle())
|
||||
.build();
|
||||
|
||||
final Pageable pageable = PageRequest.of(page, size);
|
||||
@@ -198,7 +198,7 @@ class QuizControllerTest {
|
||||
final GetQuizResponse quizResponse = GetQuizResponse.builder()
|
||||
.title("quiz")
|
||||
.quizLevelList(Arrays.asList(QuizLevel.JUNIOR.name(), QuizLevel.SENIOR.name()))
|
||||
.category(enumMapperValue(QuizCategory.JAVA))
|
||||
.category(enumMapperValue(QuizCategory.JAVA).getTitle())
|
||||
.build();
|
||||
|
||||
// when
|
||||
@@ -222,7 +222,7 @@ class QuizControllerTest {
|
||||
final GetQuizResponse quizResponse = GetQuizResponse.builder()
|
||||
.title("quiz")
|
||||
.quizLevelList(Arrays.asList(QuizLevel.JUNIOR.name(), QuizLevel.SENIOR.name()))
|
||||
.category(enumMapperValue(QuizCategory.JAVA))
|
||||
.category(enumMapperValue(QuizCategory.JAVA).getTitle())
|
||||
.build();
|
||||
|
||||
final Pageable pageable = PageRequest.of(page, size);
|
||||
|
||||
@@ -8,7 +8,6 @@ import com.mangkyu.employment.interview.app.quiz.entity.Quiz;
|
||||
import com.mangkyu.employment.interview.enums.common.EnumMapperType;
|
||||
import com.mangkyu.employment.interview.enums.common.EnumMapperValue;
|
||||
import com.mangkyu.employment.interview.testutils.EntityCreationUtils;
|
||||
import org.apache.commons.lang3.StringUtils;
|
||||
import org.junit.jupiter.api.Test;
|
||||
|
||||
import java.sql.Timestamp;
|
||||
@@ -66,7 +65,7 @@ class QuizDtoConverterTest {
|
||||
assertThat(result.getTitle()).isEqualTo(quiz.getTitle());
|
||||
assertThat(result.getQuizLevelList().size()).isEqualTo(quiz.getQuizLevel().size());
|
||||
assertThat(result.getCreatedAt()).isEqualTo(Timestamp.valueOf(quiz.getCreatedAt()).getTime());
|
||||
assertThat(result.getCategory().getCode()).isEqualTo(enumMapperValue(quiz.getQuizCategory()).getCode());
|
||||
assertThat(result.getCategory()).isEqualTo(quiz.getQuizCategory().getTitle());
|
||||
assertThat(result.getAnswerResourceId()).isEqualTo(answer.getResourceId());
|
||||
}
|
||||
|
||||
@@ -83,7 +82,7 @@ class QuizDtoConverterTest {
|
||||
assertThat(result.getTitle()).isEqualTo(quiz.getTitle());
|
||||
assertThat(result.getQuizLevelList().size()).isEqualTo(quiz.getQuizLevel().size());
|
||||
assertThat(result.getCreatedAt()).isEqualTo(Timestamp.valueOf(quiz.getCreatedAt()).getTime());
|
||||
assertThat(result.getCategory().getCode()).isEqualTo(enumMapperValue(quiz.getQuizCategory()).getCode());
|
||||
assertThat(result.getCategory()).isEqualTo(quiz.getQuizCategory().getTitle());
|
||||
assertThat(result.getAnswerResourceId()).isNull();
|
||||
}
|
||||
|
||||
@@ -102,7 +101,7 @@ class QuizDtoConverterTest {
|
||||
assertThat(result.getTitle()).isEqualTo(quiz.getTitle());
|
||||
assertThat(result.getQuizLevelList().size()).isEqualTo(quiz.getQuizLevel().size());
|
||||
assertThat(result.getCreatedAt()).isEqualTo(Timestamp.valueOf(quiz.getCreatedAt()).getTime());
|
||||
assertThat(result.getCategory()).isNull();
|
||||
assertThat(result.getCategory()).isEqualTo(quiz.getQuizCategory().getTitle());
|
||||
assertThat(result.getAnswerResourceId()).isEqualTo(answer.getResourceId());
|
||||
}
|
||||
|
||||
@@ -119,7 +118,7 @@ class QuizDtoConverterTest {
|
||||
assertThat(result.getTitle()).isEqualTo(quiz.getTitle());
|
||||
assertThat(result.getQuizLevelList().size()).isEqualTo(quiz.getQuizLevel().size());
|
||||
assertThat(result.getCreatedAt()).isEqualTo(Timestamp.valueOf(quiz.getCreatedAt()).getTime());
|
||||
assertThat(result.getCategory()).isNull();
|
||||
assertThat(result.getCategory()).isEqualTo(quiz.getQuizCategory().getTitle());
|
||||
assertThat(result.getAnswerResourceId()).isNull();
|
||||
}
|
||||
|
||||
|
||||
@@ -1,7 +1,5 @@
|
||||
package com.mangkyu.employment.interview.app.quiz.service;
|
||||
|
||||
import com.mangkyu.employment.interview.erros.errorcode.CommonErrorCode;
|
||||
import com.mangkyu.employment.interview.erros.exception.RestApiException;
|
||||
import com.mangkyu.employment.interview.app.quiz.dto.*;
|
||||
import com.mangkyu.employment.interview.app.quiz.entity.Quiz;
|
||||
import com.mangkyu.employment.interview.app.quiz.repository.QuizRepository;
|
||||
@@ -13,6 +11,8 @@ import com.mangkyu.employment.interview.enums.common.EnumMapperValue;
|
||||
import com.mangkyu.employment.interview.enums.factory.EnumMapperFactory;
|
||||
import com.mangkyu.employment.interview.enums.value.QuizCategory;
|
||||
import com.mangkyu.employment.interview.enums.value.QuizLevel;
|
||||
import com.mangkyu.employment.interview.erros.errorcode.CommonErrorCode;
|
||||
import com.mangkyu.employment.interview.erros.exception.RestApiException;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import org.junit.jupiter.api.extension.ExtendWith;
|
||||
import org.mockito.InjectMocks;
|
||||
@@ -139,7 +139,7 @@ class QuizServiceTest {
|
||||
// then
|
||||
assertThat(result.getResourceId()).isEqualTo(quiz.getResourceId());
|
||||
assertThat(result.getTitle()).isEqualTo(quiz.getTitle());
|
||||
assertThat(result.getCategory()).isEqualTo(enumMapperFactory.getElement(EnumMapperKey.QUIZ_CATEGORY, quiz.getQuizCategory()));
|
||||
assertThat(result.getCategory()).isEqualTo(quiz.getQuizCategory().getTitle());
|
||||
assertThat(result.getQuizLevelList().size()).isEqualTo(quiz.getQuizLevel().size());
|
||||
assertThat(result.getCreatedAt()).isEqualTo(Timestamp.valueOf(quiz.getCreatedAt()).getTime());
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user