Skip to content

Commit f13e97b

Browse files
committed
DefaultApplications: use applicationsV3 for routes
1 parent 63ae428 commit f13e97b

File tree

2 files changed

+26
-21
lines changed

2 files changed

+26
-21
lines changed

cloudfoundry-operations/src/main/java/org/cloudfoundry/operations/applications/DefaultApplications.java

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@
5656
import org.cloudfoundry.client.v2.applications.CreateApplicationResponse;
5757
import org.cloudfoundry.client.v2.applications.DockerCredentials;
5858
import org.cloudfoundry.client.v2.applications.InstanceStatistics;
59-
import org.cloudfoundry.client.v2.applications.ListApplicationRoutesRequest;
6059
import org.cloudfoundry.client.v2.applications.ListApplicationServiceBindingsRequest;
6160
import org.cloudfoundry.client.v2.applications.RemoveApplicationRouteRequest;
6261
import org.cloudfoundry.client.v2.applications.RemoveApplicationServiceBindingRequest;
@@ -116,6 +115,7 @@
116115
import org.cloudfoundry.client.v3.applications.GetApplicationSshEnabledRequest;
117116
import org.cloudfoundry.client.v3.applications.GetApplicationSshEnabledResponse;
118117
import org.cloudfoundry.client.v3.applications.ListApplicationProcessesRequest;
118+
import org.cloudfoundry.client.v3.applications.ListApplicationRoutesRequest;
119119
import org.cloudfoundry.client.v3.applications.ListApplicationsRequest;
120120
import org.cloudfoundry.client.v3.applications.SetApplicationCurrentDropletRequest;
121121
import org.cloudfoundry.client.v3.applications.UpdateApplicationFeatureRequest;
@@ -1096,7 +1096,8 @@ private Mono<ApplicationInstancesResponse> getApplicationInstances(String applic
10961096
t -> Mono.just(ApplicationInstancesResponse.builder().build()));
10971097
}
10981098

