08.02 4조 코드 리뷰

이호석

• 테스트의 ExtendsWith를 사용해서 AOP를 적용할 수 있는데, 메소드 이름이 동일하더라도 충돌이 일어나지 않음, 또한 애노테이션 속성이 배열이기 때문에 여러개의 인자들을 넣을 수 있다.
• @BeforeEachCalllback
• DB에서 중복 되는 정보들 분리하기 (유저 이름같은것들)

김준기

물어보고 싶은거

• 댓글 작성 후, 조회하는 댓글 데이터는?
  • 무엇을 더 중요하게 생각하느냐에 따라 결정하면 될듯

개발 팁 공유

• JUnit 프레임워크에서 제공하는 @BeforeEachCallback, @AfterEachCallback 등 콜백 인터페이스를 Implements하여, 적용하고자 하는 테스트 클래스의 @ExtendsWith와 결합해서 사용하면 편리함
• 테스트 독립성을 위해 DB 테이블을 create, drop해야 할 때가 있는데, 이 때 활용하면 편리하게 테스트 데이터를 클린업할 수 있음

새로운 키워드

• TestInstance

• purify.min.js → xml 공격 방지에 도움되는 라이브러리

• marked.min.js

• 서블릿을 직접 생성 및 매핑 정보도 직접 설정함으로서 final 필드를 갖고 있도록 할 수 있음

• 싱글 커넥션 → 커넥션 자체가 문제는 없는데, I/O 가 문제가 될 듯

  • 커넥션 하나로도 이정도 서비스는 감당은 가능함

• 성능 테스팅 툴 → Locust (파이썬 기반)

박정제

• 데이터베이스 커넥션 풀을 제대로 닫아줘야 커넥션들이 닫힌다
• 테스트의 ExtendsWith를 사용해서 AOP를 적용할 수 있다!
• @BeforeEachCalllback

김현수

[한 일]

• 🔧 리팩토링
  • ConfigLoader 추가
  • RepositoryFactory 추가
  • ServiceLocator 추가
  • ConnectionPool 추가
  • DB
    • username join
  • 테스트 코드 작성
    • MySQL Repo 테스트

[좋은 인사이트]

• Junit의 Before*Callback

김현종

• 미뤄두고 있던 db 정보 properties로 저도 분리해야겠습니다… 다들 하셨는데 저만 안했군요.
• 테스트 코드 작성시에 @BeforeEachCallBack등을 통해 @Before 등의 보일러 플레이트를 줄일 수 있군요 꿀팁
• 마크다운 도입하니까 상당히 좋네요 예뻐요
• final 과 volatile 의 관계 새로운 지식이었습니다. 굿
• Locust를 통한 부하테스트도 참고가 됐습니다.