Test: Upgrade JUnit, JMock

cheesecrust commented 4 weeks ago

🔗 Related Issue

⌨️ What I did

JUnit 버전 업그레이드

test 메서드들에서 JUnit3 의 TestCase 와 JUnit4 의 @Test 를 JUnit5 의 @Test 로 변경하였습니다. JUnit3 의 TestCase 의 경우는 메서드가 test 로 시작하는 경우를 TestCase 로 인식하기 때문에 이러한 메서드 위에 @Test 를 붙여주었습니다.

JUnit4 의 @Test((expected = TimeoutException.class) 를 assertThrows 로 변경하였습니다. 모두 JUnit5 로 버전업을 함에 따라 assert 부분도 모두 교체하였습니다. junit.Assert -> junti5.Assertion

Junit5 의 assert 에서는 메세지 인자의 위치가 바뀌어 이를 모두 수정하였습니다.

JMock 업그레이드

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

MockObjectTestCase -> TestCase 위 처럼 변경함에 따라 Mock 을 위한 Mockery 객체를 도입하였습니다. Mockery 객체 생성을 위해 @BeforeEach 를 도입하여 테스트 시작 전에 생성하고 사용하도록 하였습니다. 기존 JMock1 에서는 기대를 모두 충족했는지 자동으로 확인했지만, JMock2 에서는 context.assertIsSatisfied(); 를 활용해야하므로 @AfterEach 로 각각을 호출 하였습니다.
기댓값 설정 문법 변경

기존

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

변경 후

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

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

cheesecrust commented 3 weeks ago

이쪽 커밋 타입을 INTERNAL -> TEST로 변경해야 할 것 같네요.

진행한 작업들을 추상화하여 첫 코멘트에 작성해주시면 좋겠습니다.

타입과 커멘트 수정하였습니다.

cheesecrust commented 2 weeks ago

rebase 해주시고, develop 기준일 때 수행되는 테스트 개수와 본 pr의 커밋 기준 수행되는 테스트 개수 검증 부탁드립니다.

$ mvn clean verify 수행시

Tests run: 904

로 같습니다.

jhpark816 commented 2 weeks ago

@cheesecrust 빠짐없이 수정하고 로컬에서 테스트 완료하고 마지막으로 스스로 리뷰를 진행하여 더 이상 문제가 없다고 판단될 경우만 수정 사항을 github에 올려주고 리뷰를 명시적으로 요청해 주세요.

jhpark816 commented 1 week ago

@uhm0311 리뷰 진행해 주세요.

jhpark816 commented 1 week ago

@brido4125 리뷰 부탁합니다.

oliviarla commented 1 week ago

@brido4125 public 제거하는 변경 사항은 본 PR에 적용하지는 말고, 패키지별로 PR 나누어 보내 바로바로 머지되도록 진행하는 것이 좋을 것 같습니다. 해당 작업할 때 테스트 이름도 testXXX 형태에서 XXX로 변경하면 좋을 것 같고요..

(일괄 변경이 가능하더라도 전체적으로 다시 봐야할 것 같아서 부담이 됩니다..ㅜ)

jhpark816 commented 1 week ago

수고했습니다.

naver / arcus-java-client