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.

Sign up for GitHub

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

Jump to bottom

引数が 4つのバージョンの IsMailAddress の単体テストを追加する #811

Closed
m-tmatma opened this issue Mar 24, 2019 · 2 comments
Closed

引数が 4つのバージョンの IsMailAddress の単体テストを追加する #811

m-tmatma opened this issue Mar 24, 2019 · 2 comments
Labels
UnitTest
Milestone
v2.4.0

Comments

@m-tmatma
Copy link
Member

要望機能

引数が 4つのバージョンの IsMailAddress の単体テストを追加する

#808 (comment)

単体テストを復活する事自体は良いと思いますが、試験している関数が offset 引数が無い方だけというのが気になりました。まぁでも間接的には試験している事になるし、offset 引数を指定した場合のテストのバリエーションは後から追加すれば良いのかもしれませんね。
@m-tmatma m-tmatma mentioned this issue Mar 24, 2019
Merged
@ds14050 ds14050 mentioned this issue Mar 31, 2019
Merged
@ds14050
Copy link
Contributor

ds14050 commented Mar 31, 2019

他人事ではないので最低限ですがテストを用意してみました。>#823

@beru beru added the UnitTest label Apr 26, 2019
@m-tmatma
Copy link
Member Author

マージされたのでクローズ

@m-tmatma m-tmatma added this to the v2.4.0 milestone Apr 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
UnitTest
Projects
None yet
Milestone
v2.4.0
Development

No branches or pull requests
3 participants
@beru @m-tmatma @ds14050