Yes, we should release this ObjectInserter.  But that's a lot of code to change, and I plan to move most tests to an in-memory repository which won't do compression/decompression of data.  Thus we wouldn't need to worry about leaking the Inflater handle during the test.

Also, the tests run in their own JVM, and do so pretty quickly right now.  Leaking Inflaters and waiting for the GC to fix things up with a finalizer isn't a big deal.  In fact, the GC might not even bother to run the finalizers, there isn't all that much garbage created during the tests that needs finalization, and the JVM might just terminate before its cleaned up.

Long story short, its just not worth it given what we leak, and the amount of code that needs to change in current tests.

Newer tests should however release stuff, yes, just to keep the programming style consistent.