Repository: jclouds-labs-openstack Updated Branches: refs/heads/1.8.x 6b4ab4fef -> 16e87b7f0
Prefer Boolean over primitive boolean in OpenStack Neutron domain classes Project: http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/repo Commit: http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/commit/16e87b7f Tree: http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/tree/16e87b7f Diff: http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/diff/16e87b7f Branch: refs/heads/1.8.x Commit: 16e87b7f04b4193f266cb33773a9deffa23079ea Parents: 6b4ab4f Author: Jeremy Daggett <[email protected]> Authored: Fri Sep 26 08:37:06 2014 -0700 Committer: Jeremy Daggett <[email protected]> Committed: Wed Oct 1 09:28:43 2014 -0700 ---------------------------------------------------------------------- .../neutron/v2/domain/ExternalGatewayInfo.java | 8 +++++++- .../openstack/neutron/v2/domain/Network.java | 8 ++++---- .../jclouds/openstack/neutron/v2/domain/Port.java | 18 +++++++++--------- .../openstack/neutron/v2/domain/Router.java | 6 +++--- .../neutron/v2/extensions/RouterApiMockTest.java | 10 +++++----- .../neutron/v2/features/PortApiMockTest.java | 12 ++++++------ 6 files changed, 34 insertions(+), 28 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/blob/16e87b7f/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/ExternalGatewayInfo.java ---------------------------------------------------------------------- diff --git a/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/ExternalGatewayInfo.java b/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/ExternalGatewayInfo.java index cab2d79..4dd6505 100644 --- a/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/ExternalGatewayInfo.java +++ b/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/ExternalGatewayInfo.java @@ -109,7 +109,13 @@ public class ExternalGatewayInfo { return this; } - public Builder enableSnat(boolean enableSnat) { + /** + * Provide the enableSnat status to the ExternalGatewayInfo's Builder. + * + * @return the Builder. + * @see ExternalGatewayInfo#getEnableSnat() + */ + public Builder enableSnat(Boolean enableSnat) { this.enableSnat = enableSnat; return this; } http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/blob/16e87b7f/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Network.java ---------------------------------------------------------------------- diff --git a/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Network.java b/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Network.java index 74a6ec5..ac40019 100644 --- a/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Network.java +++ b/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Network.java @@ -181,7 +181,7 @@ public class Network { * @return the adminStateUp of the Network */ @Nullable - public Boolean isAdminStateUp() { + public Boolean getAdminStateUp() { return adminStateUp; } @@ -192,7 +192,7 @@ public class Network { * @return true if the network resource can be accessed by any tenant or not, false if not */ @Nullable - public Boolean isShared() { + public Boolean getShared() { return shared; } @@ -233,7 +233,7 @@ public class Network { * @return the external of the Network */ @Nullable - public Boolean isExternal() { + public Boolean getExternal() { return external; } @@ -241,7 +241,7 @@ public class Network { * @return the portSecurity of the Network */ @Nullable - public Boolean isPortSecurity() { + public Boolean getPortSecurity() { return portSecurity; } http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/blob/16e87b7f/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Port.java ---------------------------------------------------------------------- diff --git a/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Port.java b/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Port.java index 2c3d95d..a9ab61e 100644 --- a/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Port.java +++ b/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Port.java @@ -252,7 +252,7 @@ public class Port { * @return the administrative state of port. If false, port does not forward packets. */ @Nullable - public boolean isAdminStateUp() { + public Boolean getAdminStateUp() { return adminStateUp; } @@ -354,7 +354,7 @@ public class Port { * @return the portSecurity of the Port */ @Nullable - public Boolean isPortSecurity() { + public Boolean getPortSecurity() { return portSecurity; } @@ -370,7 +370,7 @@ public class Port { * @return the macLearning of the Port */ @Nullable - public Boolean isMacLearning() { + public Boolean getMacLearning() { return macLearning; } @@ -510,9 +510,9 @@ public class Port { * Provide the adminStateUp to the Port's Builder. * * @return the Builder. - * @see Port#isAdminStateUp() + * @see Port#getAdminStateUp() */ - public ParameterizedBuilderType adminStateUp(boolean adminStateUp) { + public ParameterizedBuilderType adminStateUp(Boolean adminStateUp) { port.adminStateUp = adminStateUp; return self(); } @@ -647,9 +647,9 @@ public class Port { * Provide the portSecurity to the Port's Builder. * * @return the Builder. - * @see Port#isPortSecurity() + * @see Port#getPortSecurity() */ - public ParameterizedBuilderType portSecurity(boolean portSecurity) { + public ParameterizedBuilderType portSecurity(Boolean portSecurity) { port.portSecurity = portSecurity; return self(); } @@ -669,9 +669,9 @@ public class Port { * Provide the macLearning to the Port's Builder. * * @return the Builder. - * @see Port#isMacLearning() + * @see Port#getMacLearning() */ - public ParameterizedBuilderType macLearning(boolean macLearning) { + public ParameterizedBuilderType macLearning(Boolean macLearning) { port.macLearning = macLearning; return self(); } http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/blob/16e87b7f/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Router.java ---------------------------------------------------------------------- diff --git a/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Router.java b/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Router.java index 5550c89..0b06531 100644 --- a/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Router.java +++ b/openstack-neutron/src/main/java/org/jclouds/openstack/neutron/v2/domain/Router.java @@ -111,7 +111,7 @@ public class Router { * @return the adminStateUp of the Router */ @Nullable - public Boolean isAdminStateUp() { + public Boolean getAdminStateUp() { return adminStateUp; } @@ -209,9 +209,9 @@ public class Router { * Provide the adminStateUp to the Router's Builder. * * @return the Builder. - * @see Router#isAdminStateUp() + * @see Router#getAdminStateUp() */ - public ParameterizedBuilderType adminStateUp(boolean adminStateUp) { + public ParameterizedBuilderType adminStateUp(Boolean adminStateUp) { router.adminStateUp = adminStateUp; return self(); } http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/blob/16e87b7f/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/extensions/RouterApiMockTest.java ---------------------------------------------------------------------- diff --git a/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/extensions/RouterApiMockTest.java b/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/extensions/RouterApiMockTest.java index 346e820..00232f8 100644 --- a/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/extensions/RouterApiMockTest.java +++ b/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/extensions/RouterApiMockTest.java @@ -58,7 +58,7 @@ public class RouterApiMockTest extends BaseNeutronApiMockTest { NeutronApi neutronApi = api(server.getUrl("/").toString(), "openstack-neutron", overrides); RouterApi api = neutronApi.getRouterApi("RegionOne").get(); - Router.CreateRouter createRouter = Router.createBuilder().name("another_router").adminStateUp(true) + Router.CreateRouter createRouter = Router.createBuilder().name("another_router").adminStateUp(Boolean.TRUE) .externalGatewayInfo(ExternalGatewayInfo.builder().networkId("8ca37218-28ff-41cb-9b10-039601ea7e6b").build()) .build(); @@ -77,7 +77,7 @@ public class RouterApiMockTest extends BaseNeutronApiMockTest { assertEquals(router.getName(), "another_router"); assertEquals(router.getExternalGatewayInfo().getNetworkId(), "8ca37218-28ff-41cb-9b10-039601ea7e6b"); assertEquals(router.getStatus(), NetworkStatus.ACTIVE); - assertEquals(router.isAdminStateUp().booleanValue(), true); + assertEquals(router.getAdminStateUp(), Boolean.TRUE); assertEquals(router.getId(), "8604a0de-7f6b-409a-a47c-a1cc7bc77b2e"); assertEquals(router.getTenantId(), "6b96ff0cb17a4b859e1e575d221683d3"); } finally { @@ -96,7 +96,7 @@ public class RouterApiMockTest extends BaseNeutronApiMockTest { NeutronApi neutronApi = api(server.getUrl("/").toString(), "openstack-neutron", overrides); RouterApi api = neutronApi.getRouterApi("RegionOne").get(); - Router.CreateRouter createRouter = Router.createBuilder().name("another_router").adminStateUp(true) + Router.CreateRouter createRouter = Router.createBuilder().name("another_router").adminStateUp(Boolean.TRUE) .externalGatewayInfo(ExternalGatewayInfo.builder().networkId("8ca37218-28ff-41cb-9b10-039601ea7e6b").build()) .build(); @@ -250,7 +250,7 @@ public class RouterApiMockTest extends BaseNeutronApiMockTest { assertEquals(router.getName(), "router1"); assertEquals(router.getExternalGatewayInfo().getNetworkId(), "3c5bcddd-6af9-4e6b-9c3e-c153e521cab8"); assertEquals(router.getStatus(), NetworkStatus.ACTIVE); - assertEquals(router.isAdminStateUp().booleanValue(), true); + assertEquals(router.getAdminStateUp(), Boolean.TRUE); assertEquals(router.getId(), "a9254bdb-2613-4a13-ac4c-adc581fba50d"); assertEquals(router.getTenantId(), "33a40233088643acb66ff6eb0ebea679"); } finally { @@ -315,7 +315,7 @@ public class RouterApiMockTest extends BaseNeutronApiMockTest { assertEquals(router.getName(), "another_router"); assertEquals(router.getExternalGatewayInfo().getNetworkId(), "8ca37218-28ff-41cb-9b10-039601ea7e6b"); assertEquals(router.getStatus(), NetworkStatus.ACTIVE); - assertEquals(router.isAdminStateUp().booleanValue(), true); + assertEquals(router.getAdminStateUp(), Boolean.TRUE); assertEquals(router.getId(), "8604a0de-7f6b-409a-a47c-a1cc7bc77b2e"); assertEquals(router.getTenantId(), "6b96ff0cb17a4b859e1e575d221683d3"); } finally { http://git-wip-us.apache.org/repos/asf/jclouds-labs-openstack/blob/16e87b7f/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/features/PortApiMockTest.java ---------------------------------------------------------------------- diff --git a/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/features/PortApiMockTest.java b/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/features/PortApiMockTest.java index a9a2dc4..8f7ebc8 100644 --- a/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/features/PortApiMockTest.java +++ b/openstack-neutron/src/test/java/org/jclouds/openstack/neutron/v2/features/PortApiMockTest.java @@ -63,7 +63,7 @@ public class PortApiMockTest extends BaseNeutronApiMockTest { Port.CreatePort createPort = Port.createBuilder("6aeaf34a-c482-4bd3-9dc3-7faf36412f12") .name("port1") - .adminStateUp(true) + .adminStateUp(Boolean.TRUE) .deviceId("d6b4d3a5-c700-476f-b609-1493dd9dadc0") .allowedAddressPairs(ImmutableSet.of(AddressPair.builder("12", "111.222.333.444").build())) .build(); @@ -110,7 +110,7 @@ public class PortApiMockTest extends BaseNeutronApiMockTest { Port.CreatePort createPort = Port.createBuilder("6aeaf34a-c482-4bd3-9dc3-7faf36412f12") .name("port1") - .adminStateUp(true) + .adminStateUp(Boolean.TRUE) .deviceId("d6b4d3a5-c700-476f-b609-1493dd9dadc0") .allowedAddressPairs(ImmutableSet.of(AddressPair.builder("12", "111.222.333.444").build())) .build(); @@ -248,14 +248,14 @@ public class PortApiMockTest extends BaseNeutronApiMockTest { Port.CreatePort createPort1 = Port.createBuilder("64239a54-dcc4-4b39-920b-b37c2144effa") .name("port1") - .adminStateUp(true) + .adminStateUp(Boolean.TRUE) .deviceId("24df1d04-d5cb-41e1-8de5-61ed77c558df") .securityGroups(ImmutableSet.of("dbc107f4-afcd-4d5a-9352-f68f82241d5b")) .build(); Port.CreatePort createPort2 = Port.createBuilder("e6031bc2-901a-4c66-82da-f4c32ed89406") .name("port2") - .adminStateUp(false) + .adminStateUp(Boolean.FALSE) .securityGroups( ImmutableSet.of("8bf3f7cc-8471-40b1-815f-9da47e79775b", "dbc107f4-afcd-4d5a-9352-f68f82241d5b")) .build(); @@ -293,14 +293,14 @@ public class PortApiMockTest extends BaseNeutronApiMockTest { Port.CreatePort createPort1 = Port.createBuilder("64239a54-dcc4-4b39-920b-b37c2144effa") .name("port1") - .adminStateUp(true) + .adminStateUp(Boolean.TRUE) .deviceId("24df1d04-d5cb-41e1-8de5-61ed77c558df") .securityGroups(ImmutableSet.of("dbc107f4-afcd-4d5a-9352-f68f82241d5b")) .build(); Port.CreatePort createPort2 = Port.createBuilder("e6031bc2-901a-4c66-82da-f4c32ed89406") .name("port2") - .adminStateUp(false) + .adminStateUp(Boolean.FALSE) .securityGroups( ImmutableSet.of("8bf3f7cc-8471-40b1-815f-9da47e79775b", "dbc107f4-afcd-4d5a-9352-f68f82241d5b")) .build();
