[TB-27] 영수증 금액 검증에 따른 API 수정

[baekjaehyuk]

📌 작업 개요

• 영수증에 기재된 금액과 영수증 구매 내역 리스트의 가격 총합을 비교하여 일치하는 지에 대한 로직 추가

---

✅ 작업 내용

1. Receipt 엔티티 필드 추가
2. 비즈니스 도메인 서비스단에서 금액 검증 로직 (영수증 등록과 수정할 경우)
3. 영수증 조회 시에 일치하는 지에 대한 여부 반환

---

📂 리뷰 요구사항

• 현재 설계된 아키텍쳐 흐름 상 비교 로직에 대해 적절하게 구현되었는지 리뷰 부탁드립니다.

---

[Copilot]

Pull Request Overview

This PR introduces a new receipt amount validation logic by comparing the receipt’s total price with the sum of its items, and reflects the result through the new isAmountMatched field.

• Added the isAmountMatched field to the Receipt domain, mapper, and response DTOs.
• Implemented a check in ReceiptEditor to verify the amount match and updated both create and update services accordingly.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/main/java/com/ClubAccount_BE/receipt/mapper/ReceiptMapper.java	Added mapping for the new isAmountMatched field.
src/main/java/com/ClubAccount_BE/receipt/domain/service/ReceiptEditor.java	Introduced checkAmountMatch method to validate receipt amounts.
src/main/java/com/ClubAccount_BE/receipt/domain/Receipt.java	Extended the domain model with the isAmountMatched field and its setter.
src/main/java/com/ClubAccount_BE/receipt/application/service/UpdateReceiptService.java	Updated receipt amount verification logic in the update flow.
src/main/java/com/ClubAccount_BE/receipt/application/service/CreateReceiptService.java	Updated receipt amount verification logic in the create flow.
src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/dto/response/ReceiptResponse.java	Updated DTO to include the isAmountMatched field.
src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/dto/response/ReceiptDetailResponse.java	Updated detailed DTO to include the isAmountMatched field.

Comments suppressed due to low confidence (1)

src/main/java/com/ClubAccount_BE/receipt/domain/Receipt.java:90

• [nitpick] The method name 'checkAmountMatched' is used to set the match status, which could be misleading. Consider renaming it to 'setAmountMatched' for better clarity.

public void checkAmountMatched(boolean isAmountMatched) {

[codecov]

Codecov Report

Attention: Patch coverage is 16.66667% with 20 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
.../receipt/adapter/out/ReceiptRepositoryAdapter.java	0.00%	6 Missing ⚠️
...count_BE/receipt/domain/service/ReceiptEditor.java	0.00%	4 Missing ⚠️
...eipt/application/service/CreateReceiptService.java	0.00%	2 Missing ⚠️
...eipt/application/service/UpdateReceiptService.java	0.00%	2 Missing ⚠️
...ava/com/ClubAccount_BE/receipt/domain/Receipt.java	33.33%	2 Missing ⚠️
...ter/in/web/dto/response/ReceiptDetailResponse.java	0.00%	1 Missing ⚠️
...t/adapter/in/web/dto/response/ReceiptResponse.java	0.00%	1 Missing ⚠️
.../adapter/out/persistence/entity/ReceiptEntity.java	0.00%	1 Missing ⚠️
...m/ClubAccount_BE/receipt/mapper/ReceiptMapper.java	75.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[tiemo0708]

고생하셨습니다!
리뷰단거만 한번 확인해주세요

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud