feathers icon indicating copy to clipboard operation
feathers copied to clipboard

Published 20 hours ago • verified publisher icon feathersjs

test(adapter-tests): NotFound integer & add remove:NotFound tests

Open fratzinger opened this issue 1 year ago • 0 comments

This PR adds tests to @feathersjs/adapter-tests:

  1. add test for id as number: The tests for NotFound (.get, .update, .patch) use a string as id. This does not work for postgres + sequelize with serial ids because sequelize throws a GeneralError with invalid input syntax for type integer: "568225fbfe21222432e836ff" which is perfectly fine. I added tests for an integer id. So every adapter can pick which tests to use.
  2. .remove + NotFound was missing. Added it and also added .remove + NotFound (integer)

fratzinger avatar May 13 '24 09:05 fratzinger