Added threads tests

[cszsol]

No description provided.

[WonderPG]

The endpoints should rather be tested with the dependency override mechanism of FastAPI. We had these tests in neuroagent previously, and in theory they had little links to langchain so few of them will need modifications. Thank you for handling this though !

[kanesoban]

The endpoints should rather be tested with the dependency override mechanism of FastAPI. We had these tests in neuroagent previously, and in theory they had little links to langchain so few of them will need modifications. Thank you for handling this though !

My thinking was that unit tests should only test the function and mock anything else

https://stackoverflow.com/questions/5357601/whats-the-difference-between-unit-tests-and-integration-tests

When you don't mock the depended functions and let the function have side effects such as writing to a DB, that is usually called an integration test.
But since we don't have dedicated integration tests, I guess it is OK to do it this way.