Fix ClassLoader leak

[stuartwdouglas]

No description provided.

[Sanne]

Looks good, but I also noticed that often the QuarkusClassloader isn't actually closed, so is this going to be fired?

(I don't know how it happens, I have only seen several QuarkusClassloader in heap dumps which have the closed boolean set to false - maybe a false alarm as I've not figured out yet if what I'm observing if this is a legit situation)

[stuartwdouglas]

@Sanne it should be closed, are you sure you are not seeing the active CL for the currently running tests?

Also sometimes the CL is supposed to last the life of the test run, depending on the type of tests.

[Sanne]

Thanks, I'll double check tomorrow.

I'm fairly sure I saw more than one QuarkusClassloader open, but now that you explained in a parallel conversation about the CL and Vertx instances needed for dev-ui that might explain it.