🚀 2단계 - 장바구니(상품 상세)

[ethanchaee]

구현

• 상품 상세화면
• 상품 상세화면 이동
  • 상품 목록에서 상품 클릭 시, 상품 상세화면으로 이동
  • 뒤로가기 버튼 또는 아이콘 클릭 시, 직전 화면으로 이동
• 장바구니 화면
• 장바구니 화면 이동
  • 상품 목록에서 장바구니 아이콘 클릭 시, 장바구니 화면으로 이동
  • 상품 상세에서 장바구니 담기 버튼 클릭 시, 장바구니 화면으로 이동
  • 뒤로가기 버튼 또는 아이콘 클릭 시, 직전 화면으로 이동

영직님, step2 리뷰도 잘부탁드려요 :)

Activity 관점에서 테스트 하려다 보니, ActivitySenario 가 적절하지 않은 것 같아서 찾아보니 IntentsRule 라는게 있어서 사용해봤습니다.

[laco-dev]

2단계 미션 고생 하셨습니다.

컴포저블에서 화면을 이동하다보니 Intent를 비롯해 테스트를 위한 여러 고민을 많이 하신 모습이 너무 좋았습니다.

다만 우리의 목표는 이처럼 테스트 하기 어려운 것을 보다 테스트 하기 쉽도록 나누는 연습을 하는 것에 있기 때문에 관련해서 몇가지 의견을 같이 드렸습니다.

그 점을 충분히 인지한 상태에서 지금의 도전을 하시는 점이라면 남겨주세요. :)
참고해서 리뷰 하도록 하겠습니다.

[laco-dev]

디스크 캐시는 어떤 이유로 코드를 추가하셨는지 알 수 있을까요?
Coil에서 기본적으로 동작을 하고 있는것으로 알고 있습니다.

[laco-dev]

별도 컴포넌트로 분리할 때 크기 제약사항을 그대로 사용하는 것은 다소 위험합니다.
가령, 나는 썸네일 이미지를 정사각형이 아닌 다른 크기로 그리고 싶다고 하더라도 항상 정사각형으로 나오게 됩니다.

xml 에서도 layout_height, layout_width는 직접 지정하는 것 처럼 크기는 외부에서 넣어줄 수 있도록 하는 방법이 좋습니다.

Thumbnail(modifier: Modifier = Modifier) {
   AsyncImage(modifier = modifier.xxx)
}

[laco-dev]

지금 구현에서는 장바구니 버튼은 항상 보이는데요.
이 컴포넌트를 사용하는 입장에서는 showCartButton = false라고 지정하면, 버튼이 안보인다고 생각하는게 자연스럽지 않을까요?

결국 비슷한 사례가 생길때마다 파라미터로 엄청나게 많은 필드가 필요하게 될텐데 그럴 땐 어떻게 대응할 것인지 궁금합니다.
예를 들어 액션 버튼의 종류가 10개가 넘고 화면 이동이 수십가지라면요?

[laco-dev]

PreviewParameter를 사용해서 테스트 케이스를 작성하는 방법도 좋아요.
다만 매번 만들기 번거롭거나 할 경우에는 그냥 하나의 테스트에서 여러개를 작성하는 방법도 있습니다.

참고만 해주세요~

fun Preview() {
   val texts = listOf("버튼", "...", "...")
   Column {
       texts.forEach { ... }
   }
}

[laco-dev]

컴포저블 내에서 화면을 이동하는 방법도 좋습니다. 이를 테스트 하기 위해 고민해주신 것 포함해서요

그런데 화면 이동이라는 것은 결국 컨트롤러(액티비티)의 역할이라고 생각을 합니다.
특정 컴포저블이 정말 화면에 강하게 결합되어 있어 내부에서 이동을 할 수도 있겠지만 적어도 화면 이동과 같은 것은 테스트하기 정말 어렵게 만들어집니다.

그렇기에 람다를 끌어올려서 테스트 가능한 것과 테스트 하기 어려운 것을 분리하는 연습이 필요합니다.

ProductDetailScreen(
   onCartButtonClick: () -> Unit
   ...
) 

나아가 화면 이동을 테스트 하는 것은 UI 테스트와는 살짝 다른 통합 테스트 관점에서 바라봐야 합니다.
실제 구현에서는 네트워크 콜이나 뷰모델 비즈니스 로직 등 여러가지 문제에 직면하게 되니까요.

[laco-dev]

뒤로가기를 Dispatchers를 이용해서 호출을 하시는군요!?
저는 액티비티면 관성처럼 Activity.onBackPressed()를 호출하는 버릇 처럼 되버렸는데 하나 배웠습니다.

[laco-dev]

요기서도 Scaffold가 그다지 의미가 있어보이지 않아요.
ProductDetailScreen 내부에 Scaffold가 있어야 자연스럽지 않을까요?