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

실습 - cucumber 전환 #422

Open
wants to merge 6 commits into
base: default-sample
Choose a base branch
from
Open

Conversation

bactoria
Copy link

default-sample 에서 작업을 진행 했습니다

  1. path.feature 가 동작하도록 기존 PathAcceptanceTest 에 있던 코드들을 모두 StationStepDef 로 옮기기
  2. path.feature 보강하기 (Given, When, Then)
  3. 공유 객체
  4. 인수 테스트 격리

처음에는 약간 막막했지만,
Given 절을 추가하기 (087f625) 작업 이후 cucumber에 대해 흥미를 느끼기 시작했던 것 같아요.


어려웠던 부분

처음에 요구사항을 보고는 벙쪄있었던 순간이 있었어요. 😭
개인적인 의견 몇가지 남겨보겠습니다.

  1. station.feature 가 메서드명인지 파일이름인지 가늠이 되지 않았어요.

  2. 한글만 적혀있는데도 테스트가 성공하는데, 이게 왜 동작하는건지 파악이 필요했어요.
    처음에는 StationAcceptanceTest 를 한참 찾아봤는데, StationStepDef 와 관련이 있더라고요.

  3. station.feature 가 cucumber를 사용한 건지도 모르는 상태에서 요구사항을 보니, 무엇을 어떻게 만들어야 할지 몰랐던 것 같아요.
    => 기존의 테스트(StationAcceptanceTest) 를 cucumber로 전환한 것이 (station.feature, StationStepDef) 임을 이해하는데 오래 걸리다 보니, 요구사항을 보았을 때 cucumber 기반으로 추가하라는 부분에서 당황했던 것 같아요.

위 3가지는 시간을 쓰면 결국 해결 할 수 있는 사소한 부분이지만,
수강생들에게 친절하게 알려줘도 되는 부분이 아닐까 하는 생각이 들어서 적어보았어요.

  1. 파일 경로가 더 자세히 나와있으면 더 좋을 것 같아요.
  2. StationStepDef 도 함께 참고하도록 명시가 된다면 더 좋을 것 같아요.
  3. station.feature 가 cucumber 기반의 테스트라는 것을 수강생이 쉽게 인지할 수 있으면 더 좋을 것 같아요.

@bactoria bactoria self-assigned this Jan 27, 2024
Copy link
Contributor

@boorownie boorownie left a comment

Choose a reason for hiding this comment

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

상당히 꼼꼼한 피드백 감사합니다.
말씀해주신 부분이 수강생 분들이 느꼈을 어려움이 될 수 있었을텐데
꼼꼼히 피드백 주셔서 그대로 수정합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants