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

객체를 생성하는 공통 코드를 위로 올리면 어떨까요? #1

Open
currenjin opened this issue Nov 30, 2022 · 15 comments
Open

Comments

@currenjin
Copy link

void n번째_relay_될_때_parentNovelId는_직전_relayNovelId_다() {

@currenjin currenjin changed the title 공통적으로 객체를 생성하는 코드를 위로 올리면 어떨까요? 객체를 생성하는 공통 코드를 위로 올리면 어떨까요? Nov 30, 2022
@currenjin
Copy link
Author

OpeningId, NovelBoard

@currenjin
Copy link
Author

OpeningId 는 다 같은데 그냥 픽스쳐로 놓으면 안 될까여
NovelBoard 도 가능할 것 같고

@currenjin
Copy link
Author

그리고 클래스 이름이 ParentNovelFinderTest 인데 Find 자체가 잘 드러나지 않은 것 같기도.. 이거 재밌네요

@currenjin
Copy link
Author

이렇게 재밌는 프로젝트를 하시다니

@6161990
Copy link
Owner

6161990 commented Nov 30, 2022

공통으로 빼는 것보다 하나의 테스트 안에서 given when then 을 함께 보여주는게 나을 것 같았습니다요

@currenjin
Copy link
Author

공통으로 빼는 것보다 하나의 테스트 안에서 given when then 을 함께 보여주는게 나을 것 같았습니다요

given 에서는 어떤 것을 표현하고 싶은 건가요?

@6161990
Copy link
Owner

6161990 commented Nov 30, 2022

모든 것은 opening 으로부터 시작된다.

@6161990
Copy link
Owner

6161990 commented Nov 30, 2022

현진님 저 아이디어 뱅크에요
머릿 속에 재밌는 프로젝트 많아여 구미가 땡기시면 연낙주세요

@currenjin
Copy link
Author

currenjin commented Nov 30, 2022

모든 것은 opening 으로부터 시작된다.

opening 과정을 beforeEach 로 빼면 그 표현이 퇴색될까요?

@currenjin
Copy link
Author

현진님 저 아이디어 뱅크에요 머릿 속에 재밌는 프로젝트 많아여 구미가 땡기시면 연낙주세요

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 하나 알려주세요

@6161990
Copy link
Owner

6161990 commented Nov 30, 2022

네 when 과 then 과 멀리 떨어지는게 싫어요
다 가족이야

@currenjin
Copy link
Author

네 when 과 then 과 멀리 떨어지는게 싫어요 다 가족이야

그러면 opening 과정을 extract 해서 관리하는 건 어떠세요?

@6161990
Copy link
Owner

6161990 commented Nov 30, 2022

읽으려면 비포이치까지 스크롤 위로 올리는게 귀찮아요!!!!!!!

@6161990
Copy link
Owner

6161990 commented Nov 30, 2022

하나 알려주면 하셔야돼요

@currenjin
Copy link
Author

하나 알려주면 하셔야돼요

바빠요

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

No branches or pull requests

2 participants