Ah, I missed that. I used it when I originally wrote the test (has been annoying me for a while to have to do .directory().getAbsolutePath() all the time), but then I found the TestBatchInserter class, and changed it to use the idiom from there to get the batch inserter.

I'd vote to keep it, it's useful when we write new tests.