YEL-139 [test] 구글 결제 관련 명세 수정 및 일부 코드 로직 수정

team-yello · Aug 27, 2023 · 9461884 · 9461884
2 parents 483d214 + a195120
commit 9461884
Show file tree

Hide file tree

Showing 11 changed files with 47 additions and 19 deletions.
diff --git a/src/docs/asciidoc/google.adoc b/src/docs/asciidoc/google.adoc
@@ -12,8 +12,8 @@ include::{snippets}/api/v1/purchase/verifyGoogleSubscriptionTransaction/http-res
 
 === 요청
 
-include::{snippets}/api/v1/friend/findAllRecommendSchoolFriends/http-request.adoc[]
+include::{snippets}/api/v1/purchase/verifyGoogleTicketTransaction/http-request.adoc[]
 
 === 응답
 
-include::{snippets}/api/v1/purchase/verifyAppleTicketTransaction/http-response.adoc[]
+include::{snippets}/api/v1/purchase/verifyGoogleTicketTransaction/http-response.adoc[]
diff --git a/src/main/java/com/yello/server/domain/authorization/service/AuthManagerImpl.java b/src/main/java/com/yello/server/domain/authorization/service/AuthManagerImpl.java
@@ -71,7 +71,7 @@ public void validateSignupRequest(SignUpRequest signUpRequest) {
                 throw new UserConflictException(UUID_CONFLICT_USER_EXCEPTION);
             });
 
-        userRepository.findByYelloId(signUpRequest.yelloId())
+        userRepository.findByYelloIdNotFiltered(signUpRequest.yelloId())
             .ifPresent(action -> {
                 throw new UserConflictException(YELLOID_CONFLICT_USER_EXCEPTION);
             });
