Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add back tests for limit fetch option #56

Open
jgaehring opened this issue Dec 1, 2021 · 0 comments
Open

Add back tests for limit fetch option #56

jgaehring opened this issue Dec 1, 2021 · 0 comments

Comments

@jgaehring
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant