[BE] pull request test (#625)

* refactor: 비로그인 시 pin detail에서 image 추가 버튼 숨기기 * [FE] Feat/#554 search 지도 검색 기능 구현 (#555) * feat: SearchBar 컴포넌트 구현 * feat: Search 페이지 구현 * feat: 홈페이지에 searchBar 적용 * refactor: 불필요한 memo 제거 * [FE] refactor/#450 모아보기 기능 비회원도 가능하도록 수정 (#516) * refactor: 모아보기 기능 비회원도 가능하도록 수정 * fix: 화면 터지던 에러 해결 * fix: 모아보기 삭제 하는 기능 에러 해결 * refactor: 비로그인 시 모아보기 페이지에서 홈으로 이동하면 모아보기 삭제되도록 수정 * refactor: 충돌 해결 과정에서 롤백된 Navbar 전체 코드 수정 * refactor: 네이밍 수정이 필요한 부분들 적용 * refactor: TopicInfo에서는 모아보기 담기 기능 제한 및 토스트 알림 적용 * refactor: NavBar 롤백하며 빼먹은 코드 useNavigator에 적용 * refactor: 필요없어진 컴포넌트 삭제 * fix: pinDetail 페이지 split 에러 해결 * refactor: 비로그인 때 모아보기 할 시 toast 알람 뜨도록 변경 * refactor: topicInfo에서는 모아보기 클릭 안되도록 수정 * refactor: 모아보기 highlight 설정 변경 * chore: eslint 설정 적용 (#557) * [FE] Refactor/eslint ESLint 추가 적용 (#559) * style: api, apiHooks, mocks, pages eslint 적용 * chore: github actions 브랜치 수정 * chore: zustand 설치 * feat/#564 poi 적용 (#566) * feat: 자동완성 컴포넌트 구현 * feat: getPoi 구현 * feat: 자동완성 컴포넌트 구현 * feat: 핀 추가 페이지에 자동완성 입력창 추가 * refactor: 줌 레벨 설정 조정 및 타입 추가 * refactor: tmap api key 추가 * feat: Poi 타입 설정 * feat: 자동완성 debounce 적용 * refactor: 리뷰 반영 * refactor: 2차 리뷰 반영 * hotfix: dotenv 추가 (#574) * hotfix: env 이름 형식 통일 (#575) * hotfix: DotenvWebpackPlugin추가 (#576) * fix: require 수정 (#577) * [FE] Feature/#560 메인 페이지에 swiper 및 배너 적용 (#569) * feat: Swiper 구현 * remove: 사용하지 않는 tooltip 컴포넌트 제거 * design: swiper 버튼 디자인 수정 * feat: banner 메인페이지에 도입 * design: swiper tabbox color 제거 및 swiper 가운데 정렬 swiper cursor pointer 옵션 또한 추가한다. * refactor: swiper as 프롭으로 시멘틱 태그 지정 기능 추가 * design: topicCard UI 업데이트 * design: 배너 1140 가로 크기 대응 * design: 스크롤바 영역 제거 목표한 브라우저 환경에서 동작 확인 * feat: topicCard 컨테이너에 swiper 도입 * design: Home 콘텐츠 가로 1140px로 확장 * feat: 뷰포트가 모바일 사이즈 744px 미만일때 Text 크기를 한 단계씩 낮추는 기능 추가 * refactor: scroll bar 보이도록 롤백 * refactor: topicCard 반응형 로직 수정 * feat: media query 분기점에 따라 tab 갯수 조절하는 기능 추가 * design: topicCard 이미지 최대 크기 조정 및 모아보기 버튼 위치 수정 * refactor: swiper 버튼은 모바일 사이즈에서 보이지 않도록 일괄 지정 * design: 반응형 페이지 width 수정 * chore: 배너 이미지 확장자 변경 webp, 그에 따른 설정 * refactor: swiper tab 반응형 상태에선 height 자식에 따라 지정되도록 변경 * refactor: topicCard swiper media query 추가 * design: layout padding 제거 (반응형 오류 해결 위함) * feat: banner 반응형 기능 및 라우팅 기능 추가 * refactor: 반응형 조건에 따라 부모에 가로 세로 크기를 맞추되 한계점 지정 * design: 반응형 상태에서 layout margin이 어색한 문제 해결 * refactor: swipe value 상수화 * fix: 탭 갯수와 element 갯수가 같을 경우 Tabbox 숨김 처리 Tabbox 갯수가 elementsTab 속성을 고려하지 않고 Tab 갯수로만 고정되던 오류를 해결한다. * design: topicCard max width 값 지정 * refactor: 불필요한 import 제거 * refactor: grid 컴포넌트 생성 및 bookmark, profile에 적용 * design: Home, Bookmark, Profile에 미디어텍스트 적용 * design: 전체보기 및 검색결과 페이지에 미디어 텍스트 적용 * refactor: 검색결과 topicCard 리스트를 Grid로 설정 * rename: 컴포넌트 파일명 파스칼 케이스로 변경 * design: media Space 컴포넌트 생성 및 반응형 space 적용 * design: 모바일 환경에서 하단 일부가 네브바에 가리는 오류 수정 * refactor: 토픽카드에서 즐겨찾기 버튼 제거 * refactor: 불필요한 import 제거 * [FE] Refactor/#572 핀 이미지 태그로 교체 및 아이콘 소스 검토 (#573) * refactor: pin 을 image url 방식에서 tag 방식으로 변경 * rename: topicCard 모아보기 및 즐겨찾기 아이콘 변경 * rename: topicInfo 모아보기 및 공유하기 아이콘 수정 * refactor: 수정하기 버튼 및 아이콘 일괄 수정 * refactor: 기본 토픽 이미지 및 프로필 이미지 수정 이미지 url 을 mapbefine s3로 지정한다. * rename: search bar 아이콘 수정 * refactor: favicon 경로 s3 로 변경 * refactor: 이미지 공통 컴포넌트 지정 및 중복 코드 제거 * rename: 대소문자 추적 오류로 임시로 네이밍 변경 * Hotfix/swiper UI (#579) * design: 배너, 토픽카드 이미지 오류 수정 머지 과정에서 일부 충돌 사항을 해결한다. * refactor: swiper 영역은 콘텐츠가 클릭되지 않던 불편함 해소 * rename: git 파일 대소문자 추적 불가 오류로 인해 임시로 바꿨던 파일명 롤백 * [FE] Feature/#550 토픽 이미지 수정 및 핀 이미지 삭제 기능 (#583) * feat: 핀 이미지 삭제 기능 구현 * feat: 토픽 사진 수정 기능 예비 * refactor: 핀 이미지 삭제 시 재렌더링 안되던 문제 해결 * refactor: 핀 수정 기능 명세 연결 및 구현 * refactor: 지도 사진 수정할 때 사진만 수정되도록 변경 * refactor: 함수 이름 변경 및 지도 사진 수정 타이틀 작성 * refactor: 지도 수정 버튼 위치 이동 * [FE] Feat/#588 현재 위치 이동 버튼 (#589) * feat: 지도 위 현재 위치로 버튼 추가 * design: 현재 위치 아이콘 변경 및 위치 변경 * refactor: Toast로 사용자에게 위치 정보 상태를 명시 --------- Co-authored-by: semnil5202 <semnil5202@gmail.com> * [FE] Refactor/#578 지도 핀 Boundary 조정 (#590) * refactor: mobile 사이즈에서는 지도 축소 제한을 한 단계 낮추도록 변경 * refactor: mapInstance zustand 전역상태로 분리 * fix: 지도가 브라우저 넓이에 맞지 않는 오류 수정 * design: 현재 위치 조정 버튼 크기 확장 * refactor: 현재 위치 정보 가져오기 로직 수정 * Hotfix/current position (#594) * feat: 현재 위치 아이콘 추가 * refactor: 사용자 현재 위치 가져오기 기능을 토글형식으로 변경 * refactor: 사용자 위치 조회 로직 수정 (#596) * refactor: 현재 위치 요청 로직 변경 (#599) * refactor: watchPosition 에서 getCurrentPosition으로 변경 (#600) * [FE] Feat/#592 핀 디테일의 핀 이미지 클릭 시 모달을 통해 크게 보이는 기능 구현 (#595) * feat: 이미지 클릭 시 이미지 모달 띄우는 기능 추가 및 반응형 적용 * refactor: context 역할에 맞게 이름 수정 * refactor: fetchDelete 안에 있는 불필요한 isThrow 속성 제거 * refactor: 필요없는 코드 및 수정 필요한 이름 변경 * refactor: 명시적인 함수 이름으로 수정 * [FE] Feat/#597 댓글 기능 구현 (#598) * feat: 댓글 기능 컴포넌트 구현 * feat: 핀 상세보기 페이지 댓글기능 반영 * chore: 자동완성 컴포넌트 lint * refacotr: api 응답 명세 반영 * feat: 대댓글 구현 * feat: 수정 삭제 기능 구현 * refactor: 수정하기 기능 수정 * refactor: 수정 삭제 권한 반영 * refactor: 댓글 재요청 로직 수정 * refactor :수정 삭제시 commentId에서 pinId로 변경 * refactor: fetch로직 수정 * refactor: 에러 수정 * refactor: default prod url 수정 * refactor: 리뷰 반영 * fix: 운영 서버 workflow Dockerfile 이름 수정 * fix: 운영서버 ci/cd 를 위한 Dockerfile-prod 에 개행 추가 * [FE] Feat/#604 image swiper 적용 및 image size limit 적용 (#608) * refactor: PinDetail 빼먹었던 부분 다시 추가 * refactor: image wrapper에 key값 적용 * feat: 핀 디테일 이미지에 스와이퍼 적용 * refactor: 이미지 3개 이하면 스와이프 안 나오도록 수정 * refactor: 비로그인 시 이미지 추가 버튼 안 보이도록 수정 * feat: 토픽 사진 추가할 때 사이즈 제한 주기 * refactor: 필요없는 코드 삭제 * [FE] Refactor/#593 모아보기와 단일조회페이지 분리 및 리팩토링 (#607) * refactor: isNotTabBoxShow에 따라 스와이퍼 버튼 위치 조정 * design: banner TabBox 라인 삭제 * refactor: 모아보기 기능 SelectedTopic 페이지에서 분리 * refactor: resizeMap 로직 훅으로 분리 및 재사용 * refactor: 네비게이션 하이라이트 기능 훅으로 분리 및 재사용 * refactor: tag 관련 로직 훅으로 분리 및 재사용 * refactor: 토스트 ui 변경 * refactor: seetogether 페이지 분리 * fix: 모아보기 페이지에서 모아보기 삭제 시 발생하던 오류 수정 * refactor: 불필요한 import 제거 * refactor: 네비게이션 하이라이트 topics에서 무효화 및 로그인 페이지 오류 수정 * fix: 회원일때는 url 상태로 토픽 조회 하도록 변경하여 새로고침 오류 수정 * fix: 모아보기를 모두 비운 후 핀이 남아있는 오류 수정 * fix: 모아보기 상태에서 토픽 이미지 수정이 선택한 토픽으로 되지 않는 오류 수정 * chore: swiper 제거 후 npm 배포 버전으로 import * refactor: search 페이지 서스펜스 제거 * fix: 핀 디테일 때 맵 클릭 시 움직이지 않던 에러 해결 * fix: 누락된 TMAP_API_KEY 반영 * fix: 모아보기 클릭이벤트 수정 * refactor: pinDetail 페이지 줌인 * fix: banner url 수정 * [FE] Hotfix/map panto 오류 수정 (#611) * fix: 지도 조회 시 핀 바운더리 경계면에 맞게 줌인 되지 않던 오류 수정 * design: 풀핀 여백이 보이는 오류 수정 * test: 버그 해결을 위한 테스트 추가 * fix: 핀 삭제 시 토픽의 핀 개수 반영 안되는 문제 해결 --------- Co-authored-by: GC-Park <kuyt3122@naver.com> Co-authored-by: ParkGeunCheol <72205402+GC-Park@users.noreply.github.com> Co-authored-by: afds4567 <33995840+afds4567@users.noreply.github.com> Co-authored-by: 이세민 <89172499+semnil5202@users.noreply.github.com> Co-authored-by: Doy <doyoungwork@gmail.com> Co-authored-by: semnil5202 <semnil5202@gmail.com> Co-authored-by: 준팍(junpak) <112045553+junpakPark@users.noreply.github.com>
woowacourse-teams · Nov 18, 2023 · 252431c · 252431c
1 parent 9a26db2
commit 252431c
Show file tree

Hide file tree

Showing 165 changed files with 3,387 additions and 1,384 deletions.
diff --git a/.github/workflows/fe-merge-dev.yml b/.github/workflows/fe-merge-dev.yml
@@ -4,7 +4,7 @@ on:
   workflow_dispatch:
 
   pull_request:
-    branches: [develop-FE-2]
+    branches: [develop-FE]
     types: [closed]
     paths: frontend/**
 
@@ -35,6 +35,7 @@ jobs:
         working-directory: frontend
         env:
           REACT_APP_GOOGLE_ANALYTICS: ${{ secrets.REACT_APP_GOOGLE_ANALYTICS }}
+          REACT_APP_TMAP_API_KEY: ${{ secrets.REACT_APP_TMAP_API_KEY }}
           APP_URL: "https://mapbefine.kro.kr/api"
 
       - name: upload to artifact

diff --git a/.github/workflows/fe-merge-prod.yml b/.github/workflows/fe-merge-prod.yml
@@ -35,6 +35,7 @@ jobs:
         working-directory: frontend
         env:
           REACT_APP_GOOGLE_ANALYTICS: ${{ secrets.REACT_APP_GOOGLE_ANALYTICS }}
+          REACT_APP_TMAP_API_KEY: ${{ secrets.REACT_APP_TMAP_API_KEY }}
           APP_URL: "https://mapbefine.com/api"
 
       - name: upload to artifact

diff --git a/.github/workflows/fe-pull-request.yml b/.github/workflows/fe-pull-request.yml
@@ -3,7 +3,7 @@ name: Frontend CI For Test Validation
 on:
   # pull request open과 reopen 시 실행한다.
   pull_request:
-    branches: [main, develop-FE-2]
+    branches: [main, develop-FE]
     paths: frontend/**
 
 defaults:

diff --git a/backend/Dockerfile-prod b/backend/Dockerfile-prod
@@ -1,3 +1,3 @@
-FROM openjdk:17
-COPY build/libs/mapbefine.jar mapbefine.jar
+FROM openjdk:17 
+COPY build/libs/mapbefine.jar mapbefine.jar 
 ENTRYPOINT ["java", "-jar","-Dspring.profiles.active=prod", "mapbefine.jar"]
diff --git a/backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java b/backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java
@@ -117,6 +117,7 @@ public void deletePin(Long pinId) {
                 .orElseThrow(() -> new PinNotFoundException(PIN_NOT_FOUND, pinId));
 
         pin.decreaseTopicPinCount();
+        pinRepository.flush();
         pinImageRepository.deleteAllByPinId(pinId);
         pinRepository.deleteById(pin.getId());
     }

diff --git a/backend/src/test/java/com/mapbefine/mapbefine/admin/AdminIntegrationTest.java b/backend/src/test/java/com/mapbefine/mapbefine/admin/AdminIntegrationTest.java
@@ -1,6 +1,6 @@
 package com.mapbefine.mapbefine.admin;
 
-import static io.restassured.RestAssured.given;
+import static io.restassured.RestAssured.*;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.springframework.http.HttpHeaders.AUTHORIZATION;
 
@@ -23,7 +23,8 @@
 import com.mapbefine.mapbefine.topic.TopicFixture;
 import com.mapbefine.mapbefine.topic.domain.Topic;
 import com.mapbefine.mapbefine.topic.domain.TopicRepository;
-import io.restassured.common.mapper.TypeRef;
+import io.restassured.common.mapper.*;
+import java.util.List;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -32,8 +33,6 @@
 import org.springframework.http.HttpStatus;
 import org.springframework.http.MediaType;
 
-import java.util.List;
-
 class AdminIntegrationTest extends IntegrationTest {
 
     @Autowired
@@ -124,15 +123,13 @@ void findMemberDetail_Success() {
                 .extract()
                 .as(new TypeRef<>() {
                 });
-        System.out.println("====" + topic.getPinCount());
-        //then
 
+        //then
         AdminMemberDetailResponse expected = AdminMemberDetailResponse.of(
                 member,
                 member.getCreatedTopics(),
                 member.getCreatedPins()
         );
-
         assertThat(response).usingRecursiveComparison()
                 .ignoringFields("updatedAt")
                 .ignoringFields("topics.updatedAt")
@@ -218,6 +215,9 @@ void deletePin_Success() {
                 .when().delete("/admin/pins/" + pin.getId())
                 .then().log().all()
                 .statusCode(HttpStatus.NO_CONTENT.value());
+
+        Topic updatedTopic = topicRepository.findById(topic.getId()).get();
+        assertThat(updatedTopic.getPinCount()).isEqualTo(0);
     }
 
     @Test

diff --git a/backend/src/test/java/com/mapbefine/mapbefine/admin/application/AdminCommandServiceTest.java b/backend/src/test/java/com/mapbefine/mapbefine/admin/application/AdminCommandServiceTest.java
@@ -160,6 +160,20 @@ void deletePin_Success() {
         assertThat(pinRepository.existsById(pin.getId())).isFalse();
     }
 
+    @DisplayName("핀 삭제 시, 토픽의 핀 개수를 1 감소시킨다.")
+    @Test
+    void deletePin_Success_decreaseTopicPinCount() {
+        //given
+        assertThat(pin.isDeleted()).isFalse();
+        int pinCountBeforeDelete = topic.getPinCount();
+
+        //when
+        adminCommandService.deletePin(pin.getId());
+
+        //then
+        assertThat(topic.getPinCount()).isEqualTo(pinCountBeforeDelete - 1);
+    }
+
     @DisplayName("Admin인 경우, 핀 이미지를 삭제할 수 있다.")
     @Test
     void deletePinImage_Success() {

diff --git a/frontend/.eslintrc.js b/frontend/.eslintrc.js
@@ -19,6 +19,27 @@ module.exports = {
     ],
     'simple-import-sort/imports': 'error',
     'simple-import-sort/exports': 'error',
+    'no-undef': 'off',
+    'no-unused-vars': 'off',
+    '@typescript-eslint/no-unused-vars': 'off',
+    'import/prefer-default-export': 'off',
+    'no-use-before-define': 'off',
+    'react/require-default-props': 'off',
+    'react/destructuring-assignment': 'off',
+    'react/jsx-no-constructed-context-values': 'off',
+    'no-restricted-globals': 'off',
+    'no-shadow': 'off',
+    'consistent-return': 'off',
+    'no-restricted-syntax': 'off',
+    'no-await-in-loop': 'off',
+    'no-param-reassign': 'off',
+    'jsx-a11y/tabindex-no-positive': 'off',
+    'react/no-array-index-key': 'off',
+    'react/jsx-no-useless-fragment': 'off',
+    'no-unused-expressions': 'off',
+    'react/jsx-props-no-spreading': 'off',
+    'react/no-unused-prop-types': 'off',
+    'import/no-extraneous-dependencies': 'off',
   },
   settings: {
     'import/resolver': {