[Feat/#18] 파일 삭제 기능 - [merged] #77

Closed TaeYeongKwak closed 1 year ago

In GitLab by @tommysgit on Feb 14, 2023, 10:28

Merges feat/#18 -> dev

📋 구현 내용

작업 내용에 대한 개요

파일 삭제 기능 (@SQLDelete @Preremove기능 활용)

🔍 테스트 케이스

MinioClient 객체만 모킹하고 나머지 객체들은 빈 객체를 주입하여 실제로 @SQLDelete와 @Preremove가 잘 작동되는지 테스트 진행

📢 코멘트

해당 기능을 모두 모킹으로 테스트 해보려 했지만, 실제 객체를 주입하여 확인하는 것이 정확하다고 판단하여 새로운 클래스를 만들어 진행하였습니다.

@AnTaeWook @muyongKim @TaeYeongKwak

In GitLab by @tommysgit on Feb 14, 2023, 10:34

added 1 commit

6abbe71a - Fix : Minio 객체 이름 변경

Compare with previous version

컨벤션에 맞춰 변수들끼리 붙여놓으면 좋을 것 같습니다.

In GitLab by @tommysgit on Feb 14, 2023, 10:48

added 1 commit

f83dbf39 - Fix : 멤버 변수 컨벤션 수정

Compare with previous version

In GitLab by @tommysgit on Feb 14, 2023, 10:50

Commented on src/main/java/com/gabia/weat/gcellapiserver/service/ExcelInfoService.java line 30

반영하였습니다.

In GitLab by @tommysgit on Feb 14, 2023, 10:50

resolved all threads

In GitLab by @muyongKim on Feb 14, 2023, 10:52

approved this merge request

In GitLab by @AnTaeWook on Feb 14, 2023, 10:56

Commented on src/main/java/com/gabia/weat/gcellapiserver/controller/ExcelInfoController.java line 50

삭제 메서드는 응답 상태 코드를 204(NO_CONTENT) 쓰고 void 타입을 가지는 것이 좀 더 Restful 할 것 같습니다!

In GitLab by @tommysgit on Feb 14, 2023, 11:01

added 1 commit

4bedf557 - Fix : 응답 status 수정

Compare with previous version

In GitLab by @tommysgit on Feb 14, 2023, 11:02

Commented on src/main/java/com/gabia/weat/gcellapiserver/controller/ExcelInfoController.java line 50

감사합니다. 반영하였습니다.

In GitLab by @tommysgit on Feb 14, 2023, 11:02

resolved all threads

In GitLab by @tommysgit on Feb 14, 2023, 11:03

approved this merge request

In GitLab by @AnTaeWook on Feb 14, 2023, 11:11

approved this merge request

In GitLab by @tommysgit on Feb 14, 2023, 13:52

mentioned in commit 3cfdfd657ae0564c316ca4ccad8633e790a79bce

G-GCELL / api-server