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

e2e 테스트 추가 #21

Closed
moreal opened this issue Jun 15, 2020 · 6 comments
Closed

e2e 테스트 추가 #21

moreal opened this issue Jun 15, 2020 · 6 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@moreal
Copy link
Contributor

moreal commented Jun 15, 2020

Windows, macOS 테스트를 CI에서 자동으로 돌리면 좋을 것 같습니다.

@riemannulus riemannulus added this to the 차세대 론처 v2 milestone Jun 15, 2020
@riemannulus
Copy link
Member

riemannulus commented Jun 15, 2020

프론트엔드 테스트 프레임워크는 뭘 써도 크게 차이 안 날 거 같은데. (mocha + chai가 좀 더 가볍고 빠르다는 이야기는 있으나 올인원 툴은 아니라 귀찮고, jest는 올인원이라 좋긴 한데 좀 느리고, ava는 사용량이 적고...)

엔드-투-엔드 테스트는 어떻게 만들어야 하나 좀 고민했는데 이미 제공하고 있었군요 (...)

https://www.electronjs.org/spectron

이거 쓰면 될 거 같습니다.

@riemannulus
Copy link
Member

https://github.com/h3poteto/electron-mock-ipc

IPC모킹은 이걸로 하면 될 거 같습니다.

@moreal moreal self-assigned this Jun 16, 2020
@riemannulus riemannulus self-assigned this Jun 16, 2020
@moreal
Copy link
Contributor Author

moreal commented Jun 16, 2020

Mock child_process.spawn

child_process.spawn 를 mock 하는 방법에 대해서 서치를 해보니 sinon.js를 사용하거나 mock-spawn을 사용하는 것 같습니다. sinon-js가 사용자가 더 많고 현재도 계속 업데이트 되고 있어서 더 좋은 것 같습니다.

Reference

https://stackoverflow.com/questions/26839932/how-to-mock-the-node-js-child-process-spawn-function

@moreal
Copy link
Contributor Author

moreal commented Jun 16, 2020

GraphQL Mock은 아래 링크의 문서를 참조하면 좋을 것 같습니다.

Reference

https://www.apollographql.com/docs/react/development-testing/client-schema-mocking/#gatsby-focus-wrapper
https://www.apollographql.com/docs/tutorial/schema/#write-schema

@riemannulus riemannulus removed this from the 사내 테스트 준비 milestone Jun 22, 2020
@riemannulus riemannulus added enhancement New feature or request help wanted Extra attention is needed labels Jun 25, 2020
@dahlia dahlia changed the title 테스트 추가 & CI 적용 자동화된 테스트 추가 Jun 26, 2020
@dahlia dahlia mentioned this issue Jun 26, 2020
Closed
@dahlia
Copy link
Contributor

dahlia commented Jun 26, 2020

CI와 자동화된 테스트는 독리적인 부분이므로 (테스트를 안 해도 CI에서 빌드 자동화 정도는 가능하고, CI에서 안 돌리더라도 자동화된 테스트가 있으면 로컬 개발시 활용 가능) 이 이슈는 테스트 자동화만을 다루도록 주제를 고칩니다. CI 주제는 #62 이슈에서 따로 다룹니다.

@riemannulus riemannulus changed the title 자동화된 테스트 추가 e2e 테스트 추가 Aug 27, 2020
@riemannulus
Copy link
Member

#284 에서 해결되었습니다.

@mu-hun mu-hun removed their assignment Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants