ets_cache: Cache {ok, Val} rather than Val

[weiss]

If the update_fun() returns ok, cache {ok, Val} rather than Val. Otherwise the ets_cache:update() caller would always have to specify {ok, Val}.

[mremond]

It makes sense but I think we should maybe add a unit test to prevent regression.