Skip to content

Commit eb36f92

Browse files
committed
Fix minor style issues with SubnetworkInfo and test class
1 parent d19f7c6 commit eb36f92

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

gcloud-java-compute/src/main/java/com/google/gcloud/compute/SubnetworkInfo.java

+5-5
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ public Subnetwork apply(SubnetworkInfo subnetwork) {
6767
/**
6868
* A builder for {@code SubnetworkInfo} objects.
6969
*/
70-
public static abstract class Builder {
70+
public abstract static class Builder {
7171

7272
abstract Builder id(String id);
7373

@@ -81,15 +81,15 @@ public static abstract class Builder {
8181
/**
8282
* Sets an optional textual description of the subnetwork.
8383
*/
84-
abstract public Builder description(String description);
84+
public abstract Builder description(String description);
8585

8686
abstract Builder gatewayAddress(String gatewayAddress);
8787

8888
/**
8989
* Sets the identity of the network to which this subnetwork belongs. Only networks that are in
9090
* subnet mode can have subnetworks.
9191
*/
92-
abstract public Builder network(NetworkId network);
92+
public abstract Builder network(NetworkId network);
9393

9494
/**
9595
* Sets the range of internal IPv4 addresses that are owned by this subnetwork. This range must
@@ -98,12 +98,12 @@ public static abstract class Builder {
9898
*
9999
* @see <a href="https://wikipedia.org/wiki/Classless_Inter-Domain_Routing">CIDR</a>
100100
*/
101-
abstract public Builder ipRange(String ipRange);
101+
public abstract Builder ipRange(String ipRange);
102102

103103
/**
104104
* Creates a {@code SubnetworkInfo} object.
105105
*/
106-
abstract public SubnetworkInfo build();
106+
public abstract SubnetworkInfo build();
107107
}
108108

109109
static final class BuilderImpl extends Builder {

gcloud-java-compute/src/test/java/com/google/gcloud/compute/SubnetworkInfoTest.java

+7-7
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,9 @@ public class SubnetworkInfoTest {
3030
SubnetworkId.of("project", "region", "subnetwork");
3131
private static final String GATEWAY_ADDRESS = "192.168.1.1";
3232
private static final NetworkId NETWORK_ID = NetworkId.of("project", "network");
33-
private static final String IP_CIDR_RANGE = "192.168.0.0/16";
33+
private static final String IP_RANGE = "192.168.0.0/16";
3434
private static final SubnetworkInfo SUBNETWORK_INFO =
35-
SubnetworkInfo.builder(SUBNETWORK_ID, NETWORK_ID, IP_CIDR_RANGE)
35+
SubnetworkInfo.builder(SUBNETWORK_ID, NETWORK_ID, IP_RANGE)
3636
.id(ID)
3737
.creationTimestamp(CREATION_TIMESTAMP)
3838
.description(DESCRIPTION)
@@ -51,7 +51,7 @@ public void testToBuilder() {
5151

5252
@Test
5353
public void testToBuilderIncomplete() {
54-
SubnetworkInfo subnetworkInfo = SubnetworkInfo.of(SUBNETWORK_ID, NETWORK_ID, IP_CIDR_RANGE);
54+
SubnetworkInfo subnetworkInfo = SubnetworkInfo.of(SUBNETWORK_ID, NETWORK_ID, IP_RANGE);
5555
assertEquals(subnetworkInfo, subnetworkInfo.toBuilder().build());
5656
}
5757

@@ -63,25 +63,25 @@ public void testBuilder() {
6363
assertEquals(DESCRIPTION, SUBNETWORK_INFO.description());
6464
assertEquals(GATEWAY_ADDRESS, SUBNETWORK_INFO.gatewayAddress());
6565
assertEquals(NETWORK_ID, SUBNETWORK_INFO.network());
66-
assertEquals(IP_CIDR_RANGE, SUBNETWORK_INFO.ipRange());
66+
assertEquals(IP_RANGE, SUBNETWORK_INFO.ipRange());
6767
}
6868

6969
@Test
7070
public void testOf() {
71-
SubnetworkInfo subnetworkInfo = SubnetworkInfo.of(SUBNETWORK_ID, NETWORK_ID, IP_CIDR_RANGE);
71+
SubnetworkInfo subnetworkInfo = SubnetworkInfo.of(SUBNETWORK_ID, NETWORK_ID, IP_RANGE);
7272
assertNull(subnetworkInfo.id());
7373
assertEquals(SUBNETWORK_ID, subnetworkInfo.subnetworkId());
7474
assertNull(subnetworkInfo.creationTimestamp());
7575
assertNull(subnetworkInfo.description());
7676
assertNull(subnetworkInfo.gatewayAddress());
7777
assertEquals(NETWORK_ID, subnetworkInfo.network());
78-
assertEquals(IP_CIDR_RANGE, subnetworkInfo.ipRange());
78+
assertEquals(IP_RANGE, subnetworkInfo.ipRange());
7979
}
8080

8181
@Test
8282
public void testToAndFromPb() {
8383
compareSubnetworkInfo(SUBNETWORK_INFO, SubnetworkInfo.fromPb(SUBNETWORK_INFO.toPb()));
84-
SubnetworkInfo subnetworkInfo = SubnetworkInfo.of(SUBNETWORK_ID, NETWORK_ID, IP_CIDR_RANGE);
84+
SubnetworkInfo subnetworkInfo = SubnetworkInfo.of(SUBNETWORK_ID, NETWORK_ID, IP_RANGE);
8585
compareSubnetworkInfo(subnetworkInfo, SubnetworkInfo.fromPb(subnetworkInfo.toPb()));
8686
}
8787

0 commit comments

Comments
 (0)