Fixed issue #337 and included relevant test - ignore missing id when …

[Protoss78]

…query is used.

Summary

PR fixess issue #337
StashBefore now does not throw an error when the id is missing but a query parameter is present.
Test script was updated with a scenario where all documents are removed, by passing no id and an empty query object.

[eddyystop]

Do you think it would be cleaner to move

if (context.id === null || context.id === undefined) {
  throw new errors.BadRequest('Id is required. (stashBefore)');
}

to after the 'if' clause folllowing it?

Could you bump the patch semver to 4.2.3?

And thanks for this catch!

[Protoss78]

Added the version bump and moved the if statement. Definitely cleaner after the $disableStashBefore validation.

[eddyystop]

Thanks for this!