angdev commented 9 years ago

335

not_to raise_error 를 쓰는 부분은 거의 없긴 하더라구요.

minhyeok4dev commented 9 years ago

확인

MoojinChae commented 9 years ago

확인!

shaynekang commented 9 years ago

수고하셨습니다~

여담이지만... 지금은 Boolean 변수를 체크하는거니 큰 문제는 없지만, 보통 not_to be_valid보다는 to be_invalid를 권장합니다. not_to는 굉장히 낙관적인 Expactation 메소드라서 에러를 정확하게 catch할 수 없습니다.

가령 1, 2, 3, 4, 5 중에 3이 나와야 하는 테스트에서

  expect(foo).to eq(3) # 3이 나오면 통과
  expect(foo).not_to eq(2) # 3이 나오면 통과하자면, 1, 4, 5가 나와도 통과한다.

이렇게 되기 때문에 후자는 버그가 생겨도 테스트해서 잡아주지 못하는 경우가 생깁니다. ㅎㅎ

yhoonkim commented 9 years ago

확인

DonutWorks / Ari

not to raise error -> to be valid #379

335