INTERNAL: Upgrade to JMock version 2 - Githubissues

naver / arcus-java-client

ARCUS Java client

Apache License 2.0

50 stars 46 forks source link

INTERNAL: Upgrade to JMock version 2 #796

Closed cheesecrust closed 4 weeks ago

cheesecrust commented 1 month ago

🔗 Related Issue

https://github.com/jam2in/arcus-works/issues/599

⌨️ What I did

JMock 업그레이드

JMock1 을 JMock2 로 업그레이드 하면서 몇몇 부분을 수정하였습니다.

MockObjectTestCase -> TestCase
기댓값 설정 문법 변경

기존

clientMock.expects(once()).method("getTranscoder")
            .will(returnValue(transcoder));

변경 후

context.checking(
            new Expectations() {{
              oneOf(client).getTranscoder();
              will(returnValue(transcoder));
            }}
    );

mock 객체에서 mocking 한 해당 class 의 함수의 기댓값을 설정할 때 실제 있는 함수인지 검사하도록 변경되어 이를 위한 commandNodeChange 메서드를 추가하였습니다.

cheesecrust commented 4 weeks ago

다른 PR들을 참고하여 커밋을 1개로 통합해주세요.

https://github.com/naver/arcus-java-client/pull/799 위 pr 에 commit 을 모두 통합하여 올렸습니다!

따라서 pr closed 하겠습니다.