Issue 617:address NPE failures in CI

providers/gogrid/src/test/java/org/jclouds/gogrid/GoGridLiveTestDisabled.java

@@ -81,7 +81,8 @@ import com.google.inject.Module;
  * 
  * @author Oleksiy Yarmula
  */
-@Test(enabled = true, groups = "live")
+// NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(enabled = false, groups = "live", testName = "GoGridLiveTestDisabled")
 public class GoGridLiveTestDisabled {
 
    private GoGridClient client;

providers/gogrid/src/test/java/org/jclouds/gogrid/binders/BindIdsToQueryParamsTest.java

@@ -35,6 +35,8 @@ import com.sun.jersey.api.uri.UriBuilderImpl;
  * 
  * @author Oleksiy Yarmula
  */
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "BindIdsToQueryParamsTest")
 public class BindIdsToQueryParamsTest {
 
    @Test

providers/gogrid/src/test/java/org/jclouds/gogrid/binders/BindNamesToQueryParamsTest.java

@@ -35,6 +35,8 @@ import com.sun.jersey.api.uri.UriBuilderImpl;
  * 
  * @author Oleksiy Yarmula
  */
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "BindNamesToQueryParamsTest")
 public class BindNamesToQueryParamsTest {
 
    @Test

providers/gogrid/src/test/java/org/jclouds/gogrid/compute/GoGridComputeServiceLiveTest.java

@@ -25,7 +25,8 @@ import org.testng.annotations.Test;
 /**
  * @author Oleksiy Yarmula
  */
-@Test(groups = "live", enabled = true, sequential = true)
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "live", singleThreaded = true, testName = "GoGridComputeServiceLiveTest")
 public class GoGridComputeServiceLiveTest extends BaseComputeServiceLiveTest {
    public GoGridComputeServiceLiveTest() {
       provider = "gogrid";

providers/gogrid/src/test/java/org/jclouds/gogrid/compute/GoGridTemplateBuilderLiveTest.java

@@ -38,7 +38,8 @@ import com.google.common.collect.ImmutableSet;
  * 
  * @author Adrian Cole
  */
-@Test(groups = "live")
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "live", singleThreaded = true, testName = "GoGridTemplateBuilderLiveTest")
 public class GoGridTemplateBuilderLiveTest extends BaseTemplateBuilderLiveTest {
 
    public GoGridTemplateBuilderLiveTest() {

providers/gogrid/src/test/java/org/jclouds/gogrid/compute/config/GoGridComputeServiceContextModuleTest.java

@@ -24,7 +24,8 @@ import org.testng.annotations.Test;
 /**
  * @author Adrian Cole
  */
-@Test(groups = "unit")
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "GoGridComputeServiceContextModuleTest")
 public class GoGridComputeServiceContextModuleTest {
 
    public void testAllStatusCovered() {

providers/gogrid/src/test/java/org/jclouds/gogrid/compute/functions/ServerToNodeMetadataTest.java

@@ -54,7 +54,8 @@ import com.google.common.collect.ImmutableSet;
 /**
  * @author Adrian Cole
  */
-@Test(groups = "unit")
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "ServerToNodeMetadataTest")
 public class ServerToNodeMetadataTest {
 
    @SuppressWarnings("unchecked")

providers/gogrid/src/test/java/org/jclouds/gogrid/compute/options/GoGridTemplateOptionsTest.java

@@ -28,6 +28,8 @@ import org.testng.annotations.Test;
  * 
  * @author Andrew Kennedy
  */
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "GoGridTemplateOptionsTest")
 public class GoGridTemplateOptionsTest {
    @Test
    public void testAs() {

providers/gogrid/src/test/java/org/jclouds/gogrid/functions/ParseCredentialsFromJsonResponseTest.java

@@ -47,7 +47,8 @@ import com.google.inject.Provides;
 /**
  * @author Adrian Cole
  */
-@Test(groups = "unit")
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "ParseCredentialsFromJsonResponseTest")
 public class ParseCredentialsFromJsonResponseTest {
 
    @Test(expectedExceptions = IllegalStateException.class)

providers/gogrid/src/test/java/org/jclouds/gogrid/functions/ParseErrorFromJsonResponseTest.java

@@ -35,6 +35,8 @@ import com.google.inject.Injector;
 /**
  * @author Oleksiy Yarmula
  */
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "ParseErrorFromJsonResponseTest")
 public class ParseErrorFromJsonResponseTest {
 
    Injector i = Guice.createInjector(new GsonModule() {

providers/gogrid/src/test/java/org/jclouds/gogrid/functions/ParseJobsFromJsonResponseTest.java

@@ -51,7 +51,8 @@ import com.google.inject.Provides;
 /**
  * @author Oleksiy Yarmula
  */
-@Test(groups = "unit")
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "ParseJobsFromJsonResponseTest")
 public class ParseJobsFromJsonResponseTest {
 
    @Test

providers/gogrid/src/test/java/org/jclouds/gogrid/functions/ParseLoadBalancersFromJsonResponseTest.java

@@ -54,7 +54,8 @@ import com.google.inject.Provides;
 /**
  * @author Oleksiy Yarmula
  */
-@Test(groups = "unit")
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "ParseLoadBalancersFromJsonResponseTest")
 public class ParseLoadBalancersFromJsonResponseTest {
 
    @Test

providers/gogrid/src/test/java/org/jclouds/gogrid/functions/ParseServerNameToCredentialsMapFromJsonResponseTest.java

@@ -47,7 +47,8 @@ import com.google.inject.Provides;
 /**
  * @author Oleksiy Yarmula
  */
-@Test(groups = "unit")
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "ParseServerNameToCredentialsMapFromJsonResponseTest")
 public class ParseServerNameToCredentialsMapFromJsonResponseTest {
 
    @Test

providers/gogrid/src/test/java/org/jclouds/gogrid/functions/ParseServersFromJsonResponseTest.java

@@ -57,7 +57,8 @@ import com.google.inject.Provides;
  * 
  * @author Adrian Cole
  */
-@Test(groups = "unit")
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "ParseServersFromJsonResponseTest")
 public class ParseServersFromJsonResponseTest {
 
    @Test

providers/gogrid/src/test/java/org/jclouds/gogrid/handlers/GoGridErrorHandlerTest.java

@@ -40,6 +40,8 @@ import com.google.inject.Guice;
  * 
  * @author Oleksiy Yarmula
  */
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "GoGridErrorHandlerTest")
 public class GoGridErrorHandlerTest {
 
    @Test

providers/gogrid/src/test/java/org/jclouds/gogrid/options/AddServerOptionsTest.java

@@ -32,6 +32,8 @@ import org.testng.annotations.Test;
  * 
  * @author Adrian Cole
  */
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "AddServerOptionsTest")
 public class AddServerOptionsTest {
 
    @Test

providers/gogrid/src/test/java/org/jclouds/gogrid/predicates/ServerLatestJobCompletedTest.java

@@ -35,6 +35,8 @@ import com.google.common.collect.ImmutableSet;
 /**
  * @author Oleksiy Yarmula
  */
+//NOTE:without testName, this will not call @Before* and fail w/NPE during surefire
+@Test(groups = "unit", testName = "ServerLatestJobCompletedTest")
 public class ServerLatestJobCompletedTest {
 
     @Test