Incr doesn't initialize default value if ttl is expired

[khaskelberg]

Hey!

Currently when you incr on expired value it increases and returns expired value and doesn't invalidate it. I personally expected that it would increase the default value, if ttl is expired.

Here is an example test khaskelberg@49438f8

test "incr initializes default value if ttl is expired", %{cache: cache} do
  assert cache.incr(:another_counter_with_ttl, 1, ttl: 1000) == 1
  assert cache.incr(:another_counter_with_ttl) == 2

  :ok = Process.sleep(1010)
  assert cache.incr(:another_counter_with_ttl, 1, ttl: 1000) == 1
end

And here is the test ouput

  1) test entry: incr initializes default value if ttl is expired (Nebulex.Adapters.LocalEtsTest)
     test/nebulex/adapters/local_ets_test.exs:94
     Assertion with == failed
     code:  assert cache.incr(:another_counter_with_ttl, 1, ttl: 1000) == 1
     left:  3
     right: 1
     stacktrace:
       test/nebulex/adapters/local_ets_test.exs:99: (test)

.....................................................................................................................

  2) test entry: incr initializes default value if ttl is expired (Nebulex.Adapters.LocalWithShardsTest)
     test/nebulex/adapters/local_shards_test.exs:94
     Assertion with == failed
     code:  assert cache.incr(:another_counter_with_ttl, 1, ttl: 1000) == 1
     left:  3
     right: 1
     stacktrace:
       test/nebulex/adapters/local_shards_test.exs:99: (test)

[cabol]

Hey! Good catch! I pushed a fix already, please try it out. I will close the issue and publish a new release but feel free to re-open it if you find an issue with the fix. Thanks!