Skip to content

Commit f622a28

Browse files
authored
Merge pull request #52 from DKU-Dgaja/refactor(#51)-OAuthServiceTest-refactor
[BE] refactor(#51): OAuthServiceTest 오류수정
2 parents b51ffc4 + b40d9e0 commit f622a28

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

.github/workflows/CI_dev_be_pull_request.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,18 @@ jobs:
5555
runs-on: self-hosted
5656
needs: test
5757
steps:
58+
- name: Upload Report
59+
uses: 'actions/upload-artifact@v2'
60+
with:
61+
name: report.xml
62+
path: ${{ github.workspace }}/backend/build/reports/jacoco/test/jacocoTestReport.xml
5863
- name: 테스트 커버리지를 PR에 코멘트로 등록합니다
5964
id: jacoco
60-
uses: madrapps/jacoco-report@v1.2
65+
uses: madrapps/jacoco-report@v1.6.1
6166
with:
6267
title: 📝 테스트 커버리지 리포트
6368
paths: ${{ github.workspace }}/backend/build/reports/jacoco/test/jacocoTestReport.xml
6469
token: ${{ secrets.GITHUB_TOKEN }}
6570
min-coverage-overall: 50
6671
min-coverage-changed-files: 50
72+
update-comment: true

backend/src/test/java/com/example/backend/auth/api/service/oauth/OAuthServiceTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ class OAuthServiceTest extends TestConfig {
4545
@MockBean
4646
private KakaoAdapter kakaoAdapter;
4747

48+
@Autowired
4849
private GoogleURLBuilder googleurlBuilder;
4950

5051
@MockBean
@@ -68,7 +69,7 @@ void allUrlBuilderSuccess() {
6869
// then
6970
//assertThat(loginPages.get(0).getUrl()).isEqualTo(authorizeURL);
7071

71-
assertThat(loginPages).hasSize(2); // 리스트 크기 확인
72+
assertThat(loginPages).hasSize(3); // 리스트 크기 확인
7273

7374
// 각 플랫폼별 URL인지 확인
7475
boolean containsGithub = loginPages.stream()

0 commit comments

Comments
 (0)