Mock with kwargs as well

[ekohl]

In Ruby 2.7 sending the last arguments as a hash leads to a deprecation warning. By accepting the keyword arguments, the methods are fully wrapped.

[DavidS]

Thanks, Ewoud for this fix. I've addressed the rubocop issues in #13 , but trying to build a all-in-one integration branch in #14 showed some more issues. Please rebase this so we get a clean run.

[DavidS]

Also, if you're already adding ruby 2.7 compatibility, please also add it to the travis matris (see b52709c )

[ekohl]

Rebased. Technically the Gemfile change isn't needed, but I was in that file anyway to check.

[ekohl]

Looks like this breaks on older Ruby versions. Rather than stubbing it here, have you considered a gem like fakefs or setting up temporary directories?

I'll also consider this lower priority. My other 2 PRs would be nice to get into a release.

[DavidS]

I like the Gemfile change. The travis failure is now what I saw on #14 : seems like older rubies need a default value for kwargs?

[DavidS]

I have no preference either way.

[ekohl]

At this moment I have no time to finish this up due to other issues and it's not blocking me. I'll be happy if someone else picks this up, otherwise I'll get back to it once $work calms down again after the release.

[DavidS]

https://www.ruby-lang.org/en/news/2019/12/12/separation-of-positional-and-keyword-arguments-in-ruby-3-0/ has some guidance on how to make this work

[DavidS]

Fixed this in #22