Skip to content
This repository was archived by the owner on Sep 12, 2025. It is now read-only.

Commit f65be45

Browse files
committed
Remove workaround for /v2/apps/:guid/summary issue (#176)
This reverts commit b9994bb.
1 parent bcd3aa8 commit f65be45

File tree

2 files changed

+14
-22
lines changed

2 files changed

+14
-22
lines changed

src/main/java/com/sap/cloudfoundry/client/facade/adapters/RawCloudApplication.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,9 @@ public abstract class RawCloudApplication extends RawCloudEntity<CloudApplicatio
4444

4545
@Override
4646
public CloudApplication derive() {
47-
Resource<ApplicationEntity> resource = getResource();
48-
ApplicationEntity entity = resource.getEntity();
4947
SummaryApplicationResponse summary = getSummary();
5048
return ImmutableCloudApplication.builder()
51-
.metadata(parseResourceMetadata(resource))
49+
.metadata(parseResourceMetadata(getResource()))
5250
.name(summary.getName())
5351
.memory(summary.getMemory())
5452
.uris(toUrlStrings(summary.getRoutes()))
@@ -60,7 +58,7 @@ public CloudApplication derive() {
6058
.packageState(parsePackageState(summary.getPackageState()))
6159
.stagingError(summary.getStagingFailedDescription())
6260
.services(getNames(summary.getServices()))
63-
.env(parseEnv(entity.getEnvironmentJsons()))
61+
.env(parseEnv(summary.getEnvironmentJsons()))
6462
.space(getSpace().derive())
6563
.build();
6664
}

src/test/java/com/sap/cloudfoundry/client/facade/adapters/RawCloudApplicationTest.java

Lines changed: 12 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,12 @@ private static Map<String, String> buildExpectedEnvironment() {
162162
}
163163

164164
private static RawCloudApplication buildRawApplicationWithoutEnvironment() {
165-
return buildRawApplication(buildApplicationResourceWithoutEnvironment(), buildApplicationSummary());
165+
SummaryApplicationResponse summary = buildApplicationSummary();
166+
SummaryApplicationResponse summaryWithoutEnvironment = SummaryApplicationResponse.builder()
167+
.from(summary)
168+
.environmentJsons(null)
169+
.build();
170+
return buildRawApplication(summaryWithoutEnvironment);
166171
}
167172

168173
private static RawCloudApplication buildRawApplicationWithoutDockerInfo() {
@@ -172,7 +177,7 @@ private static RawCloudApplication buildRawApplicationWithoutDockerInfo() {
172177
.dockerImage(null)
173178
.dockerCredentials(null)
174179
.build();
175-
return buildRawApplication(buildApplicationResource(), summaryWithoutDockerInfo);
180+
return buildRawApplication(summaryWithoutDockerInfo);
176181
}
177182

178183
private static RawCloudApplication buildRawApplicationWithoutDockerCredentials() {
@@ -181,37 +186,25 @@ private static RawCloudApplication buildRawApplicationWithoutDockerCredentials()
181186
.from(summary)
182187
.dockerCredentials(null)
183188
.build();
184-
return buildRawApplication(buildApplicationResource(), summaryWithoutDockerCredentials);
189+
return buildRawApplication(summaryWithoutDockerCredentials);
185190
}
186191

187192
private static RawCloudApplication buildRawApplication() {
188-
return buildRawApplication(buildApplicationResource(), buildApplicationSummary());
193+
return buildRawApplication(buildApplicationSummary());
189194
}
190195

191-
private static RawCloudApplication buildRawApplication(Resource<ApplicationEntity> applicationResource,
192-
SummaryApplicationResponse summary) {
196+
private static RawCloudApplication buildRawApplication(SummaryApplicationResponse summary) {
193197
return ImmutableRawCloudApplication.builder()
194-
.resource(applicationResource)
198+
.resource(buildApplicationResource())
195199
.summary(summary)
196200
.stack(STACK)
197201
.space(SPACE)
198202
.build();
199203
}
200204

201205
private static Resource<ApplicationEntity> buildApplicationResource() {
202-
return buildApplicationResource(ENVIRONMENT);
203-
}
204-
205-
private static Resource<ApplicationEntity> buildApplicationResourceWithoutEnvironment() {
206-
return buildApplicationResource(null);
207-
}
208-
209-
private static Resource<ApplicationEntity> buildApplicationResource(Map<String, Object> environmentJsons) {
210206
return ApplicationResource.builder()
211207
.metadata(RawCloudEntityTest.METADATA)
212-
.entity(ApplicationEntity.builder()
213-
.environmentJsons(environmentJsons)
214-
.build())
215208
.build();
216209
}
217210

@@ -232,6 +225,7 @@ private static SummaryApplicationResponse buildApplicationSummary() {
232225
.enableSsh(SSH_ENABLED)
233226
.dockerImage(DOCKER_IMAGE)
234227
.dockerCredentials(DOCKER_CREDENTIALS)
228+
.environmentJsons(ENVIRONMENT)
235229
.command(COMMAND)
236230
.buildpack(BUILDPACK)
237231
.detectedBuildpack(DETECTED_BUILDPACK)

0 commit comments

Comments
 (0)