90
100%

Ran 10 Feb 2020 12:58PM UTC

Jobs 5

Files 2

Run time 13s

Badge

Embed ▾

pending completion

Build # 90

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Fix memoization with Hash-like objects as parameters (#25)

After adding `**kwargs` due to warnings in Ruby 2.7,
Hash-like parameters (with `#to_hash` method) convert to Hash
while passed in dynamically created by Memery method.
For example, instances of `Sequel::Model`.

Found Ruby bug: https://bugs.ruby-lang.org/issues/14909

So, changes will be in Ruby 2.8.

And for now we can:

1.  Use `**(;{})` as the additional parameter at method call, what is ugly.
2.  Define memoized methods with `eval` and exactly expected arguments,
    what is dangerous (eval is evil).
3.  Suppress warnings about keyword arguments until here are no failing tests.

First, I've chose the 3th.

But then Jeremy Evans suggest to use `ruby2_keywords`. So, here we're!

Additional line of code offensed RuboCop (method length),
so I was forced to refactor method definition.

Run Details

279 of 279 relevant lines covered (100.0%)

18.87 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	90.1 (2.4)	10 Feb 2020 12:58PM UTC	100.0	Travis Job 90.1
2	90.2 (2.5)	10 Feb 2020 12:58PM UTC	100.0	Travis Job 90.2
3	90.3 (2.6)	10 Feb 2020 12:58PM UTC	100.0	Travis Job 90.3
4	90.4 (2.7)	10 Feb 2020 12:58PM UTC	100.0	Travis Job 90.4
5	90.5 (ruby-head)	10 Feb 2020 12:58PM UTC	100.0	Travis Job 90.5

Source Files on build 90

Detailed source file information is not available for this build.

tycooon / memery / 90
100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 90

tycooon / memery / 90 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 90

tycooon / memery / 90
100%

README BADGES
x