Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor | CAKK-95 | 의존성 관리 방식 관련 리팩토링 #226

Merged
merged 4 commits into from
Dec 8, 2024

Conversation

lcomment
Copy link
Collaborator

@lcomment lcomment commented Dec 6, 2024

Issue Number

CAKK-95

Description

버전 관리를 편하게 하기 위해 의존성 관리 방식을 리팩토링 했습니다.

  • DependencyResolutionManagement 활성화
  • 의존성 관리방식을 Gradle Version Catalog로 변경

Core Code

생략

etc

@lcomment lcomment added the refactor 비즈니스 변경 없는 수정 label Dec 6, 2024
@lcomment lcomment requested a review from YongsHub December 6, 2024 08:31
@lcomment lcomment self-assigned this Dec 6, 2024
Copy link

github-actions bot commented Dec 6, 2024

Test Results

212 tests  ±0   212 ✅ ±0   46s ⏱️ ±0s
 40 suites ±0     0 💤 ±0 
 40 files   ±0     0 ❌ ±0 

Results for commit 697eb9a. ± Comparison against base commit c88aa9e.

Copy link

codecov bot commented Dec 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Impacted file tree graph

@@            Coverage Diff             @@
##             develop     #226   +/-   ##
==========================================
  Coverage      90.19%   90.19%           
  Complexity       235      235           
==========================================
  Files             59       59           
  Lines            612      612           
  Branches          17       17           
==========================================
  Hits             552      552           
  Misses            51       51           
  Partials           9        9           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c88aa9e...697eb9a. Read the comment docs.

Copy link
Contributor

@YongsHub YongsHub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

�Gradle Catalog 설명해주시면 감사합니다!

@@ -49,7 +49,7 @@ class AppleAuthProviderTest : MockitoTest() {
val oidcPublicKeyList = fixtureMonkey.giveMeBuilder(OidcPublicKeyList::class.java)
.set("keys", Arbitraries.of(oidcPublicKeyFixture).list().ofMinSize(1).ofMaxSize(10))
.sample()
val publicKey = fixtureMonkey.giveMeOne(PublicKey::class.java)
val publicKey = mock(PublicKey::class.java)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 mock Fixture 라이브러리 메서드인가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네, mockito 메서드입니다.

public class MockitoCore {

    . . .

    public <T> T mock(Class<T> typeToMock, MockSettings settings) {
        if (!(settings instanceof MockSettingsImpl)) {
            throw new IllegalArgumentException(
                    "Unexpected implementation of '"
                            + settings.getClass().getCanonicalName()
                            + "'\n"
                            + "At the moment, you cannot provide your own implementations of that class.");
        }
        MockSettingsImpl impl = (MockSettingsImpl) settings;
        MockCreationSettings<T> creationSettings = impl.build(typeToMock);
        checkDoNotMockAnnotation(creationSettings.getTypeToMock(), creationSettings);
        T mock = createMock(creationSettings);
        mockingProgress().mockingStarted(mock, creationSettings);
        return mock;
    }
. . .
}

"org.springframework.boot" -> useVersion(springBootVersion)
"io.spring.dependency-management" -> useVersion(springDependencyManagementVersion)
}
repositories {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dependencyResolutionManagement 역할이 뭔가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

블로그에 포스팅 했는데, 참고해주세요 ~~

@lcomment lcomment merged commit 7a2439a into develop Dec 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 비즈니스 변경 없는 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants