farmOS / farmOS.js

A JavaScript library for working with farmOS data structures and interacting with farmOS servers.
MIT License
17 stars 13 forks source link

Add back tests for `limit` fetch option #56

Open jgaehring opened 2 years ago

jgaehring commented 2 years ago

I got an error from my tests in CI that I didn't encounter locally: https://github.com/jgaehring/farmOS.js/runs/4377209692?check_suite_focus=true#step:8:115

Scratched my head for a while on this, until I realized that I am using a much larger testing database locally, with 1000+ assets, whereas the CI environment starts with zero assets. This is not a major issue, and I've simply deleted those tests for now (e2ea010), but it would be nice to revert that commit once I have better test data to work with, hopefully in the near future.