Skip to content

Commit 588e2e6

Browse files
committed
Merge remote-tracking branch 'origin/checkout-sdk' into checkout-sdk
# Conflicts: # commercetools/commercetools-sdk-java-checkout/src/integrationTest/java/com/commercetools/checkout/defaultconfig/CheckoutIntegrationTests.java
2 parents 5929e6a + 1ed859a commit 588e2e6

File tree

3 files changed

+39
-40
lines changed

3 files changed

+39
-40
lines changed

.git-blame-ignore-revs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,3 +7,4 @@ d0129c1095216d5c830900c8a6223ef5d4274de1
77
bbe9f971763ca1b27687a6a51067a385a0d23b04
88
de95c481329aa8b821e6e71ac35c1b8bc67e3e86
99
78c44064f4ec15091bde7a2dc590aa2b3a99341d
10+
03665b75a1e1c3a3cf28df1dec52e91b308e4368
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,15 @@
11

22
package com.commercetools.checkout.defaultconfig;
33

4+
import com.commercetools.api.client.ProjectApiRoot;
45
import com.commercetools.api.defaultconfig.ApiRootBuilder;
5-
import com.commercetools.api.models.cart.Cart;
66
import com.commercetools.api.models.cart.CartDraft;
7-
import com.commercetools.api.client.ProjectApiRoot;
87
import com.commercetools.checkout.models.transaction.TransactionDraft;
9-
import io.vrap.rmf.base.client.ApiHttpException;
8+
109
import io.vrap.rmf.base.client.oauth2.ClientCredentials;
10+
1111
import org.junit.jupiter.api.Assertions;
1212
import org.junit.jupiter.api.Test;
13-
import org.mockito.Mockito;
1413

1514
public class CheckoutIntegrationTests {
1615

@@ -22,50 +21,49 @@ public class CheckoutIntegrationTests {
2221
checkoutApiRoot = createCheckoutClient();
2322
}
2423
public static ProjectApiRoot createApiClient() {
25-
return ApiRootBuilder.ofEnvironmentVariables()
26-
.buildProjectRoot();
24+
return ApiRootBuilder.ofEnvironmentVariables().buildProjectRoot();
2725
}
2826

2927
public static com.commercetools.checkout.client.ProjectApiRoot createCheckoutClient() {
3028
return CheckoutApiRootBuilder.of()
3129
.defaultClient(ClientCredentials.of()
32-
.withClientId(System.getenv("CTP_CLIENT_ID"))
33-
.withClientSecret(System.getenv("CTP_CLIENT_SECRET"))
34-
.build(),
35-
ServiceRegion.GCP_EUROPE_WEST1)
30+
.withClientId(System.getenv("CTP_CLIENT_ID"))
31+
.withClientSecret(System.getenv("CTP_CLIENT_SECRET"))
32+
.build(),
33+
ServiceRegion.GCP_EUROPE_WEST1)
3634
.build(System.getenv("CTP_PROJECT_KEY"));
3735
}
3836

3937
// create client, then cart, then transaction
40-
@Test
41-
public void createAndGetTransactionTest() {
42-
var newCart = CartDraft.builder().currency("EUR").build();
38+
@Test
39+
public void createAndGetTransactionTest() {
40+
var newCart = CartDraft.builder().currency("EUR").build();
4341

44-
var cart = projectApiRoot
45-
.carts()
46-
.post(newCart)
47-
.executeBlocking()
48-
.getBody();
49-
Assertions.assertNotNull(cart);
42+
var cart = projectApiRoot.carts().post(newCart).executeBlocking().getBody();
43+
Assertions.assertNotNull(cart);
5044

51-
var transactionKey = "transaction-" + CheckoutApiTestUtils.randomKey();
52-
var transaction = checkoutApiRoot.with().transactions().post(
53-
TransactionDraft.builder()
54-
.key(transactionKey)
55-
.application(a -> a.key("demo-commercetools-checkout"))
56-
.cart(c -> c.id(cart.getId()))
57-
.plusTransactionItems(t -> t
58-
.amount(a -> a.centAmount(100).currencyCode("EUR"))
59-
.paymentIntegration(p -> p.key("ci-payment-integration")))
60-
.build()
45+
var transactionKey = "transaction-" + CheckoutApiTestUtils.randomKey();
46+
var transaction = checkoutApiRoot.with()
47+
.transactions()
48+
.post(TransactionDraft.builder()
49+
.key(transactionKey)
50+
.application(a -> a.key("demo-commercetools-checkout"))
51+
.cart(c -> c.id(cart.getId()))
52+
.plusTransactionItems(t -> t.amount(a -> a.centAmount(100).currencyCode("EUR"))
53+
.paymentIntegration(p -> p.key("sample-invoice-payment")))
54+
.build()
6155

62-
).executeBlocking().getBody();
56+
)
57+
.executeBlocking()
58+
.getBody();
6359

64-
// Create transaction
65-
Assertions.assertNotNull(transaction);
60+
// Create transaction
61+
Assertions.assertNotNull(transaction);
6662

67-
Assertions.assertNotNull(checkoutApiRoot.with().transactions().withId(transaction.getId()).get().executeBlocking().getBody());
68-
Assertions.assertEquals(transactionKey, transaction.getKey());
69-
Assertions.assertNotNull(checkoutApiRoot.with().transactions().withKey(transaction.getKey()).get().executeBlocking().getBody());
70-
}
63+
Assertions.assertNotNull(
64+
checkoutApiRoot.with().transactions().withId(transaction.getId()).get().executeBlocking().getBody());
65+
Assertions.assertEquals(transactionKey, transaction.getKey());
66+
Assertions.assertNotNull(
67+
checkoutApiRoot.with().transactions().withKey(transaction.getKey()).get().executeBlocking().getBody());
68+
}
7169
}

commercetools/commercetools-sdk-java-checkout/src/main/java/com/commercetools/checkout/defaultconfig/ServiceRegion.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@ public enum ServiceRegion implements ServiceRegionConfig {
99
"https://auth.europe-west1.gcp.commercetools.com")),
1010
GCP_US_CENTRAL1(new RegionHosts("https://checkout.us-central1.gcp.commercetools.com/",
1111
"https://auth.us-central1.gcp.commercetools.com")),
12-
// AWS_US_EAST_2(new RegionHosts("https://checkout.us-east-2.aws.commercetools.com/",
13-
// "https://auth.us-east-2.aws.commercetools.com")),
14-
// AWS_EU_CENTRAL_1(new RegionHosts("https://checkout.eu-central-1.aws.commercetools.com/",
15-
// "https://auth.eu-central-1.aws.commercetools.com")),
12+
// AWS_US_EAST_2(new RegionHosts("https://checkout.us-east-2.aws.commercetools.com/",
13+
// "https://auth.us-east-2.aws.commercetools.com")),
14+
// AWS_EU_CENTRAL_1(new RegionHosts("https://checkout.eu-central-1.aws.commercetools.com/",
15+
// "https://auth.eu-central-1.aws.commercetools.com")),
1616
GCP_AUSTRALIA_SOUTHEAST1(new RegionHosts("https://checkout.australia-southeast1.gcp.commercetools.com/",
1717
"https://auth.australia-southeast1.gcp.commercetools.com")),;
1818

0 commit comments

Comments
 (0)