commit 57372f31d3d98dd2dc54a9cccd9fa2d2832f0e76
author David Pursehouse <dpursehouse@collab.net> Sat Apr 21 18:30:21 2018 +0200
committer David Pursehouse <dpursehouse@collab.net> Sat Apr 21 18:42:38 2018 +0200
tree 36161a776e9097c7ba11635d188d0d1bfcb371eb
parent 045c8337a7b39032418d7924f951cc87cef4dc37

Delete index after each test in elasticsearch tests

Since change I6539845e0 the index is not deleted after each test method
because of a problem where the index was not deleted quickly enough and
the index for the next test could not be created due to the previous one
still existing. The indices are all kept and then only deleted at the
end of the run.

Leaving all the indices results in running out of file descriptors which
results in tests failing when many tests are run in sequence.

Since we now give each test index a unique name, we no longer suffer from
the original problem described above. We can safely attempt to delete the
index and it doesn't matter if it does not get deleted quickly enough and
still exists when the next test is started.

The tests for accounts, groups and projects are not currently affected by
this problem because they have far fewer test methods than the tests for
changes. The same fix is done for those tests anyway to make sure we do
not run into the same problem in future.

Bug: Issue 8816
Change-Id: I1646f5b802dd2d15b974d4c4d75ab337c4c6f462