micro-analytics / micro-analytics-cli

Public analytics as a Node.js microservice. No sysadmin experience required! 📈
MIT License
734 stars 39 forks source link

npm test not working with locally linked adapter-flat-file-db #34

Closed sean-roberts closed 7 years ago

sean-roberts commented 7 years ago

master tests are failing - also the ci build tag isn't reflecting this correctly

sean-roberts commented 7 years ago

ah, weird. Has something to do with having the db adaptor linked locally.

sean-roberts commented 7 years ago

Not too familiar with this problem. But it seems that because it's a linked project the file-db is not in the require path that jest is going to mock, thus it's still going to use the actual functions.

Not sure that there is a way to fix this that's very elegant...

sean-roberts commented 7 years ago

how is your env set up to run npm test? I keep getting lots of flak for async/await @mxstbr

mxstbr commented 7 years ago

Yeah testing changes in the adapter is a bit of a pain right now, I'm afraid...