Return an empty String instead of null when creating a mock

mockito / mockito-kotlin

Using Mockito with Kotlin

MIT License

3.1k stars 200 forks source link

Return an empty String instead of null when creating a mock #373

Open jordigarcl opened 4 years ago

jordigarcl commented 4 years ago

See #372 for reference

I believe String values in mocks should return "" by default.

It is a historically 'special' type which has always had some special 'primitive-like' privileges. I assume people expect it to return a nice value as well!

Thank you for submitting a pull request! But first:

[x] Can you back your code up with tests?
[x] Keep your commits clean: squash your commits if necessary.

jordigarcl commented 4 years ago

Has this PR been reviewed yet?

wesalvaro commented 1 year ago

What does plain Mockito do?

wesalvaro commented 1 year ago

Plain Mockito returns null.