Fix Map AppEngineCacheImpl::get(String[] keys) unwrap

ninjaframework / ninja-appengine

Google App Engine support for Ninja Framework

Apache License 2.0

40 stars 21 forks source link

Closed thibaultmeyer closed 1 year ago

raphaelbauer commented 2 years ago

Hey @thibaultmeyer - thanks for that fix 👍

two things:

Could you revert the many many many code formatting changes that do not matter? It's hard to see what you actually changed.
Could you add a test to the bug you fixed?

Many thanks!

Raphael

thibaultmeyer commented 2 years ago

@raphaelbauer

I have rollback everything that is not useful for this pull request. I added unit tests to avoid regression.