Skip to content

Commit a647d37

Browse files
committed
fix test
1 parent 7640c39 commit a647d37

File tree

3 files changed

+51
-20
lines changed

3 files changed

+51
-20
lines changed

multiapps-common-test/src/main/java/org/cloudfoundry/multiapps/common/test/Tester.java

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
package org.cloudfoundry.multiapps.common.test;
22

3-
import static org.junit.jupiter.api.Assertions.assertEquals;
4-
import static org.junit.jupiter.api.Assertions.assertTrue;
5-
import static org.junit.jupiter.api.Assertions.fail;
6-
73
import java.text.MessageFormat;
84
import java.util.List;
95
import java.util.Objects;
@@ -15,6 +11,10 @@
1511
import org.cloudfoundry.multiapps.common.util.JsonUtil;
1612
import org.junit.jupiter.api.Assertions;
1713

14+
import static org.junit.jupiter.api.Assertions.assertEquals;
15+
import static org.junit.jupiter.api.Assertions.assertTrue;
16+
import static org.junit.jupiter.api.Assertions.fail;
17+
1818
public class Tester {
1919

2020
private final Class<?> testedClass;
@@ -94,12 +94,13 @@ private void validateSuccessForSetExpectation(Set<Object> expectedSet, Set<Objec
9494
private void validateFailure(Expectation expectation, Exception e) {
9595
if (!expectation.expectsFailure()) {
9696
e.printStackTrace();
97-
fail("Test failed: " + e.toString());
97+
fail("Test failed: " + e);
9898
}
9999
String exceptionMessage = e.getMessage();
100100
assertTrue(exceptionMessage.contains(expectation.getExpectationAsString()),
101-
MessageFormat.format("Exception's message doesn't match up! Expected [{0}] to contain [{1}]!", exceptionMessage,
102-
expectation.getExpectationAsString()));
101+
MessageFormat.format("Exception's message doesn't match up! Expected [{0}] to contain [{1}]!",
102+
expectation.getExpectationAsString(), exceptionMessage)
103+
);
103104
}
104105

105106
private Object loadResourceAsJsonObject(String resource) {

multiapps-mta/src/test/java/org/cloudfoundry/multiapps/mta/resolvers/v3/DescriptorReferenceResolverTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,7 @@ static Stream<Arguments> testResolve() {
3535
new Expectation(Expectation.Type.JSON, "result-from-escaped-references.json")),
3636
// (5)
3737
Arguments.of("merged-05.yaml",
38-
new Expectation(Expectation.Type.EXCEPTION,
39-
"Unable to resolve \"qux#benefits-app-example1#benefits-app-example1##parameters##destinations##0##url\"")));
38+
new Expectation(Expectation.Type.EXCEPTION, "Unable to resolve \"module-a##module-b#invalid/0/url\"")));
4039
}
4140

4241
@ParameterizedTest
Lines changed: 42 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,50 @@
1-
_schema-version: "3"
2-
ID: com.sap.mta.v3.test.array
1+
ID: com.sap.mta.v2.test.config-02
2+
_schema-version: 3.3.0
33
version: 1.0.0
44

55
modules:
6-
- name: qux
7-
type: qux
6+
- name: module-a
7+
type: com.sap.application.content
8+
parameters:
9+
content:
10+
subaccount:
11+
existing_destinations_policy: update
12+
destinations:
13+
- Name: destination-a
14+
URL: ~{module-b/invalid/0/url}
15+
forwardAuthToken: true
16+
ServiceInstanceName: "service-a"
17+
requires:
18+
- name: service-destination
19+
parameters:
20+
content-target: true
21+
- name: service-a
22+
- name: module-b
23+
24+
- name: module-b
25+
type: application
26+
path: "appBits.zip"
827
requires:
9-
- name: benefits-app-example1
10-
group: configuration
28+
- name: service-a
29+
parameters:
30+
buildpack: staticfile_buildpack
31+
memory: 1G
32+
disk-quota: 1G
33+
provides:
34+
- name: module-b
1135
properties:
12-
array-ref: ~{benefits-app-example1##parameters##destinations##0##url}
36+
default-url: ${default-url}
1337

1438
resources:
15-
- name: benefits-app-example1
16-
type: configuration
39+
- name: service-destination
40+
type: org.cloudfoundry.managed-service
41+
parameters:
42+
service: destination
43+
service-name: service-destination
44+
service-plan: lite
45+
46+
- name: service-a
47+
type: org.cloudfoundry.managed-service
1748
parameters:
18-
destinations:
19-
- name: dest-0
49+
service: xsuaa
50+
service-plan: application

0 commit comments

Comments
 (0)