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

translating reference-test-renderer.md #190

Merged
merged 6 commits into from
Nov 27, 2019
Merged

translating reference-test-renderer.md #190

merged 6 commits into from
Nov 27, 2019

Conversation

toy9910
Copy link
Contributor

@toy9910 toy9910 commented Nov 18, 2019

@netlify
Copy link

netlify bot commented Nov 18, 2019

Deploy preview for ko-reactjs-org ready!

Built with commit 183299d

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

@@ -113,27 +113,28 @@ TestRenderer.create(element, options);
TestRenderer.act(callback);
```

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()를 사용하십시오.
Copy link
Member

Choose a reason for hiding this comment

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

  • 번역해주신 116행과 117행은 원문에서 한 라인에 작성된 문장들입니다. 포멧을 맞춰주시면 감사하겠습니다.
  • 경어체로 문체를 변경해주세요
  • markdown 링크와 code highlighting이 누락되었습니다. 아래 문서를 참고하셔서 누락된 markdown문법들을 추가해주세요

https://ko.wikipedia.org/wiki/%EB%A7%88%ED%81%AC%EB%8B%A4%EC%9A%B4


```javascript
import {create, act} from 'react-test-renderer';
import App from './app.js'; // The component being tested

// render the component
// 컴포넌트 랜더링하기
Copy link
Member

Choose a reason for hiding this comment

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

경어체로 문체를 변경해주세요

let root;
act(() => {
root = create(<App value={1}/>)
});

// make assertions on root
// root 검증하기
Copy link
Member

Choose a reason for hiding this comment

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

경어체로 문체를 변경해주세요

expect(root.toJSON()).toMatchSnapshot();

// update with some different props
// 몇몇의 다른 프로퍼티즈들을 업데이트하기
Copy link
Member

Choose a reason for hiding this comment

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

경어체로 문체를 변경해주세요

@toy9910
Copy link
Contributor Author

toy9910 commented Nov 20, 2019

수정했습니다


```javascript
import {create, act} from 'react-test-renderer';
import App from './app.js'; // The component being tested

// render the component
// 컴포넌트를 랜더링합니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// 컴포넌트를 랜더링합니다.
// 컴포넌트를 렌더링합니다.

expect(root.toJSON()).toMatchSnapshot();

// update with some different props
// 몇몇의 다른 프로퍼티즈들을 업데이트합니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// 몇몇의 다른 프로퍼티즈들을 업데이트합니다.
// 몇몇의 다른 props를 업데이트합니다.

props은 번역하면 안되는 용어라 수정 부탁드려요. (Translate Glossary)

@@ -113,27 +113,27 @@ TestRenderer.create(element, options);
TestRenderer.act(callback);
```

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()`를 사용하십시오.
Copy link
Member

Choose a reason for hiding this comment

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

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로 인해 감싸지게 되는데 보호의 의미보다는 갱신이 확실하게 되도록 하기 위해 감싸지기 때문에 감싸주세요. 정도도 충분하다고 생각해서 제안드려봐요.

@toy9910
Copy link
Contributor Author

toy9910 commented Nov 21, 2019

말씀하신 대로 수정했습니다.

taehwanno
taehwanno previously approved these changes Nov 21, 2019
Copy link
Member

@taehwanno taehwanno left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ㅎㅎ

@@ -113,27 +113,27 @@ TestRenderer.create(element, options);
TestRenderer.act(callback);
```

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()`를 사용해서 감싸주세요.
Copy link
Member

Choose a reason for hiding this comment

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

TestRenderer.create 사이에 불필요한 공백이있습니다.

Copy link
Member

Choose a reason for hiding this comment

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

자연스러운 문장을 위해 아래처럼 수정을 제안합니다.

TestRenderer.createtrestRenderer.update를 호출을 이 버전의 act()를 사용해서 감싸주세요.

TestRenderer.createtrestRenderer.update의 호출을 이 버전의 act()를 사용해서 감싸주세요.

@toy9910
Copy link
Contributor Author

toy9910 commented Nov 22, 2019

수정했습니다.

Copy link
Member

@hg-pyun hg-pyun left a comment

Choose a reason for hiding this comment

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

LGTM

@taehwanno taehwanno merged commit 60070ab into reactjs:master Nov 27, 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.

4 participants