Skip to content

Commit 6f43cf0

Browse files
committed
DefaultApplications: rename uses v3 api
1 parent fb12c5d commit 6f43cf0

File tree

2 files changed

+34
-11
lines changed

2 files changed

+34
-11
lines changed

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

Lines changed: 19 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -552,10 +552,12 @@ public Mono<Void> pushManifestV3(PushManifestV3Request request) {
552552

553553
@Override
554554
public Mono<Void> rename(RenameApplicationRequest request) {
555-
return getApplicationId(request.getName())
555+
return getApplicationIdV3(request.getName())
556556
.flatMap(
557557
applicationId ->
558-
requestUpdateApplicationName(applicationId, request.getNewName()))
558+
requestUpdateApplicationV3(
559+
applicationId,
560+
builder -> builder.name(request.getNewName())))
559561
.then()
560562
.transform(OperationsLogging.log("Rename Application"))
561563
.checkpoint();
@@ -2163,6 +2165,21 @@ private Mono<AbstractApplicationResource> requestUpdateApplication(
21632165
.cast(AbstractApplicationResource.class);
21642166
}
21652167

2168+
private Mono<Void> requestUpdateApplicationV3(
2169+
String applicationId,
2170+
UnaryOperator<org.cloudfoundry.client.v3.applications.UpdateApplicationRequest.Builder>
2171+
modifier) {
2172+
return this.cloudFoundryClient
2173+
.applicationsV3()
2174+
.update(
2175+
modifier.apply(
2176+
org.cloudfoundry.client.v3.applications
2177+
.UpdateApplicationRequest.builder()
2178+
.applicationId(applicationId))
2179+
.build())
2180+
.then();
2181+
}
2182+
21662183
private Mono<AbstractApplicationResource> requestUpdateApplicationEnvironment(
21672184
String applicationId, Map<String, Object> environment) {
21682185
return requestUpdateApplication(

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

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3271,7 +3271,7 @@ void pushUploadFails() throws IOException {
32713271

32723272
@Test
32733273
void rename() {
3274-
requestApplications(
3274+
requestApplicationsV3(
32753275
this.cloudFoundryClient, "test-app-name", TEST_SPACE_ID, "test-metadata-id");
32763276
requestUpdateApplicationRename(
32773277
this.cloudFoundryClient, "test-metadata-id", "test-new-app-name");
@@ -3289,7 +3289,7 @@ void rename() {
32893289

32903290
@Test
32913291
void renameNoApp() {
3292-
requestApplicationsEmpty(this.cloudFoundryClient, "test-app-name", TEST_SPACE_ID);
3292+
requestApplicationsEmptyV3(this.cloudFoundryClient, "test-app-name", TEST_SPACE_ID);
32933293

32943294
this.applications
32953295
.rename(
@@ -5977,19 +5977,25 @@ private static void requestUpdateApplicationHealthCheck(
59775977
private static void requestUpdateApplicationRename(
59785978
CloudFoundryClient cloudFoundryClient, String applicationId, String name) {
59795979
when(cloudFoundryClient
5980-
.applicationsV2()
5980+
.applicationsV3()
59815981
.update(
5982-
UpdateApplicationRequest.builder()
5982+
org.cloudfoundry.client.v3.applications.UpdateApplicationRequest
5983+
.builder()
59835984
.applicationId(applicationId)
59845985
.name(name)
59855986
.build()))
59865987
.thenReturn(
59875988
Mono.just(
5988-
fill(UpdateApplicationResponse.builder())
5989-
.entity(
5990-
fill(
5991-
ApplicationEntity.builder(),
5992-
"application-entity-")
5989+
org.cloudfoundry.client.v3.applications.UpdateApplicationResponse
5990+
.builder()
5991+
.id("test-application-id")
5992+
.createdAt("2016-02-08T15:45:59Z")
5993+
.state(ApplicationState.STARTED)
5994+
.name("test-application-name")
5995+
.lifecycle(
5996+
Lifecycle.builder()
5997+
.data(BuildpackData.builder().build())
5998+
.type(BUILDPACK)
59935999
.build())
59946000
.build()));
59956001
}

0 commit comments

Comments
 (0)