@@ -81,7 +81,7 @@ public void validateSignupRequest(SignUpRequest signUpRequest) {
     public Boolean renewUserData(User user) {
         final Long userId = user.getId();
 
-        if (user.getDeletedAt()!=null) {
+        if (user.getDeletedAt() != null) {
             user.renew();
 
             friendRepository.findAllByUserIdNotFiltered(userId)

diff --git a/src/main/java/com/yello/server/domain/authorization/service/AuthService.java b/src/main/java/com/yello/server/domain/authorization/service/AuthService.java
@@ -80,7 +80,7 @@ public Boolean isYelloIdDuplicated(String yelloId) {
             throw new AuthBadRequestException(YELLOID_REQUIRED_EXCEPTION);
         }
 
-        return userRepository.findByYelloId(yelloId).isPresent();
+        return userRepository.findByYelloIdNotFiltered(yelloId).isPresent();
     }
 
     @Transactional
@@ -103,7 +103,7 @@ public SignUpResponse signUp(SignUpRequest signUpRequest) {
 
     @Transactional
     public void recommendUser(String recommendYelloId, String userYelloId) {
-        if (recommendYelloId!=null && !recommendYelloId.isEmpty()) {
+        if (recommendYelloId != null && !recommendYelloId.isEmpty()) {
             User recommendedUser = userRepository.getByYelloId(recommendYelloId);
             User user = userRepository.getByYelloId(userYelloId);
 

diff --git a/src/main/java/com/yello/server/domain/purchase/dto/apple/AppleOrderResponse.java b/src/main/java/com/yello/server/domain/purchase/dto/apple/AppleOrderResponse.java
@@ -5,8 +5,7 @@
 @Builder
 public record AppleOrderResponse(
     int appAppleId,
-    String environment,
-    String JWSTransaction
+    String environment
 ) {
 
 }
diff --git a/src/main/java/com/yello/server/domain/user/repository/UserJpaRepository.java b/src/main/java/com/yello/server/domain/user/repository/UserJpaRepository.java
@@ -28,6 +28,10 @@ public interface UserJpaRepository extends JpaRepository<User, Long> {
         "and u.deletedAt is null")
     Optional<User> findByYelloId(@Param("yelloId") String yelloId);
 
+    @Query("select u from User u " +
+        "where u.yelloId = :yelloId")
+    Optional<User> findByYelloIdNotFiltered(@Param("yelloId") String yelloId);
+
     @Query("select u from User u " +
         "where u.group.id = :groupId " +
         "and u.deletedAt is null")

diff --git a/src/main/java/com/yello/server/domain/user/repository/UserRepository.java b/src/main/java/com/yello/server/domain/user/repository/UserRepository.java
@@ -20,6 +20,8 @@ public interface UserRepository {
 
     Optional<User> findByYelloId(String yelloId);
 
+    Optional<User> findByYelloIdNotFiltered(String yelloId);
+
     User getByYelloId(String yelloId);
 
     Optional<User> findByDeviceToken(String deviceToken);

diff --git a/src/main/java/com/yello/server/domain/user/repository/UserRepositoryImpl.java b/src/main/java/com/yello/server/domain/user/repository/UserRepositoryImpl.java
@@ -57,6 +57,11 @@ public Optional<User> findByYelloId(String yelloId) {
         return userJpaRepository.findByYelloId(yelloId);
     }
 
+    @Override
+    public Optional<User> findByYelloIdNotFiltered(String yelloId) {
+        return userJpaRepository.findByYelloIdNotFiltered(yelloId);
+    }
+
     @Override
     public User getByYelloId(String yelloId) {
         return userJpaRepository.findByYelloId(yelloId)

diff --git a/src/main/java/com/yello/server/global/HealthCheckController.java b/src/main/java/com/yello/server/global/HealthCheckController.java
@@ -12,4 +12,9 @@ public class HealthCheckController {
     public String healthCheck() {
         return "Yell:o world!";
     }
+
+    @GetMapping("/abc")
+    public void text() throws Exception {
+        throw new Exception("Abc");
+    }
 }
diff --git a/src/main/java/com/yello/server/infrastructure/slack/factory/SlackWebhookMessageFactory.java b/src/main/java/com/yello/server/infrastructure/slack/factory/SlackWebhookMessageFactory.java
@@ -70,35 +70,40 @@ private List<SlackAttachment> generateSlackPurchaseAttachment(
         HttpServletRequest request
     ) throws IOException {
         final SlackAttachment slackAttachment = new SlackAttachment()
-            .setColor("green")
+            .setColor("good")
             .setTitle(PURCHASE_TITLE)
             .setTitleLink(request.getContextPath())
             .setText(PURCHASE_TITLE)
-            .setColor("green")
             .setFields(generateSlackFieldList(request));
         return Collections.singletonList(slackAttachment);
     }
 
     private List<SlackField> generateSlackFieldList(
         HttpServletRequest request
     ) throws IOException {
-        final String token = request.getHeader(HttpHeaders.AUTHORIZATION).substring("Bearer ".length());
+        final String token =
+            request.getHeader(HttpHeaders.AUTHORIZATION).substring("Bearer ".length());
         final Long userId = tokenProvider.getUserId(token);
         final Optional<User> user = userRepository.findById(userId);
         final String yelloId = user.isPresent() ? user.get().getYelloId() : "null";
         final String deviceToken = user.isPresent() ? user.get().getDeviceToken() : "null";
 
-        String userInfo = String.format("userId : %d %nyelloId : %s %ndeviceToken : %s", userId, yelloId, deviceToken);
+        String userInfo =
+            String.format("userId : %d %nyelloId : %s %ndeviceToken : %s", userId, yelloId,
+                deviceToken);
 
         return Arrays.asList(
             new SlackField().setTitle("Request URL").setValue(request.getRequestURL().toString()),
-            new SlackField().setTitle("Request Time").setValue(TimeFactory.toDateFormattedString(LocalDateTime.now())),
+            new SlackField().setTitle("Request Time")
+                .setValue(TimeFactory.toDateFormattedString(LocalDateTime.now())),
             new SlackField().setTitle("Request IP").setValue(request.getRemoteAddr()),
-            new SlackField().setTitle("Request User-Agent").setValue(request.getHeader(HttpHeaders.USER_AGENT)),
+            new SlackField().setTitle("Request User-Agent")
+                .setValue(request.getHeader(HttpHeaders.USER_AGENT)),
             new SlackField().setTitle("인증/인가 정보 - Authorization")
                 .setValue(request.getHeader(HttpHeaders.AUTHORIZATION)),
             new SlackField().setTitle("Request Body")
-                .setValue(StreamUtils.copyToString(request.getInputStream(), StandardCharsets.UTF_8)),
+                .setValue(
+                    StreamUtils.copyToString(request.getInputStream(), StandardCharsets.UTF_8)),
             new SlackField().setTitle("인증/인가 정보 - 유저").setValue(userInfo)
         );
     }

diff --git a/src/test/java/com/yello/server/domain/authorization/FakeAuthManager.java b/src/test/java/com/yello/server/domain/authorization/FakeAuthManager.java
@@ -76,7 +76,7 @@ public void validateSignupRequest(SignUpRequest signUpRequest) {
                 throw new UserConflictException(UUID_CONFLICT_USER_EXCEPTION);
             });
 
-        userRepository.findByYelloId(signUpRequest.yelloId())
+        userRepository.findByYelloIdNotFiltered(signUpRequest.yelloId())
             .ifPresent(action -> {
                 throw new UserConflictException(YELLOID_CONFLICT_USER_EXCEPTION);
             });
@@ -86,7 +86,7 @@ public void validateSignupRequest(SignUpRequest signUpRequest) {
     public Boolean renewUserData(User user) {
         final Long userId = user.getId();
 
-        if (user.getDeletedAt()!=null) {
+        if (user.getDeletedAt() != null) {
             user.renew();
 
             friendRepository.findAllByUserIdNotFiltered(userId)

diff --git a/src/test/java/com/yello/server/domain/user/FakeUserRepository.java b/src/test/java/com/yello/server/domain/user/FakeUserRepository.java
@@ -18,12 +18,12 @@ public class FakeUserRepository implements UserRepository {
 
     @Override
     public User save(User user) {
-        if (user.getId()!=null && user.getId() > id) {
+        if (user.getId() != null && user.getId() > id) {
             id = user.getId();
         }
 
         User newUser = User.builder()
-            .id(user.getId()==null ? ++id : user.getId())
+            .id(user.getId() == null ? ++id : user.getId())
             .recommendCount(0L)
             .name(user.getName())
             .yelloId(user.getYelloId())
@@ -82,6 +82,14 @@ public boolean existsByUuid(String uuid) {
 
     @Override
     public Optional<User> findByYelloId(String yelloId) {
+        return data.stream()
+            .filter(user -> user.getDeletedAt() == null)
+            .filter(user -> user.getYelloId().equals(yelloId))
+            .findFirst();
+    }
+
+    @Override
+    public Optional<User> findByYelloIdNotFiltered(String yelloId) {
         return data.stream()
             .filter(user -> user.getYelloId().equals(yelloId))
             .findFirst();