> @@ -112,7 +120,86 @@ protected void configure() {
>        });
>        PostConstructable postConstructable = 
> i.getInstance(PostConstructable.class);
>        assert postConstructable.isStarted;
> +   }
> +
> +   @Test
> +   void testCloserClosingState() throws InterruptedException {
> +      Injector i = createInjector();

What is this test supposed to test? Simply the sequence of states or also some 
multi-threaded stuff? If so, what exactly? Also a 
[`CountDownLatch`](http://docs.oracle.com/javase/6/docs/api/java/util/concurrent/CountDownLatch.html)
 is probably better suited to the handoff than an `AtomicBoolean` with empty 
loops.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/32/files#r4723775

Reply via email to