TEST: Fix some CI failures with Future.get() calls and minor fixes in `java.net.spy.memcached`.

[uhm0311]

🔗 Related Issue

• https://github.com/jam2in/arcus-works/issues/597

⌨️ What I did

• java.net.spy.memcached 경로의 테스트 코드가 올바르게 작성되지 않아 실제 의도하는 테스트가 되지 않는 점을 수정합니다.
• 아래는 수정 대상이 되는 예시입니다.
  • Write 연산의 결과를 확인하지 않았으나 데이터를 조회하여 수행하는 테스트
  • 테스트가 실패한 경우에도 실패 처리를 하지 않는 테스트

[oliviarla]

DoLotsOfSets 클래스도 waitForQueues 메서드 사용하지 않도록 수정해야 할 것 같습니다. 두 클래스 외에도 arcusClient.set() 메서드를 호출하고 future.get()으로 정상 처리 되었는지 검증하는 것이 필요할 것 같습니다.

[uhm0311]

waitForQueues() 메소드 사용하지 않는 PR은 따로 올리겠습니다. Write 연산의 결과를 확인하지 않는 테스트 코드가 상당히 많아 이 부분은 이슈를 따로 만들고 클래스 단위 혹은 패키지 단위로 진행할 예정입니다.

[oliviarla]

@uhm0311 그러면 이 PR도 src/test/java/net/spy/memcached 패키지에 위치한 모든 코드를 변경하는 게 어떤가요? 꼭 분리할 필요가 없는 단순한 변경이라는 생각이 들어서요..!

[uhm0311]

@oliviarla

수정했습니다.

[jhpark816]

@uhm0311 예전에도 future.get() 수행하지 않아 결과가 예기치않게 나오던 문제가 있어서, 이를 일괄 수정했던 적이 있습니다. 그 때, 모두 수정되지 않고 일부가 빠져서 이번 문제가 발생한 거죠? 같은 이유라면, 이번에도 간단하게 future.get() 호출하도록 추가하고, 리팩토링에 관한 사항은 천천히 다루는 것이 어떤가요?

[uhm0311]

@jhpark816

본 PR의 현재 성격은 단순히 Future.get()을 호출하는 것이 아닌, 올바르지 않은 형태로 구현된 테스트 코드를 수정하여 정말로 의도하는 테스트를 수행하도록 만들도록 하는 것입니다. 리팩토링 성격으로 보이는 수정도 올바르지 않은 형태로 구현된 테스트 코드를 올바른 형태로 만드는 수정입니다.

[jhpark816]

@oliviarla ArcusTimeoutMessageTest에 포함된 각 테스트에 대해 필요한 테스트인지 그리고 그 목적에 맞게 잘 구현된 테스트인지를 다시 재확인 바랍니다.

[oliviarla]

@jhpark816 ArcusTimeoutMessageTest는 타임아웃 메시지를 검증하는 테스트 클래스이며, 저는 이 클래스 자체가 필요없다고 생각합니다. 로그 메시지를 검증하는 테스트 코드를 작성하지 않듯이 TimeoutException이 발생할 때 내부의 메시지를 검증하는 것은 무의미한 테스트라고 봅니다.

기존에 존재했던 테스트들은 TimeoutMessage를 검증을 목적으로 하지만 TimeoutMessage의 일부만 검증하기 때문에 부적절한 테스트였습니다. 본 PR에서는 TimeoutMessage가 전부 잘 반환되었는지 검증하도록 변경하였는데 이는 의미 상으로는 적절하다고 생각합니다.

[jhpark816]

ArcusTimeoutMessageTest는 타임아웃 메시지를 검증하는 테스트 클래스이며, 저는 이 클래스 자체가 필요없다고 생각합니다. 로그 메시지를 검증하는 테스트 코드를 작성하지 않듯이 TimeoutException이 발생할 때 내부의 메시지를 검증하는 것은 무의미한 테스트라고 봅니다.

@uhm0311 위 코멘트와 같이 타임아웃 메시지를 검증하는 테스트는 필요하지 않다고 생각합니다. 그래서, 해당 테스트를 제거하는 것이 좋을 것 같은 데, 어떤가요?

[uhm0311]

@jhpark816

제거했습니다.

[jhpark816]

@uhm0311 commit 메시지를 아래와 같이 수정하는 것이 좋겠습니다.

TEST: Fix CI failures by calling Future.get().

[uhm0311]

@jhpark816

사소하지만 Future.get()이 아닌 변경사항도 포함되어 있습니다.

[jhpark816]

@uhm0311 메인이 Future.get() 호출이므로 아래와 같이 표현하는 것이 좋겠습니다.

TEST: Fix some CI failures with Future.get() calls and minor fixes.

[uhm0311]

@jhpark816

수정했습니다.