Repository: jclouds Updated Branches: refs/heads/1.8.x 6258a4f4e -> c13751cc2
Added assertion for versioned OpenStack extension APIs. Required for the Neutron v2 API. Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/c13751cc Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/c13751cc Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/c13751cc Branch: refs/heads/1.8.x Commit: c13751cc2f095446c78375eb456d1dbcd93b68ab Parents: 6258a4f Author: Jeremy Daggett <[email protected]> Authored: Fri Aug 22 20:19:27 2014 -0700 Committer: Jeremy Daggett <[email protected]> Committed: Tue Sep 9 15:04:16 2014 -0700 ---------------------------------------------------------------------- .../v2_0/internal/BaseOpenStackMockTest.java | 23 ++++++++++++++------ 1 file changed, 16 insertions(+), 7 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/jclouds/blob/c13751cc/apis/openstack-keystone/src/test/java/org/jclouds/openstack/v2_0/internal/BaseOpenStackMockTest.java ---------------------------------------------------------------------- diff --git a/apis/openstack-keystone/src/test/java/org/jclouds/openstack/v2_0/internal/BaseOpenStackMockTest.java b/apis/openstack-keystone/src/test/java/org/jclouds/openstack/v2_0/internal/BaseOpenStackMockTest.java index c865922..83b047c 100644 --- a/apis/openstack-keystone/src/test/java/org/jclouds/openstack/v2_0/internal/BaseOpenStackMockTest.java +++ b/apis/openstack-keystone/src/test/java/org/jclouds/openstack/v2_0/internal/BaseOpenStackMockTest.java @@ -133,7 +133,7 @@ public class BaseOpenStackMockTest<A extends Closeable> { /** * Get a string from a resource - * + * * @param resourceName * The name of the resource. * @return The content of the resource @@ -157,14 +157,23 @@ public class BaseOpenStackMockTest<A extends Closeable> { Throwables.propagate(e); } } - + /** - * Ensures server received authentication request. + * Ensures server received an extensions request. Most OpenStack APIs that expose an {@link ExtensionApi} + * should use this method. */ public void assertExtensions(MockWebServer server) { + assertExtensions(server, ""); + } + + /** + * Ensures server received a <i>versioned</i> extensions request. This is necessary for testing APIs whose version + * is not included in the Endpoint's {@link Endpoint#getPublicURL() publicURL}. + */ + public void assertExtensions(MockWebServer server, String path) { assertTrue(server.getRequestCount() >= 1); try { - assertEquals(server.takeRequest().getRequestLine(), "GET /extensions HTTP/1.1"); + assertEquals(server.takeRequest().getRequestLine(), "GET " + path + "/extensions HTTP/1.1"); } catch (InterruptedException e) { Throwables.propagate(e); } @@ -172,7 +181,7 @@ public class BaseOpenStackMockTest<A extends Closeable> { /** * Ensures the request has a json header. - * + * * @param request * @see RecordedRequest */ @@ -182,7 +191,7 @@ public class BaseOpenStackMockTest<A extends Closeable> { /** * Ensures the request has a json header for the proper REST methods. - * + * * @param request * @param method * The request method (such as GET). @@ -198,7 +207,7 @@ public class BaseOpenStackMockTest<A extends Closeable> { /** * Ensures the request is json and has the same contents as the resource * file provided. - * + * * @param request * @param method * The request method (such as GET).
