translating reference-test-renderer.md#190
Merged
taehwanno merged 6 commits intoreactjs:masterfrom Nov 27, 2019
toy9910:master
Merged
translating reference-test-renderer.md#190taehwanno merged 6 commits intoreactjs:masterfrom toy9910:master
taehwanno merged 6 commits intoreactjs:masterfrom
toy9910:master
Conversation
|
Deploy preview for ko-reactjs-org ready! Built with commit 183299d |
gnujoow
requested changes
Nov 20, 2019
|
|
||
| Similar to the [`act()` helper from `react-dom/test-utils`](/docs/test-utils.html#act), `TestRenderer.act` prepares a component for assertions. Use this version of `act()` to wrap calls to `TestRenderer.create` and `testRenderer.update`. | ||
| react-dom/test-utils에 act()와 비슷하게, TestRenderer.act는 실행을 위한 컴포넌트들을 준비한다. | ||
| TestRenderer.create 와 trestRenderer.update를 호출을 보호하기 위해 이 버전의 act()를 사용하십시오. |
Member
There was a problem hiding this comment.
- 번역해주신 116행과 117행은 원문에서 한 라인에 작성된 문장들입니다. 포멧을 맞춰주시면 감사하겠습니다.
- 경어체로 문체를 변경해주세요
- markdown 링크와 code highlighting이 누락되었습니다. 아래 문서를 참고하셔서 누락된 markdown문법들을 추가해주세요
https://ko.wikipedia.org/wiki/%EB%A7%88%ED%81%AC%EB%8B%A4%EC%9A%B4
| import App from './app.js'; // The component being tested | ||
|
|
||
| // render the component | ||
| // 컴포넌트 랜더링하기 |
| }); | ||
|
|
||
| // make assertions on root | ||
| // root 검증하기 |
| expect(root.toJSON()).toMatchSnapshot(); | ||
|
|
||
| // update with some different props | ||
| // 몇몇의 다른 프로퍼티즈들을 업데이트하기 |
Contributor
Author
|
수정했습니다 |
taehwanno
requested changes
Nov 20, 2019
| import App from './app.js'; // The component being tested | ||
|
|
||
| // render the component | ||
| // 컴포넌트를 랜더링합니다. |
Member
There was a problem hiding this comment.
Suggested change
| // 컴포넌트를 랜더링합니다. | |
| // 컴포넌트를 렌더링합니다. |
| expect(root.toJSON()).toMatchSnapshot(); | ||
|
|
||
| // update with some different props | ||
| // 몇몇의 다른 프로퍼티즈들을 업데이트합니다. |
Member
There was a problem hiding this comment.
Suggested change
| // 몇몇의 다른 프로퍼티즈들을 업데이트합니다. | |
| // 몇몇의 다른 props를 업데이트합니다. |
props은 번역하면 안되는 용어라 수정 부탁드려요. (Translate Glossary)
| ``` | ||
|
|
||
| Similar to the [`act()` helper from `react-dom/test-utils`](/docs/test-utils.html#act), `TestRenderer.act` prepares a component for assertions. Use this version of `act()` to wrap calls to `TestRenderer.create` and `testRenderer.update`. | ||
| [`react-dom/test-utils`에 `act()`](/docs/test-utils.html#act)와 비슷하게, `TestRenderer.act`는 실행을 위한 컴포넌트들을 준비합니다. `TestRenderer.create` 와 `trestRenderer.update`를 호출을 보호하기 위해 이 버전의 `act()`를 사용하십시오. |
Member
There was a problem hiding this comment.
Suggested change
| [`react-dom/test-utils`에 `act()`](/docs/test-utils.html#act)와 비슷하게, `TestRenderer.act`는 실행을 위한 컴포넌트들을 준비합니다. `TestRenderer.create` 와 `trestRenderer.update`를 호출을 보호하기 위해 이 버전의 `act()`를 사용하십시오. | |
| [`react-dom/test-utils`의 `act()`](/docs/test-utils.html#act)와 비슷하게, `TestRenderer.act`는 검증을 위한 컴포넌트를 준비합니다. `TestRenderer.create` 와 `trestRenderer.update` 호출을 이 버전의 `act()`를 사용해서 감싸주세요. |
- 어투를 전체 문서와 통일시킬게요.
- assertion은
실행보다는단언,검증으로 번역할 수 있는데 아래에 번역해놓으신검증으로 통일할게요. act는 테스트할 때 컴포넌트의 렌더링을 갱신하는 코드를 호출하기 위해 사용합니다.act로 인해 감싸지게 되는데보호의 의미보다는 갱신이 확실하게 되도록 하기 위해 감싸지기 때문에감싸주세요.정도도 충분하다고 생각해서 제안드려봐요.
Contributor
Author
|
말씀하신 대로 수정했습니다. |
gnujoow
requested changes
Nov 22, 2019
| ``` | ||
|
|
||
| Similar to the [`act()` helper from `react-dom/test-utils`](/docs/test-utils.html#act), `TestRenderer.act` prepares a component for assertions. Use this version of `act()` to wrap calls to `TestRenderer.create` and `testRenderer.update`. | ||
| [`react-dom/test-utils`의 `act()`](/docs/test-utils.html#act)와 비슷하게, `TestRenderer.act`는 검증을 위한 컴포넌트들을 준비합니다. `TestRenderer.create` 와 `trestRenderer.update`를 호출을 이 버전의 `act()`를 사용해서 감싸주세요. |
Member
There was a problem hiding this comment.
TestRenderer.create와 와 사이에 불필요한 공백이있습니다.
Member
There was a problem hiding this comment.
자연스러운 문장을 위해 아래처럼 수정을 제안합니다.
TestRenderer.create와trestRenderer.update를 호출을 이 버전의act()를 사용해서 감싸주세요.
TestRenderer.create와trestRenderer.update의 호출을 이 버전의act()를 사용해서 감싸주세요.
Contributor
Author
|
수정했습니다. |
taehwanno
approved these changes
Nov 22, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Progress