We have some tests that don't close the search integrator they create, resulting in leaks: threads still running or files not deleted when other tests start, or, more importantly, Elasticsearch indexes not deleted after the test finishes. We should find the misbehaving tests and fix them. Also, it would be great to add a forbidden API rule so that we don't try to build search integrators directly, ever (we should always use a safe API that will ensure the integrator will be closed eventually). |