feat: 전체 뉴스를 조회하는 API 구현

[whqtker]

관련 이슈

• resolves: feat: 전체 뉴스 조회 API 구현 #672

작업 내용

기존에는 특정 author-id 의 뉴스만 조회 가능했으나, 전체 뉴스를 조회할 수 있도록 API를 수정했습니다. author-id 를 선택적으로 받을 수 있도록 구현했습니다.

repository 로직같은 경우 기존에 작성된 native query를 참고했습니다.

커밋 순으로 읽으시기보단, 전체 파일 변경 보시는 게 이해하기 더 쉬울 것 같습니다.

특이 사항

해당 PR은 머지 후 체리픽해서 prod에 반영할 예정입니다 ,,!

리뷰 요구사항 (선택)

[coderabbitai]

Walkthrough

뉴스 조회 기능이 보다 유연하게 개선되었습니다. 컨트롤러의 GET 핸들러가 findNewsBySiteUserId에서 findNews로 이름이 바뀌었고, 저자 필터(author-id)가 선택사항이 되었습니다. 리포지토리 계층에 전체 뉴스를 최근 순서대로 조회하는 메서드가 추가되었으며, 커스텀 리포지토리에서 좋아요 상태를 포함한 조회 기능이 구현되었습니다. 서비스 계층의 로직은 저자 ID의 유무에 따라 조건부로 분기하도록 리팩토링되었고, 테스트는 중첩된 테스트 그룹으로 체계적으로 재구성되었습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested reviewers

• wibaek
• lsy1307
• Gyuhyeok99

---

변경사항 상세 분석

다음과 같이 5가지 주요 변경이 포함되어 있습니다.

1. NewsController의 엔드포인트 개선

   • 메서드명 변경: findNewsBySiteUserId → findNews
   • 요청 파라미터 선택사항화: author-id required 속성을 false로 설정
   • 서비스 메서드 호출 경로 조정

2. NewsRepository에 정렬 메서드 추가

   • findAllByOrderByUpdatedAtDesc() 메서드 신규 추가
   • 최신 업데이트 순서로 전체 뉴스를 조회하는 기능 제공

3. NewsCustomRepository 인터페이스 확장

   • findAllNewsWithLikeStatus(Long siteUserId) 시그니처 추가
   • 사용자의 좋아요 상태를 포함한 전체 뉴스 조회 기능 정의

4. NewsCustomRepositoryImpl의 구현 로직 확장

   • JPQL 쿼리로 좋아요 상태를 LEFT JOIN으로 판별하여 반환
   • NewsResponse DTO를 활용한 완성된 데이터 매핑

5. NewsQueryService의 조건부 라우팅

   • 메서드 시그니처 변경: findNewsByAuthorId(Long, long) → findNews(Long, Long)
   • authorId 유무에 따라 서로 다른 리포지토리 메서드 호출 분기 처리
   • 통합된 진입점으로 기능성 향상

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 6.67% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 변경사항의 핵심을 명확하게 요약하고 있으며, 전체 뉴스를 조회하는 API 구현이라는 주요 목표를 정확히 반영하고 있습니다.
Linked Issues check	✅ Passed	코드 변경사항이 #672 이슈의 목표인 전체 뉴스 조회 API 구현과 author-id 선택적 파라미터화를 완벽하게 충족하고 있습니다.
Out of Scope Changes check	✅ Passed	모든 코드 변경사항이 뉴스 조회 API 기능 구현에 직접 관련되어 있으며, 연결된 이슈의 범위 내에 있습니다.
Description check	✅ Passed	PR 설명이 템플릿의 필수 섹션을 충분히 포함하고 있으며, 작업 내용을 명확하게 설명하고 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (2)

src/main/java/com/example/solidconnection/news/repository/custom/NewsCustomRepositoryImpl.java (1)

15-35: JPQL 쿼리 중복이 존재합니다.

