update at getGroceriesWithDepIngKey tests

atherdon commented 5 years ago

[ ] tests that checking properties - move to a separated function. you can read our other test files and find a solution.
[ ] Right now these tests didn't cover a lot of cases. Try to jump on grocery.json and delete an id property. Try to delete name, make name an empty string. Delete all data from one object, only keep id, name(no departments, no ingredients inside)
[ ] ingredients for grocery lists stored at Ingredients folder. they connected via file name. just delete one of the files, that should be connected to grocery and you wouldn't see an error.
[ ] Test Deep structure takes 6000ms how to reduce this number?
[ ] we have test Ingredient by length? WTF it's doing and why we need it - please explain.
[ ] making grocery list name as empty string didn't generate an error

@shubhamrajput it's for you

atherdon commented 5 years ago

@shubhamrajput any questions/updates?

atherdon commented 5 years ago

@shubhamrajput please give me some updates

GroceriStar / groceristar-fetch