-
Notifications
You must be signed in to change notification settings - Fork 100
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
Add 'Concept' page of 'Fixture Monkey Options' section for kor doc #914
Add 'Concept' page of 'Fixture Monkey Options' section for kor doc #914
Conversation
|
||
## 프로퍼티 | ||
|
||
문서에서는 클래스 객체의 특성을 나타낼 때, `필드` 대신에 일관적으로`프로퍼티`라는 용어를 사용합니다. |
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.
일관적으로와 프로퍼티 사이에 공백이 빠진 것 같습니다.
용어를 사용합니다. 사이에는 공백이 두개인것 같습니다.
문서에서는 클래스 객체의 특성을 나타낼 때, `필드` 대신에 일관적으로`프로퍼티`라는 용어를 사용합니다. | ||
이 용어는 Kotlin의 '프로퍼티'와 동일한 이름이지만, Fixture Monkey에서는 컨셉이 다릅니다. | ||
|
||
Fixture Monkey의 초기 구조는 주로 필드에 기반하고 있었으며, 이는 메서드와 기타 메커니즘을 통한 구성 및 제어에 제한을 뒀습니다. |
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.
제한을 두었습니다 보다 제약을 가했습니다가 의미에 조금 더 어울릴 것 같습니다!
이 용어는 Kotlin의 '프로퍼티'와 동일한 이름이지만, Fixture Monkey에서는 컨셉이 다릅니다. | ||
|
||
Fixture Monkey의 초기 구조는 주로 필드에 기반하고 있었으며, 이는 메서드와 기타 메커니즘을 통한 구성 및 제어에 제한을 뒀습니다. | ||
예를 들어, 오직 필드에만 의존할 경우 Setter 메서드로 어노테이션들에 접근할 수 없습니다. |
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.
setter 메서드에 존재하는 어노테이션에 접근할 수 없습니다 가 원문의 의미와 더 맞는 것 같습니다.
`객체 프로퍼티`는 불변 객체 정보를 나타내는 프로퍼티로 다음을 포함합니다: | ||
|
||
- **property**: 객체 자체의 프로퍼티입니다. | ||
- **propertyNameResolver**: 프로퍼티 명이 해결되는 방식을 결정합니다. |
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.
프로퍼티명이 결정되는 방식을 정의합니다. 가 좀 더 자연스러운 것 같습니다.
|
||
컨테이너 타입의 프로퍼티는 불변 컨테이너 정보를 나타내는 `컨테이너 프로퍼티`로 다음을 포함합니다: | ||
|
||
- **elementProperties**: 요소 프로퍼티의 리스트. |
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.
모든 요소 프로퍼티들을 담은 리스트 가 더 자연스러울 것 같습니다.
컨테이너 타입의 프로퍼티는 불변 컨테이너 정보를 나타내는 `컨테이너 프로퍼티`로 다음을 포함합니다: | ||
|
||
- **elementProperties**: 요소 프로퍼티의 리스트. | ||
- **containerInfo**: 컨테이너의 크기를 결정하는 `임의 컨테이너 정보`. |
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.
이 부분 클래스명이라서 임의 컨테이너보다는 ArbitraryContainerInfo
로 남겨두는 것이 좋은 것 같습니다.
## 옵션들 | ||
|
||
Fixture Monkey의 여러 옵션들은 공통적인 특성을 공유합니다. | ||
예를 들어,`ObjectPropertyGenerator`를 수정하는 관련 옵션들을 살펴보겠습니다. |
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.
관련이 없어도 될 것 같습니다
> `defaultObjectPropertyGenerator`, `pushObjectPropertyGenerator`, `pushAssignableTypeObjectPropertyGenerator`, `pushExactTypeObjectPropertyGenerator` | ||
|
||
접두사 default가 붙은 옵션은 Fixture Monkey에서 생성한 모든 프로퍼티에 기본값으로 적용됩니다. | ||
이러한 기본값은 모든 프로퍼티 타입에 균일하게 영향을 미치는 기본 동작을 설정합니다. |
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.
균일하게보다는 전역적으로가 더 어울릴 것 같습니다.
이러한 push 옵션에는 세 가지 바리에이션이 있습니다. | ||
|
||
- push~ : MatcherOperator를 매개변수로 받습니다. | ||
- pushAssignableType~: 이 옵션은 주어진 타입(옵션과 관련된)을 할당할 수 있는 모든 프로퍼티 타입에 명시된 설정을 적용합니다. |
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.
(옵션과 관련된) 을 여기서는 제거해도 될 것 같습니다.
@SooKim1110 피드백 적용하여 수정해봤습니다. |
컨테이너 타입의 프로퍼티는 불변 컨테이너 정보를 나타내는 `컨테이너 프로퍼티`로 다음을 포함합니다: | ||
|
||
- **elementProperties**: 모든 요소 프로퍼티들을 담은 리스트. | ||
- **containerInfo**: 컨테이너의 크기를 결정하는 `ObjectPropertyGenerator`. |
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.
이 부분 ArbitraryContainerInfo
이 맞을 것 같습니다.
여기만 수정해주시면 될 것 같습니다! 번역해주시느라 고생하셨습니다 👍
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.
확인을 못했네요. 감사합니다.
ObjectPropertyGenerator -> ArbitraryContainerInfo
Summary
'Concept' page of 'Fixture Monkey Options' section related to #864
If you find something awkward or something we can improve, please comment.
thank you.