Skip to content

Commit 4d908bf

Browse files
committed
fix: api key 한개 사용하도록 수정
1 parent 8b25c77 commit 4d908bf

File tree

3 files changed

+4
-43
lines changed

3 files changed

+4
-43
lines changed

src/main/java/apptive/team5/youtube/YoutubeApiKeyProvider.java

Lines changed: 0 additions & 33 deletions
This file was deleted.

src/main/java/apptive/team5/youtube/service/YoutubeService.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
import apptive.team5.global.exception.ExceptionCode;
44
import apptive.team5.global.exception.ExternalApiConnectException;
5-
import apptive.team5.youtube.YoutubeApiKeyProvider;
65
import apptive.team5.youtube.domain.YoutubeInfo;
76
import apptive.team5.youtube.dto.YoutubeSearchRequest;
87
import apptive.team5.youtube.dto.YoutubeVideoResponse;
@@ -12,6 +11,7 @@
1211
import com.google.api.services.youtube.model.SearchListResponse;
1312
import com.google.api.services.youtube.model.VideoListResponse;
1413
import lombok.RequiredArgsConstructor;
14+
import org.springframework.beans.factory.annotation.Value;
1515
import org.springframework.http.HttpStatus;
1616
import org.springframework.stereotype.Service;
1717
import org.springframework.transaction.annotation.Transactional;
@@ -20,14 +20,14 @@
2020
import java.util.Collections;
2121
import java.util.List;
2222
import java.util.Objects;
23-
import java.util.Optional;
2423

2524
@Transactional
2625
@Service
2726
@RequiredArgsConstructor
2827
public class YoutubeService {
2928

30-
private final YoutubeApiKeyProvider apiKeyProvider;
29+
@Value("${youtube.api.key}")
30+
private String apiKey;
3131
private static final GsonFactory gsonFactory = new GsonFactory();
3232
private final YoutubeInfoLowService youtubeInfoLowService;
3333

@@ -42,8 +42,6 @@ public List<YoutubeVideoResponse> searchVideo(YoutubeSearchRequest searchRequest
4242
.toList();
4343
}
4444

45-
String apiKey = apiKeyProvider.nextKey();
46-
4745
YouTube youtube = new YouTube.Builder(
4846
new NetHttpTransport(), gsonFactory, request -> {}
4947
).build();

src/test/java/apptive/team5/youtube/service/YoutubeServiceTest.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package apptive.team5.youtube.service;
22

3-
import apptive.team5.youtube.YoutubeApiKeyProvider;
43
import apptive.team5.youtube.domain.YoutubeInfo;
54
import apptive.team5.youtube.dto.YoutubeSearchRequest;
65
import apptive.team5.youtube.dto.YoutubeVideoResponse;
@@ -12,7 +11,6 @@
1211
import org.springframework.test.context.junit.jupiter.SpringExtension;
1312

1413
import java.util.List;
15-
import java.util.Optional;
1614

1715
import static org.assertj.core.api.SoftAssertions.*;
1816
import static org.mockito.ArgumentMatchers.any;
@@ -29,8 +27,6 @@ class YoutubeServiceTest {
2927
@Mock
3028
private YoutubeInfoLowService youtubeInfoLowService;;
3129

32-
@Mock
33-
private YoutubeApiKeyProvider youtubeApiKeyProvider;
3430

3531

3632
@Test
@@ -58,7 +54,7 @@ void getYoutubeVideoResponseWithSpotifyId() {
5854
});
5955

6056
verify(youtubeInfoLowService).findBySpotifyId(any());
61-
verifyNoMoreInteractions(youtubeInfoLowService, youtubeApiKeyProvider);
57+
verifyNoMoreInteractions(youtubeInfoLowService);
6258

6359
}
6460

0 commit comments

Comments
 (0)