1099-
private Mono<List<RouteResource>> getApplicationRoutes(String applicationId) {
1099+
private Mono<List<org.cloudfoundry.client.v3.routes.RouteResource>> getApplicationRoutes(
1100+
String applicationId) {
11001101
return requestApplicationRoutes(applicationId).collectList();
11011102
}
11021103

@@ -1490,11 +1491,11 @@ private Mono<Void> prepareDomainsAndRoutes(
14901491
String applicationId,
14911492
List<DomainSummary> availableDomains,
14921493
ApplicationManifest manifest,
1493-
List<RouteResource> existingRoutes,
1494+
List<org.cloudfoundry.client.v3.routes.RouteResource> existingRoutes,
14941495
RandomWords randomWords) {
14951496
if (Optional.ofNullable(manifest.getNoRoute()).orElse(false)) {
14961497
return Flux.fromIterable(existingRoutes)
1497-
.map(ResourceUtils::getId)
1498+
.map(org.cloudfoundry.client.v3.routes.RouteResource::getId)
14981499
.flatMap(routeId -> requestRemoveRouteFromApplication(applicationId, routeId))
14991500
.then();
15001501
}
@@ -1524,7 +1525,9 @@ private Mono<Void> prepareDomainsAndRoutes(
15241525
}
15251526

15261527
List<String> existingRouteIds =
1527-
existingRoutes.stream().map(ResourceUtils::getId).collect(Collectors.toList());
1528+
existingRoutes.stream()
1529+
.map(org.cloudfoundry.client.v3.routes.RouteResource::getId)
1530+
.collect(Collectors.toList());
15281531

15291532
return getPushRouteIdFromRoute(availableDomains, manifest, randomWords)
15301533
.filter(routeId -> !existingRouteIds.contains(routeId))
@@ -1640,11 +1643,12 @@ private Mono<ApplicationInstancesResponse> requestApplicationInstances(String ap
16401643
ApplicationInstancesRequest.builder().applicationId(applicationId).build());
16411644
}
16421645

1643-
private Flux<RouteResource> requestApplicationRoutes(String applicationId) {
1644-
return PaginationUtils.requestClientV2Resources(
1646+
private Flux<org.cloudfoundry.client.v3.routes.RouteResource> requestApplicationRoutes(
1647+
String applicationId) {
1648+
return PaginationUtils.requestClientV3Resources(
16451649
page ->
16461650
this.cloudFoundryClient
1647-
.applicationsV2()
1651+
.applicationsV3()
16481652
.listRoutes(
16491653
ListApplicationRoutesRequest.builder()
16501654
.applicationId(applicationId)

cloudfoundry-operations/src/test/java/org/cloudfoundry/operations/applications/DefaultApplicationsTest.java

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,6 @@
5555
import org.cloudfoundry.client.v2.applications.DockerCredentials;
5656
import org.cloudfoundry.client.v2.applications.GetApplicationResponse;
5757
import org.cloudfoundry.client.v2.applications.InstanceStatistics;
58-
import org.cloudfoundry.client.v2.applications.ListApplicationRoutesRequest;
59-
import org.cloudfoundry.client.v2.applications.ListApplicationRoutesResponse;
6058
import org.cloudfoundry.client.v2.applications.ListApplicationServiceBindingsRequest;
6159
import org.cloudfoundry.client.v2.applications.ListApplicationServiceBindingsResponse;
6260
import org.cloudfoundry.client.v2.applications.RemoveApplicationRouteRequest;
@@ -124,11 +122,14 @@
124122
import org.cloudfoundry.client.v3.BuildpackData;
125123
import org.cloudfoundry.client.v3.DockerData;
126124
import org.cloudfoundry.client.v3.Lifecycle;
125+
import org.cloudfoundry.client.v3.Pagination;
127126
import org.cloudfoundry.client.v3.applications.ApplicationState;
128127
import org.cloudfoundry.client.v3.applications.GetApplicationEnvironmentRequest;
129128
import org.cloudfoundry.client.v3.applications.GetApplicationEnvironmentResponse;
130129
import org.cloudfoundry.client.v3.applications.GetApplicationSshEnabledRequest;
131130
import org.cloudfoundry.client.v3.applications.GetApplicationSshEnabledResponse;
131+
import org.cloudfoundry.client.v3.applications.ListApplicationRoutesRequest;
132+
import org.cloudfoundry.client.v3.applications.ListApplicationRoutesResponse;
132133
import org.cloudfoundry.client.v3.applications.ListApplicationsRequest;
133134
import org.cloudfoundry.client.v3.applications.ListApplicationsResponse;
134135
import org.cloudfoundry.client.v3.applications.UpdateApplicationFeatureRequest;
@@ -4241,30 +4242,30 @@ private static void requestApplicationNullUsage(
42414242
private static void requestApplicationRoutes(
42424243
CloudFoundryClient cloudFoundryClient, String applicationId, String routeId) {
42434244
when(cloudFoundryClient
4244-
.applicationsV2()
4245+
.applicationsV3()
42454246
.listRoutes(
4246-
ListApplicationRoutesRequest.builder()
4247+
org.cloudfoundry.client.v3.applications.ListApplicationRoutesRequest
4248+
.builder()
42474249
.applicationId(applicationId)
42484250
.page(1)
42494251
.build()))
42504252
.thenReturn(
42514253
Mono.just(
4252-
fill(ListApplicationRoutesResponse.builder())
4254+
fill(org.cloudfoundry.client.v3.applications
4255+
.ListApplicationRoutesResponse.builder())
42534256
.resource(
4254-
fill(RouteResource.builder())
4255-
.metadata(
4256-
fill(Metadata.builder())
4257-
.id(routeId)
4258-
.build())
4257+
fill(org.cloudfoundry.client.v3.routes.RouteResource
4258+
.builder())
4259+
.id(routeId)
42594260
.build())
4260-
.totalPages(1)
4261+
.pagination(Pagination.builder().totalPages(1).build())
42614262
.build()));
42624263
}
42634264

42644265
private static void requestApplicationRoutesEmpty(
42654266
CloudFoundryClient cloudFoundryClient, String applicationId) {
42664267
when(cloudFoundryClient
4267-
.applicationsV2()
4268+
.applicationsV3()
42684269
.listRoutes(
42694270
ListApplicationRoutesRequest.builder()
42704271
.applicationId(applicationId)
@@ -4273,7 +4274,7 @@ private static void requestApplicationRoutesEmpty(
42734274
.thenReturn(
42744275
Mono.just(
42754276
fill(ListApplicationRoutesResponse.builder())
4276-
.totalPages(1)
4277+
.pagination(Pagination.builder().totalPages(1).build())
42774278
.build()));
42784279
}
42794280

0 commit comments

Comments
 (0)