JpPrefecture::Prefecture.find(name: xxx)で空文字、nilを与えた場合の挙動がおかしい

[k-motoyan]

JpPrefecture::Prefecture.find(name: xxx)に対して、引数に空の文字列、nilが与えられた場合の挙動がおかしいように思います。

JpPrefecture::Prefecture.find(name: '') # => <JpPrefecture::Prefecture:0x007fdd21b17938 @code=1, @name="北海道", @name_e="Hokkaido", @name_h="ほっかいどう", @name_k="ホッカイドウ", @zips=[10000..70895, 400000..996509], @area="北海道">

JpPrefecture::Prefecture.find(name: nil) # => NoMethodError: undefined method `downcase' for nil:NilClass

これらの挙動については、nilを返すか、例外を投げるかしたほうが良いように思います。

[chocoby]

レポート & Pull Request ありがとうございます！

都道府県が見つからない場合は nil を返しているので、このケースも nil を返すようにした方が良さそうですね。早めに Pull Request を確認しますので少々お待ちください。

[k-motoyan]

マージありがとうございました 🙇

[chocoby]

ありがとうございました！