-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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 only japanese word spec #1750
Conversation
@@ -12,8 +12,8 @@ ja: | |||
- "#{city_prefix}#{Name.last_name}#{city_suffix}" | |||
- "#{Name.last_name}#{city_suffix}" | |||
street_name: | |||
- "#{Name.first_name}#{street_suffix}" |
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.
Jpananese address not have street_suffix in first_name and street_suffix.
assert Faker::Name.name_with_middle.is_a? String | ||
assert Faker::Name.female_first_name.is_a? String |
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 is not written original Japanese I18n.
f6d2c11
to
5739c85
Compare
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.
Thanks for fixing these! I wonder if there's a custom matcher we could create assert_not_english
or something 🤔
47c76f2
to
4ef89ad
Compare
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.
looks good 👍
* add only japanese word spec * add assert_not_english method
* add only japanese word spec * add assert_not_english method
there expect translate in Japanese.
Japanese use only Kana words without alphabets.
but there is no any test. so I add test.