> @@ -51,7 +53,10 @@ public GoogleComputeEngineServiceLiveTest() {
>     @Override
>     protected Properties setupProperties() {
>        Properties props = super.setupProperties();
> -      setCredentialFromPemFile(props, provider + ".credential");
> +      if 
> (!System.getProperties().containsKey(OAuthProperties.CREDENTIAL_TYPE)
> +            || 
> !System.getProperty(OAuthProperties.CREDENTIAL_TYPE).equalsIgnoreCase(CredentialType.BEARER_TOKEN_CREDENTIALS.toString()))
>  {

[minor, since this is a test] Would
```
if(!System.getProperty(OAuthProperties.CREDENTIAL_TYPE, "")
    .equalsIgnoreCase(CredentialType.BEARER_TOKEN_CREDENTIALS.toString())) {
```
also work here? I'm assuming `""` will *not* be equal to 
BEARER_TOKEN_CREDENTIALS, so this would also be `false` if the property is not 
present.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/34/files#r15602663

Reply via email to