Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

프로젝트 초기 구성 설정 #9

Merged
merged 5 commits into from
Nov 21, 2024
Merged

프로젝트 초기 구성 설정 #9

merged 5 commits into from
Nov 21, 2024

Conversation

ingyeomnote
Copy link
Collaborator

📗 요약

Gradle 빌드 도구를 사용하여 wheel-go 프로젝트의 초기 구성을 설정하였습니다.

🛠️ 작업 사항

  • build.gradle 파일 구성: Spring 의존성 추가 및 Java 버전 설정
  • web.xml 작성: Spring MVC 설정과 DispatcherServlet 초기화
  • dispatcher-servlet.xml 작성: Spring MVC 설정 및 컨트롤러 매핑 설정
  • context-common.xml 작성: 공통 빈 설정 및 컴포넌트 스캔 설정
  • TestController 작성: 테스트용 컨트롤러 추가

🗨️ 리뷰어에게 공유사항

  • 현재 프로젝트의 초기 설정과 Spring 의존성 추가가 완료되었습니다.
  • 각 XML 설정 파일에 대한 초기 구성이 적절한지, 추가적으로 필요한 설정이 있는지 확인 부탁드립니다.
  • TestController는 기본적인 엔드포인트를 제공하므로, 작동 여부 확인을 위해 테스트 권장합니다.

@ingyeomnote ingyeomnote self-assigned this Nov 15, 2024
@roam-f-lab
Copy link

현재 컨트롤로만 있는 것으로 확인했습니다.
실행 테스트를 어떻게 하신건가요?
로컬에서 기본적인 서버 실행을 로컬에서 할 수 있도록 웹 컨테이너 구성해주세요.
Junit으로 단위 테스트도 구성해주세요.

build.gradle Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Nov 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

자바 설정으로 변경해주세요.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

자바 설정으로 변경해주세요.

Copy link

@roam-f-lab roam-f-lab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코멘트 확인해주세요.

import javax.servlet.ServletException;
import javax.servlet.ServletRegistration;

public class MyWebAppInitializer implements WebApplicationInitializer {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AbstractAnnotationConfigDispatcherServletInitializer 에 대해서도 한번 찾아보세요.

build.gradle Outdated

implementation "org.apache.tomcat.embed:tomcat-embed-core:9.0.68" // 내장 톰캣 기능 제공
implementation "org.apache.tomcat.embed:tomcat-embed-jasper:9.0.68" // JSP 처리 기능 제공

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API 외에 JSP로 프론트를 구성하실 예정인가요?

build.gradle Outdated

testImplementation "org.junit.jupiter:junit-jupiter-api:5.8.1"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

단위 테스트 제거하지 말고 구성해주세요.

@ingyeomnote ingyeomnote merged commit 372af59 into main Nov 21, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants