Fix checkstyle warnings 1.1.0
authorPiotr Marcinkiewicz <piotr.marcinkiewicz@nokia.com>
Wed, 24 Jun 2020 12:06:48 +0000 (14:06 +0200)
committerPiotr Marcinkiewicz <piotr.marcinkiewicz@nokia.com>
Thu, 25 Jun 2020 07:00:36 +0000 (09:00 +0200)
Issue-ID: AAF-1152
Signed-off-by: Piotr Marcinkiewicz <piotr.marcinkiewicz@nokia.com>
Change-Id: I77777db7080000c22ef3db9ff8e755dac8258c9e

12 files changed:
certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitableException.java
certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/CsrFactory.java
certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertFileWriterException.java
certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CsrGenerationException.java
certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PemConversionException.java
certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PkEncodingException.java
certServiceClient/src/main/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactory.java
certServiceClient/src/main/java/org/onap/aaf/certservice/client/httpclient/exception/HttpClientException.java
certServiceClient/src/test/java/org/onap/aaf/certservice/client/certification/writer/CertFileWriterTest.java
certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/EnvsForCsrTest.java
certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/CsrConfigurationFactoryTest.java
certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactoryTest.java

index 439c048..3a7a542 100644 (file)
@@ -20,8 +20,8 @@
 package org.onap.aaf.certservice.client.api;
 
 public abstract class ExitableException extends Exception {
-    public ExitableException(Throwable e) {
-        super(e);
+    public ExitableException(Throwable cause) {
+        super(cause);
     }
 
     public ExitableException(String message) {
index e381a8c..0faf5ca 100644 (file)
@@ -67,10 +67,10 @@ public class CsrFactory {
         LOGGER.info("Creation of CSR has been started with following parameters: {}", configuration.toString());
         String csrParameters = getMandatoryParameters().append(getOptionalParameters()).toString();
         X500Principal subject = new X500Principal(csrParameters);
-        PKCS10CertificationRequest request = createPKCS10Csr(subject, keyPair);
+        PKCS10CertificationRequest request = createPkcs10Csr(subject, keyPair);
 
         LOGGER.info("Creation of CSR has been completed successfully");
-        return convertPKCS10CsrToPem(request);
+        return convertPkcs10CsrToPem(request);
     }
 
     private StringBuilder getMandatoryParameters() {
@@ -92,7 +92,7 @@ public class CsrFactory {
         return optionalParameters.toString();
     }
 
-    private PKCS10CertificationRequest createPKCS10Csr(X500Principal subject, KeyPair keyPair) throws CsrGenerationException {
+    private PKCS10CertificationRequest createPkcs10Csr(X500Principal subject, KeyPair keyPair) throws CsrGenerationException {
         JcaPKCS10CertificationRequestBuilder builder = new JcaPKCS10CertificationRequestBuilder(subject, keyPair.getPublic());
 
         if (isParameterPresent(configuration.getSans())) {
@@ -114,7 +114,7 @@ public class CsrFactory {
         return contentSigner;
     }
 
-    private String convertPKCS10CsrToPem(PKCS10CertificationRequest request) throws CsrGenerationException {
+    private String convertPkcs10CsrToPem(PKCS10CertificationRequest request) throws CsrGenerationException {
         final StringWriter stringWriter = new StringWriter();
         try (JcaPEMWriter pemWriter = new JcaPEMWriter(stringWriter)) {
             LOGGER.info("Conversion of CSR to PEM has been started");
index e723ca1..7912f6d 100644 (file)
@@ -24,8 +24,8 @@ import org.onap.aaf.certservice.client.api.ExitableException;
 
 public class CertFileWriterException extends ExitableException {
 
-    public CertFileWriterException(Throwable e) {
-        super(e);
+    public CertFileWriterException(Throwable cause) {
+        super(cause);
     }
 
     @Override
index 8c93a22..ba501cd 100644 (file)
@@ -25,8 +25,8 @@ import org.onap.aaf.certservice.client.api.ExitableException;
 public class CsrGenerationException extends ExitableException {
     private static final ExitStatus EXIT_STATUS = ExitStatus.CSR_GENERATION_EXCEPTION;
 
-    public CsrGenerationException(Throwable e) {
-        super(e);
+    public CsrGenerationException(Throwable cause) {
+        super(cause);
     }
 
     public ExitStatus applicationExitStatus() {
index 4d1cdaa..cad3388 100644 (file)
@@ -25,8 +25,8 @@ import org.onap.aaf.certservice.client.api.ExitableException;
 public class PemConversionException extends ExitableException {
     private static final ExitStatus EXIT_STATUS = ExitStatus.PEM_CONVERSION_EXCEPTION;
 
-    public PemConversionException(Throwable e) {
-        super(e);
+    public PemConversionException(Throwable cause) {
+        super(cause);
     }
 
     public PemConversionException(String message) {
index 20d623a..89976bf 100644 (file)
@@ -25,8 +25,8 @@ import org.onap.aaf.certservice.client.api.ExitableException;
 public class PkEncodingException extends ExitableException {
     private static final ExitStatus EXIT_STATUS = ExitStatus.PK_TO_PEM_ENCODING_EXCEPTION;
 
-    public PkEncodingException(Throwable e) {
-        super(e);
+    public PkEncodingException(Throwable cause) {
+        super(cause);
     }
 
     public ExitStatus applicationExitStatus() {
index ef74d83..7da8fa5 100644 (file)
@@ -53,7 +53,7 @@ public class SslContextFactory {
         String truststorePassword = envsForTls.getTruststorePassword()
                 .orElseThrow(() -> new TlsConfigurationException(createEnvMissingMessage(TlsConfigurationEnvs.TRUSTSTORE_PASSWORD)));
 
-        return createSSLContext(keystorePath, keystorePassword, truststorePath, truststorePassword);
+        return createSslContext(keystorePath, keystorePassword, truststorePath, truststorePassword);
     }
 
     private String createEnvMissingMessage(TlsConfigurationEnvs keystorePath) {
@@ -69,7 +69,7 @@ public class SslContextFactory {
         return keyStore;
     }
 
-    private SSLContext createSSLContext(String keystorePath, String keystorePassword, String truststorePath, String truststorePassword) throws TlsConfigurationException {
+    private SSLContext createSslContext(String keystorePath, String keystorePassword, String truststorePath, String truststorePassword) throws TlsConfigurationException {
         try {
             KeyStore identityKeystore = setupKeystore(keystorePath, keystorePassword);
             KeyStore trustKeystore = setupKeystore(truststorePath, truststorePassword);
index bc48554..e9f0f0a 100644 (file)
@@ -26,8 +26,8 @@ import org.onap.aaf.certservice.client.api.ExitableException;
 public class HttpClientException extends ExitableException {
     private static final ExitStatus EXIT_STATUS = ExitStatus.HTTP_CLIENT_EXCEPTION;
 
-    public HttpClientException(Throwable e) {
-        super(e);
+    public HttpClientException(Throwable cause) {
+        super(cause);
     }
 
     @Override
index e7ebb32..61c4d83 100644 (file)
@@ -70,7 +70,7 @@ class CertFileWriterTest {
     }
 
     @Test
-    void certFileWriterShouldThrowPemToPKCS12ConverterExceptionWhenOutputDirectoryDoesNotExist() {
+    void certFileWriterShouldThrowCertFileWriterExceptionWhenOutputDirectoryDoesNotExist() {
         // given
         final byte[] data = new byte[]{-128, 1, 2, 3, 0};
         CertFileWriter certFileWriter = new CertFileWriter(OUTPUT_PATH + "thisPathDoesNotExist/");
index 8e0f5bb..656bbe5 100644 (file)
@@ -83,7 +83,7 @@ class EnvsForCsrTest {
     }
 
     @Test
-    void shouldReturnSystemEnvOUVariableWhenItWasDefined() {
+    void shouldReturnSystemEnvOuVariableWhenItWasDefined() {
         // given
         when(envsForCsr.readEnv(CsrConfigurationEnvs.ORGANIZATION_UNIT)).thenReturn(Optional.of(TEST_ENV));
 
@@ -96,7 +96,7 @@ class EnvsForCsrTest {
     }
 
     @Test
-    public void shouldReportThatSystemEnvOUVariableIsNotPresentWhenItWasNotDefined() {
+    public void shouldReportThatSystemEnvOuVariableIsNotPresentWhenItWasNotDefined() {
         // when
         final Optional<String> testEnv = envsForCsr.getOrganizationUnit();
 
@@ -171,7 +171,7 @@ class EnvsForCsrTest {
     }
 
     @Test
-    void shouldReturnSystemEnvSANSVariableWhenItWasDefined() {
+    void shouldReturnSystemEnvSansVariableWhenItWasDefined() {
         // given
         when(envsForCsr.readEnv(CsrConfigurationEnvs.SANS)).thenReturn(Optional.of(TEST_ENV));
 
@@ -184,7 +184,7 @@ class EnvsForCsrTest {
     }
 
     @Test
-    public void shouldReportThatSystemEnvSANSVariableIsNotPresentWhenItWasNotDefined() {
+    public void shouldReportThatSystemEnvSansVariableIsNotPresentWhenItWasNotDefined() {
         // when
         final Optional<String> testEnv = envsForCsr.getSubjectAlternativesName();
 
index 5abd0b1..b9b9b07 100644 (file)
@@ -53,8 +53,8 @@ public class CsrConfigurationFactoryTest {
     private CsrConfigurationFactory testedFactory;
     private Condition<CsrConfigurationException> expectedExitCodeCondition = new Condition<>("Correct exit code") {
         @Override
-        public boolean matches(CsrConfigurationException e) {
-            return e.applicationExitStatus() == CSR_CONFIGURATION_EXCEPTION;
+        public boolean matches(CsrConfigurationException exception) {
+            return exception.applicationExitStatus() == CSR_CONFIGURATION_EXCEPTION;
         }
     };
 
index e71e989..8e6e836 100644 (file)
@@ -173,7 +173,7 @@ public class SslContextFactoryTest {
     }
 
     @Test
-    public void shouldReturnSSLContext() throws TlsConfigurationException {
+    public void shouldReturnSslContext() throws TlsConfigurationException {
         // Given
         String keystorePath = getResourcePath(VALID_KEYSTORE_NAME);
         String truststorePath = getResourcePath(VALID_TRUSTSTORE_NAME);