diff options
author | efiacor <fiachra.corcoran@est.tech> | 2023-04-18 10:50:00 +0100 |
---|---|---|
committer | efiacor <fiachra.corcoran@est.tech> | 2023-04-18 14:18:26 +0100 |
commit | 288ee60da14e8d50ce9628c5dc6225f0475295f4 (patch) | |
tree | 1fe4a44fe960338becdc1edde7d34592c211102b /sdc-distribution-client/src/test | |
parent | bcbf218b70363f9ee3569d2dfe9b32d6a90fa70a (diff) |
[SDC-DISTRO-CLIENT] Add truststore config
Signed-off-by: efiacor <fiachra.corcoran@est.tech>
Change-Id: Ie22aa2d78ebb2e0265a6d307ffd12a1656ca60d7
Issue-ID: SDC-4476
Diffstat (limited to 'sdc-distribution-client/src/test')
3 files changed, 16 insertions, 34 deletions
diff --git a/sdc-distribution-client/src/test/java/org/onap/sdc/http/HttpAsdcClientTest.java b/sdc-distribution-client/src/test/java/org/onap/sdc/http/HttpAsdcClientTest.java index 6193e4c..57a6623 100644 --- a/sdc-distribution-client/src/test/java/org/onap/sdc/http/HttpAsdcClientTest.java +++ b/sdc-distribution-client/src/test/java/org/onap/sdc/http/HttpAsdcClientTest.java @@ -65,7 +65,7 @@ class HttpSdcClientTest { void shouldCreateInitializedHttpClient() { // given TestConfiguration configuration = new TestConfiguration(); - configuration.setUseHttpsWithSDC(false); + configuration.setUseHttpsWithSDC(true); // when final HttpRequestFactory httpRequestFactory = new HttpRequestFactory( @@ -78,7 +78,7 @@ class HttpSdcClientTest { // then assertNotNull(httpSdcClient); - assertEquals(HttpClientFactory.HTTP, httpSdcClient.getHttpSchema()); + assertEquals(HttpClientFactory.HTTPS, httpSdcClient.getHttpSchema()); } @Test diff --git a/sdc-distribution-client/src/test/java/org/onap/sdc/impl/DistributionClientTest.java b/sdc-distribution-client/src/test/java/org/onap/sdc/impl/DistributionClientTest.java index 7354c6c..2037345 100644 --- a/sdc-distribution-client/src/test/java/org/onap/sdc/impl/DistributionClientTest.java +++ b/sdc-distribution-client/src/test/java/org/onap/sdc/impl/DistributionClientTest.java @@ -29,24 +29,16 @@ import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; import fj.data.Either; -import java.io.IOException; import java.util.ArrayList; -import java.util.Arrays; import java.util.List; import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.Test; -import org.junit.jupiter.api.Assertions; -import org.junitpioneer.jupiter.SetEnvironmentVariable; import org.mockito.Mockito; -import org.onap.sdc.api.IDistributionClient; import org.onap.sdc.api.consumer.IConfiguration; -import org.onap.sdc.api.notification.IArtifactInfo; import org.onap.sdc.api.notification.IVfModuleMetadata; import org.onap.sdc.api.results.IDistributionClientResult; -import org.onap.sdc.http.HttpSdcClient; import org.onap.sdc.http.SdcConnectorClient; import org.onap.sdc.utils.ArtifactTypeEnum; -import org.onap.sdc.utils.ArtifactsUtils; import org.onap.sdc.utils.DistributionActionResultEnum; import org.onap.sdc.utils.Pair; import org.onap.sdc.utils.TestConfiguration; @@ -159,11 +151,11 @@ class DistributionClientTest { // no consumerId TestConfiguration testConsumerId = new TestConfiguration(); - testConsumerId.setComsumerID(null); + testConsumerId.setConsumerID(null); validationResult = client.init(testConsumerId, new TestNotificationCallback()); assertEquals(DistributionActionResultEnum.CONF_MISSING_CONSUMER_ID, validationResult.getDistributionActionResult()); - testConsumerId.setComsumerID(""); + testConsumerId.setConsumerID(""); validationResult = client.init(testConsumerId, new TestNotificationCallback()); assertEquals(DistributionActionResultEnum.CONF_MISSING_CONSUMER_ID, validationResult.getDistributionActionResult()); diff --git a/sdc-distribution-client/src/test/java/org/onap/sdc/utils/TestConfiguration.java b/sdc-distribution-client/src/test/java/org/onap/sdc/utils/TestConfiguration.java index 529124e..a132cd0 100644 --- a/sdc-distribution-client/src/test/java/org/onap/sdc/utils/TestConfiguration.java +++ b/sdc-distribution-client/src/test/java/org/onap/sdc/utils/TestConfiguration.java @@ -34,7 +34,7 @@ public class TestConfiguration implements IConfiguration { private List<String> relevantArtifactTypes; private String consumerGroup; private String environmentName; - private String comsumerID; + private String consumerID; private final String kafkaSecurityProtocolConfig; private final String kafkaSaslMechanism; private final String kafkaSaslJaasConfig; @@ -45,7 +45,6 @@ public class TestConfiguration implements IConfiguration { private boolean activateServerTLSAuth; private boolean isFilterInEmptyResources; private boolean useHttpsWithSDC; - private List<String> msgBusAddress; private String httpProxyHost; private int httpProxyPort; private String httpsProxyHost; @@ -56,7 +55,7 @@ public class TestConfiguration implements IConfiguration { public TestConfiguration() { this.sdcAddress = "localhost:8443"; - this.comsumerID = "mso-123456"; + this.consumerID = "mso-123456"; this.consumerGroup = "mso-group"; this.environmentName = "PROD"; this.password = "password"; @@ -69,13 +68,9 @@ public class TestConfiguration implements IConfiguration { this.user = "mso-user"; this.keyStorePath = "etc/sdc-client.jks"; this.keyStorePassword = "Aa123456"; - this.activateServerTLSAuth = false; + this.activateServerTLSAuth = true; this.isFilterInEmptyResources = false; this.useHttpsWithSDC = true; - msgBusAddress = new ArrayList<>(); - msgBusAddress.add("kafka-bootstrap1:9092"); - msgBusAddress.add("kafka-bootstrap2:9092"); - msgBusAddress.add("kafka-bootstrap3:9092"); this.kafkaSecurityProtocolConfig = "SASL_PLAINTEXT"; this.kafkaSaslMechanism = "PLAIN"; this.kafkaSaslJaasConfig = "org.apache.kafka.common.security.scram.ScramLoginModule required username=admin password=admin-secret;"; @@ -151,11 +146,6 @@ public class TestConfiguration implements IConfiguration { } @Override - public String getConsumerID() { - return comsumerID; - } - - @Override public String getKeyStorePath() { return keyStorePath; } @@ -165,8 +155,8 @@ public class TestConfiguration implements IConfiguration { return keyStorePassword; } - public String getComsumerID() { - return comsumerID; + public String getConsumerID() { + return consumerID; } @Override @@ -194,8 +184,8 @@ public class TestConfiguration implements IConfiguration { return useSystemProxy; } - public void setComsumerID(String comsumerID) { - this.comsumerID = comsumerID; + public void setConsumerID(String consumerID) { + this.consumerID = consumerID; } public void setSdcAddress(String sdcAddress) { @@ -263,7 +253,7 @@ public class TestConfiguration implements IConfiguration { final int prime = 31; int result = 1; result = prime * result + ((sdcAddress == null) ? 0 : sdcAddress.hashCode()); - result = prime * result + ((comsumerID == null) ? 0 : comsumerID.hashCode()); + result = prime * result + ((consumerID == null) ? 0 : consumerID.hashCode()); result = prime * result + ((consumerGroup == null) ? 0 : consumerGroup.hashCode()); result = prime * result + ((environmentName == null) ? 0 : environmentName.hashCode()); result = prime * result + ((password == null) ? 0 : password.hashCode()); @@ -298,10 +288,10 @@ public class TestConfiguration implements IConfiguration { return false; } else if (!sdcAddress.equals(other.sdcAddress)) return false; - if (comsumerID == null) { - if (other.comsumerID != null) + if (consumerID == null) { + if (other.consumerID != null) return false; - } else if (!comsumerID.equals(other.comsumerID)) + } else if (!consumerID.equals(other.consumerID)) return false; if (consumerGroup == null) { if (other.consumerGroup != null) @@ -348,7 +338,7 @@ public class TestConfiguration implements IConfiguration { return "TestConfiguration [sdcAddress=" + sdcAddress + ", user=" + user + ", password=" + password + ", pollingInterval=" + pollingInterval + ", pollingTimeout=" + pollingTimeout + ", relevantArtifactTypes=" + relevantArtifactTypes + ", consumerGroup=" + consumerGroup - + ", environmentName=" + environmentName + ", comsumerID=" + comsumerID + "]"; + + ", environmentName=" + environmentName + ", comsumerID=" + consumerID + "]"; } @Override |