>   * @author Adrian Cole
>   */
>  // NOTE:without testName, this will not call @Before* and fail w/NPE during 
> surefire
>  @Test(groups = "unit", testName = "TransientBlobRequestSignerTest")
> -public class TransientBlobRequestSignerTest extends 
> BaseAsyncClientTest<LocalAsyncBlobStore> {
> +public class TransientBlobRequestSignerTest extends
> +      BaseAsyncClientTest<LocalBlobStore> {

Done. BaseAsyncClientTest extends BaseAsyncApiTest which in turn extends 
BaseRestApiTest. AFAICT there isn't anything about BaseAsyncClientTest 
(everything I see it does is done synchronously). I think this may have been 
named Async since the tests built on top this were supposed to be async. But I 
may be wrong.

Happy to change this in a separate commit. For now, would prefer this commit be 
only about LocalAsyncBlobStore <-> LocalBlobStore.

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

Reply via email to