You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Adrian Cole <no...@github.com> on 2014/10/06 01:14:50 UTC
[jclouds] Move off confusing test names. (#552)
These tests have nothing to do with Async (at least not any more).
will kill BaseAsyncClientTest after this publishes and I change abiquo not to use it.
You can merge this Pull Request by running:
git pull https://github.com/adriancole/jclouds adrian.confusing-test-names
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds/pull/552
-- Commit Summary --
* Move off confusing test names.
-- File Changes --
M apis/atmos/src/test/java/org/jclouds/atmos/AtmosClientTest.java (4)
M apis/atmos/src/test/java/org/jclouds/atmos/blobstore/AtmosBlobRequestSignerTest.java (4)
M apis/cloudservers/src/test/java/org/jclouds/cloudservers/CloudServersClientTest.java (4)
M apis/cloudstack/src/test/java/org/jclouds/cloudstack/internal/BaseCloudStackApiTest.java (4)
M apis/cloudstack/src/test/java/org/jclouds/cloudstack/internal/BaseCloudStackExpectTest.java (4)
M apis/ec2/src/test/java/org/jclouds/ec2/features/BaseEC2ApiTest.java (4)
M apis/ec2/src/test/java/org/jclouds/ec2/internal/BaseEC2ExpectTest.java (4)
M apis/elasticstack/src/test/java/org/jclouds/elasticstack/ElasticStackApiTest.java (4)
M apis/rackspace-cloudidentity/src/test/java/org/jclouds/rackspace/cloudidentity/v1_1/internal/BaseKeystoneRestClientExpectTest.java (4)
M apis/s3/src/test/java/org/jclouds/s3/internal/BaseS3ClientExpectTest.java (4)
M apis/s3/src/test/java/org/jclouds/s3/internal/BaseS3ClientTest.java (4)
M apis/swift/src/test/java/org/jclouds/openstack/swift/CommonSwiftClientTest.java (4)
M apis/swift/src/test/java/org/jclouds/openstack/swift/internal/BaseSwiftExpectTest.java (4)
M apis/swift/src/test/java/org/jclouds/openstack/swift/internal/BaseSwiftKeystoneExpectTest.java (4)
M apis/vcloud/src/test/java/org/jclouds/vcloud/VCloudVersionsApiTest.java (5)
M apis/vcloud/src/test/java/org/jclouds/vcloud/compute/BaseVCloudComputeServiceExpectTest.java (4)
M apis/vcloud/src/test/java/org/jclouds/vcloud/internal/BaseVCloudApiTest.java (4)
M apis/vcloud/src/test/java/org/jclouds/vcloud/internal/VCloudLoginApiTest.java (4)
M blobstore/src/test/java/org/jclouds/blobstore/TransientBlobRequestSignerTest.java (4)
M blobstore/src/test/java/org/jclouds/blobstore/internal/BaseBlobSignerExpectTest.java (4)
M common/openstack/src/test/java/org/jclouds/openstack/internal/OpenStackAuthClientTest.java (4)
M common/openstack/src/test/java/org/jclouds/openstack/keystone/v1_1/internal/BaseKeystoneRestClientExpectTest.java (4)
M core/src/test/java/org/jclouds/http/IntegrationTestClientExpectTest.java (4)
M core/src/test/java/org/jclouds/rest/annotationparsing/DelegateAnnotationExpectTest.java (4)
M core/src/test/java/org/jclouds/rest/annotationparsing/JAXBResponseParserAnnotationExpectTest.java (4)
M core/src/test/java/org/jclouds/rest/annotationparsing/PATCHAnnotationExpectTest.java (4)
M core/src/test/java/org/jclouds/rest/annotationparsing/ProvidesAnnotationExpectTest.java (4)
M core/src/test/java/org/jclouds/rest/internal/BaseAsyncClientTest.java (3)
R core/src/test/java/org/jclouds/rest/internal/BaseRestAnnotationProcessingTest.java (2)
D core/src/test/java/org/jclouds/rest/internal/BaseRestClientExpectTest.java (25)
M providers/aws-ec2/src/test/java/org/jclouds/aws/ec2/features/BaseAWSEC2ApiTest.java (4)
M providers/aws-s3/src/test/java/org/jclouds/aws/s3/internal/BaseAWSS3ClientExpectTest.java (4)
M providers/azureblob/src/test/java/org/jclouds/azureblob/AzureBlobClientTest.java (4)
M providers/azureblob/src/test/java/org/jclouds/azureblob/binders/BindAzureBlobMetadataToRequestTest.java (4)
M providers/azureblob/src/test/java/org/jclouds/azureblob/blobstore/AzureBlobRequestSignerTest.java (4)
M providers/glesys/src/test/java/org/jclouds/glesys/GleSYSApiTest.java (4)
M providers/gogrid/src/test/java/org/jclouds/gogrid/features/BaseGoGridApiTest.java (4)
M providers/gogrid/src/test/java/org/jclouds/gogrid/features/BaseGoGridHttpApiExpectTest.java (4)
M providers/hpcloud-objectstorage/src/test/java/org/jclouds/hpcloud/objectstorage/internal/BaseHPCloudObjectStorageBlobStoreExpectTest.java (4)
-- Patch Links --
https://github.com/jclouds/jclouds/pull/552.patch
https://github.com/jclouds/jclouds/pull/552.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552
Re: [jclouds] Move off confusing test names. (#552)
Posted by Adrian Cole <no...@github.com>.
will address any feedback post-commit. this one is trivial, yet locking up the most important part of the unasyncing.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57957932
Re: [jclouds] Move off confusing test names. (#552)
Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1719](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1719/) FAILURE
Looks like there's a problem with this pull request
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57974725
Re: [jclouds] Move off confusing test names. (#552)
Posted by Adrian Cole <no...@github.com>.
cherry-picked into master, 1.8.x
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57958087
Re: [jclouds] Move off confusing test names. (#552)
Posted by Adrian Cole <no...@github.com>.
Closed #552.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#event-174313938
Re: [jclouds] Move off confusing test names. (#552)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1234](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1234/) ABORTED
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57969856
Re: [jclouds] Move off confusing test names. (#552)
Posted by Adrian Cole <ad...@gmail.com>.
ps failure was due to cloudfiles checkstyle. fixing in a follow-up
Re: [jclouds] Move off confusing test names. (#552)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1235](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1235/) UNSTABLE
Looks like there's a problem with this pull request
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57958297
Re: [jclouds] Move off confusing test names. (#552)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests-java-6 #146](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/146/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57956836
Re: [jclouds] Move off confusing test names. (#552)
Posted by Andrew Phillips <no...@github.com>.
Thanks for catching the abiquo dependency on this. +1 - looks good to me, too.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57967504
Re: [jclouds] Move off confusing test names. (#552)
Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1720](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1720/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57976870
Re: [jclouds] Move off confusing test names. (#552)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests-java-6 #145](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/145/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/552#issuecomment-57956273