findAllNewsWithLikeStatus와 findNewsByAuthorIdWithLikeStatus(Line 37-59)의 SELECT 절과 JOIN 절이 거의 동일하고, WHERE 절 유무만 다릅니다. 현재 메서드가 2개라 관리 부담이 크지 않지만, 향후 SELECT 필드가 변경되면 두 곳을 동시에 수정해야 하는 점은 인지해두시면 좋겠습니다.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@src/main/java/com/example/solidconnection/news/repository/custom/NewsCustomRepositoryImpl.java`
around lines 15 - 35, The SELECT and LEFT JOIN parts used by
findAllNewsWithLikeStatus and findNewsByAuthorIdWithLikeStatus are duplicated;
extract the common JPQL SELECT/JOIN into a single reusable constant or private
helper method (e.g., BASE_NEWS_JPQL or buildBaseNewsQuery()) that returns the
SELECT ... FROM ... LEFT JOIN portion, then have findAllNewsWithLikeStatus
append no WHERE and findNewsByAuthorIdWithLikeStatus append "WHERE n.authorId =
:authorId" (or similar) to that base string, and update both methods to call
entityManager.createQuery on the composed JPQL and set only the necessary
parameters (siteUserId and optionally authorId). Ensure NewsResponse constructor
usage and parameter names remain unchanged.

src/test/java/com/example/solidconnection/news/service/NewsQueryServiceTest.java (1)

114-178: 특정 사용자의 소식지 조회에서 빈 목록 테스트가 누락되었습니다.

전체_소식지를_조회한다 그룹에는 빈 목록 반환 테스트가 있지만(Line 104-111), 특정_사용자의_소식지를_조회한다 그룹에는 해당 작성자의 소식지가 없을 때의 테스트가 빠져 있습니다. 대칭적인 커버리지를 위해 추가를 권장합니다.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@src/test/java/com/example/solidconnection/news/service/NewsQueryServiceTest.java`
around lines 114 - 178, Add a missing empty-list test inside the nested class
특정_사용자의_소식지를_조회한다: create an author SiteUser with no News fixtures, call
newsQueryService.findNews(null, author.getId()), and assert that the returned
NewsListResponse.newsResponseList() is empty and any isLiked values are null
(use NewsListResponse, NewsResponse, and findNews to locate code). Name the test
clearly (e.g., 작성자의_소식지가_없으면_빈_목록을_반환한다) and mirror the assertions style used in
the 전체_소식지를_조회한다 empty-list test.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In
`@src/main/java/com/example/solidconnection/news/repository/custom/NewsCustomRepositoryImpl.java`:
- Around line 15-35: The SELECT and LEFT JOIN parts used by
findAllNewsWithLikeStatus and findNewsByAuthorIdWithLikeStatus are duplicated;
extract the common JPQL SELECT/JOIN into a single reusable constant or private
helper method (e.g., BASE_NEWS_JPQL or buildBaseNewsQuery()) that returns the
SELECT ... FROM ... LEFT JOIN portion, then have findAllNewsWithLikeStatus
append no WHERE and findNewsByAuthorIdWithLikeStatus append "WHERE n.authorId =
:authorId" (or similar) to that base string, and update both methods to call
entityManager.createQuery on the composed JPQL and set only the necessary
parameters (siteUserId and optionally authorId). Ensure NewsResponse constructor
usage and parameter names remain unchanged.

In
`@src/test/java/com/example/solidconnection/news/service/NewsQueryServiceTest.java`:
- Around line 114-178: Add a missing empty-list test inside the nested class
특정_사용자의_소식지를_조회한다: create an author SiteUser with no News fixtures, call
newsQueryService.findNews(null, author.getId()), and assert that the returned
NewsListResponse.newsResponseList() is empty and any isLiked values are null
(use NewsListResponse, NewsResponse, and findNews to locate code). Name the test
clearly (e.g., 작성자의_소식지가_없으면_빈_목록을_반환한다) and mirror the assertions style used in
the 전체_소식지를_조회한다 empty-list test.