TLDR:
@feathersjs/memory: .update() with a query (resulting in NotFound) updates data instead of preventing the change.

adapter-tests has a test case for .update + id + query but it produces a false positive (at least for @feathersjs/memory). It checks for NotFound but does not safely detect if definitely nothing was changed.

In case of an .update() with a query, the current implementation of @feathersjs/memory changes the data first, stores the changed data and then throws afterwards.

So I included an explicit check in @feathersjs/adapter-tests and fixed it by passing params to the first _.get() in update.

While being at it, I also replaced @feathersjs/commons with native methods (third commit).