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

Translate Test renderer #37

Merged
merged 8 commits into from Mar 6, 2019
Merged

Conversation

hiddenest
Copy link
Contributor

@hiddenest hiddenest commented Feb 19, 2019

Progress

- Translated API Reference > Test Renderer.
@hiddenest hiddenest changed the title Test renderer Translate Test renderer Feb 19, 2019
@tesseralis tesseralis mentioned this pull request Feb 19, 2019
98 tasks

### `testRenderer.toTree()` {#testrenderertotree}

```javascript
testRenderer.toTree()
```

Return an object representing the rendered tree. Unlike `toJSON()`, the representation is more detailed than the one provided by `toJSON()`, and includes the user-written components. You probably don't need this method unless you're writing your own assertion library on top of the test renderer.
렌더된 트리를 나타내는 객체를 반환합니다. `toJSON()`에서 반환되는 값과 달리 더욱 자세한 정보가 반환되며, 반환 값에는 사용자가 작성한 컴포넌트 역시 포함되어있습니다. 이 함수는 당신이 검증을 위해 당신만의 고유한 라이브러리를 테스트 렌더러 위에 사용하지 않는 한 필요하지 않을 것입니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"이 함수는 당신이 검증을 위해 당신만의 고유한 라이브러리를 테스트 렌더러 위에 사용하지 않는 한 필요하지 않을 것입니다."에서
당신이, 당신만의 부분이 좀 어색해서 이렇게 번역 해보았습니다.

"이 함수는 테스트 렌더러 위에 별도의 assertion 라이브러리를 만드는 게 아니라면 필요하지 않을 것입니다."

별도로 assertion 용어는 번역을 해야 될지 잘 판단이 안되서 원문 그대로 두었습니다.

Copy link
Contributor Author

@hiddenest hiddenest Feb 20, 2019

Choose a reason for hiding this comment

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

당신이, 당신만의 를 안쓰는 방향으로 수정해보겠습니다! 정말로 감사합니다!
이 글 전체적으로 make assertion 이라는 표현이 많이 나오는데, 의미상으로 '검증'에 가까운듯 하여 검증으로 통일하여 번역했습니다.
검증(assertion)으로 의역과 영문을 병기할지, 아니면 assertion하다로 통일할지가 조금 고민되네요... ㅠㅠ

Copy link
Contributor

Choose a reason for hiding this comment

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

react 문서를 보면 test 관련 챕터에서 assert, assertion 단어가 나오는데
다 같은 의미로 사용되고 있는거 같아서 한 방향으로 맞추면 좋을거 같네요.

Test Utilities
Shallow Renderer
Test Renderer

Copy link
Contributor

Choose a reason for hiding this comment

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

일단 Shallow Renderer 파트는 검증으로 맞춰서 초안 작성하는 방향으로 가보겠습니다.
다른 의견 있으시면 언제든지 여기에 적어주세요!

Copy link
Collaborator

Choose a reason for hiding this comment

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

@hmu332233 @hiddenest 우선 #2 (comment) 여기에 assert 용어 추가 했습니다. 다른 분들 의견있는지 받아보고 없다면 "검증"으로 하는걸로 하는게 어떨까 합니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@simsim0709 좋습니다! Test Utilities를 번역하고 계신 @ymin1103 님을 멘션합니다 :)

Copy link
Contributor

Choose a reason for hiding this comment

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

답장이 늦어 죄송합니다. assert는 검증으로 통일하도록 하겠습니다.

@hiddenest
Copy link
Contributor Author

@simsim0709 코멘트 반영하였습니다! 감사합니다 👍👍👍

@netlify
Copy link

netlify bot commented Feb 22, 2019

Deploy preview for ko-reactjs-org ready!

Built with commit 3f7bb8a

https://deploy-preview-37--ko-reactjs-org.netlify.com

@taehwanno
Copy link
Member

upstream과 sync되면서 master 브랜치의 모든 링크 주소가 https로 변경되어 직접 충돌 해결했습니다.

@taehwanno taehwanno added needs review Needs review A pull request ready to be reviewed needs +1 approval and removed needs review Needs review A pull request ready to be reviewed labels Feb 28, 2019
@simsim0709 simsim0709 added the needs author response Changes requested needing author's input label Mar 2, 2019
@hiddenest
Copy link
Contributor Author

@simsim0709님! 리뷰해주신 사항을 이전에 모두 반영하였습니다 :)
다시 한 번 확인을 부탁드려도 될까요?

@taehwanno taehwanno removed the needs author response Changes requested needing author's input label Mar 3, 2019
simsim0709
simsim0709 previously approved these changes Mar 6, 2019
Copy link
Collaborator

@simsim0709 simsim0709 left a comment

Choose a reason for hiding this comment

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

수고하셨어요~ 👍

@taehwanno
Copy link
Member

몇가지 텍스트 직접 수정했습니다. 고생하셨습니다 🚀 🚀 🚀 🚀 🚀

@taehwanno taehwanno merged commit 140dbbd into reactjs:master Mar 6, 2019
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.

None yet

5 participants