-
Notifications
You must be signed in to change notification settings - Fork 445
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 to korean for testing-recipes.md #201
Conversation
✔️ Deploy Preview for ko-reactjs-org ready! 🔨 Explore the source changes: 3b86536 🔍 Inspect the deploy log: https://app.netlify.com/sites/ko-reactjs-org/deploys/60d73c89dbfde8000740cdae 😎 Browse the preview: https://deploy-preview-201--ko-reactjs-org.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 쉼표가 과도하게 많아 보입니다. 모범 사례에서는 제한적인 쉼표 사용을 권장합니다.
- 띄어쓰기, 맞춤법 오류도 많아 보입니다. 푸시 전에 맞춤법 검사기로 한 번 확인하고 올려주시면 좋습니다.
content/docs/testing-recipes.md
Outdated
unmountComponentAtNode(container); | ||
container.remove(); | ||
container = null; | ||
}); | ||
``` | ||
|
||
You may use a different pattern, but keep in mind that we want to execute the cleanup _even if a test fails_. Otherwise, tests can become "leaky", and one test can change the behavior of another test. That makes them difficult to debug. | ||
다른 패턴을 사용할 수도 있지만, 테스트가 실패하더라도 정리(clean up)를 해야 한다는 것을 기억해야 합니다. 테스트는 취약점이 될 수 있고, 하나의 테스트는 다른 테스트의 동작 방식을 변형시킬 수 있습니다. 이는 디버깅을 어렵게 만듭니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...정리를 해야한다는 것을 기억해야 합니다.
그냥 정리는 해야합니다.
정도로도 충분해 보입니다.
Otherwise
부분을 정리를 하지 않으면
으로 하면 어떨까요?
leaky
는 여기서 앞서나온 isolated
의 반대 의미로 사용되었습니다. 테스트가 완전히 격리되지 못하고 다른 테스트에 영향을 미친다는 의미인거죠.
동작 방식을 바꾼다
보다는 동작에 영향을 준다
로 제안합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leaky 를 번역하기가 까다롭네요.
아래와 같이 '빈틈' 은 어떨까요?
정리하지 않으면 테스트에 완전히 격리가 되지 않은 '빈틈'이 생기기 되고, 하나의 테스트는 다른 테스트의 동작에 영향을 줄 수 있습니다. 이는 디버깅을 어렵게 만듭니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
일단 눈에 띄는 대로 코멘트와 제안을 남겨두었습니다.
검토해보시고 수정할 부분은 수정해주세요.
감사합니다.
개인사정이 신경을 못쓰다 보니 벌써 이렇게 오래되었네요. 상세한 리뷰 감사합니다. |
content/docs/testing-recipes.md
Outdated
|
||
--- | ||
|
||
### `act()` {#act} | ||
|
||
When writing UI tests, tasks like rendering, user events, or data fetching can be considered as "units" of interaction with a user interface. React provides a helper called `act()` that makes sure all updates related to these "units" have been processed and applied to the DOM before you make any assertions: | ||
UI 테스트, 렌더링과 같은 작업, 유저 이벤트, 데이터 가져오기는 유저 인터페이스와의 상호작용하는 "구성단위"로 간주 됩니다. 리액트는 'act()'라 불리는 함수를 제공하는데, 이 함수는 "구성단위"와 관련된 모든 업데이트가 단언이 실행되기 전에 처리되고 DOM에 적용되도록 돕습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
피드백 감사합니다.
변경완료 했습니다.
@cdsleaf 혹시 현재 작업 계속 가능한 상태이실까요? |
깜박하고 있었습니다. 죄송해요 |
@cdsleaf 안녕하세요 ㅎㅎ 리뷰를 진행해도 괜찮을까용? 푸쉬가 아직 되지 않은 것 같아서 확인차 멘션드립니다. |
@taehwanno 이미 푸쉬했고요 리뷰 부탁드립니다. |
@taehwanno 리뷰 진행해도 될 것 같습니다! |
안녕하세요, 계속 개인 일정상 늦어졌네요. 다음주 내로 살펴보고 반영할 수 있도록 하겠습니다. 미리 공유드려요! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
직접 몇 가지 수정했습니다. 고생하셨습니다!
Progress