From 34d11834cbe76992246f3a4c5df5807af2e7d0d2 Mon Sep 17 00:00:00 2001 From: Piotr Marcinkiewicz Date: Wed, 24 Jun 2020 14:06:48 +0200 Subject: Fix checkstyle warnings Issue-ID: AAF-1152 Signed-off-by: Piotr Marcinkiewicz Change-Id: I77777db7080000c22ef3db9ff8e755dac8258c9e --- .../org/onap/aaf/certservice/client/api/ExitableException.java | 4 ++-- .../org/onap/aaf/certservice/client/certification/CsrFactory.java | 8 ++++---- .../client/certification/exception/CertFileWriterException.java | 4 ++-- .../client/certification/exception/CsrGenerationException.java | 4 ++-- .../client/certification/exception/PemConversionException.java | 4 ++-- .../client/certification/exception/PkEncodingException.java | 4 ++-- .../client/configuration/factory/SslContextFactory.java | 4 ++-- .../client/httpclient/exception/HttpClientException.java | 4 ++-- .../client/certification/writer/CertFileWriterTest.java | 2 +- .../onap/aaf/certservice/client/configuration/EnvsForCsrTest.java | 8 ++++---- .../client/configuration/factory/CsrConfigurationFactoryTest.java | 4 ++-- .../client/configuration/factory/SslContextFactoryTest.java | 2 +- 12 files changed, 26 insertions(+), 26 deletions(-) diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitableException.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitableException.java index 439c0488..3a7a542b 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitableException.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitableException.java @@ -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) { diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/CsrFactory.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/CsrFactory.java index e381a8ce..0faf5cab 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/CsrFactory.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/CsrFactory.java @@ -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"); diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertFileWriterException.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertFileWriterException.java index e723ca11..7912f6d7 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertFileWriterException.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertFileWriterException.java @@ -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 diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CsrGenerationException.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CsrGenerationException.java index 8c93a227..ba501cd6 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CsrGenerationException.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CsrGenerationException.java @@ -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() { diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PemConversionException.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PemConversionException.java index 4d1cdaa6..cad3388b 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PemConversionException.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PemConversionException.java @@ -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) { diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PkEncodingException.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PkEncodingException.java index 20d623aa..89976bf3 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PkEncodingException.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/PkEncodingException.java @@ -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() { diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactory.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactory.java index ef74d830..7da8fa58 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactory.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactory.java @@ -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); diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/httpclient/exception/HttpClientException.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/httpclient/exception/HttpClientException.java index bc48554a..e9f0f0ad 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/httpclient/exception/HttpClientException.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/httpclient/exception/HttpClientException.java @@ -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 diff --git a/certServiceClient/src/test/java/org/onap/aaf/certservice/client/certification/writer/CertFileWriterTest.java b/certServiceClient/src/test/java/org/onap/aaf/certservice/client/certification/writer/CertFileWriterTest.java index e7ebb32b..61c4d835 100644 --- a/certServiceClient/src/test/java/org/onap/aaf/certservice/client/certification/writer/CertFileWriterTest.java +++ b/certServiceClient/src/test/java/org/onap/aaf/certservice/client/certification/writer/CertFileWriterTest.java @@ -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/"); diff --git a/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/EnvsForCsrTest.java b/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/EnvsForCsrTest.java index 8e0f5bbf..656bbe59 100644 --- a/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/EnvsForCsrTest.java +++ b/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/EnvsForCsrTest.java @@ -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 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 testEnv = envsForCsr.getSubjectAlternativesName(); diff --git a/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/CsrConfigurationFactoryTest.java b/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/CsrConfigurationFactoryTest.java index 5abd0b13..b9b9b078 100644 --- a/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/CsrConfigurationFactoryTest.java +++ b/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/CsrConfigurationFactoryTest.java @@ -53,8 +53,8 @@ public class CsrConfigurationFactoryTest { private CsrConfigurationFactory testedFactory; private Condition 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; } }; diff --git a/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactoryTest.java b/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactoryTest.java index e71e9895..8e6e8369 100644 --- a/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactoryTest.java +++ b/certServiceClient/src/test/java/org/onap/aaf/certservice/client/configuration/factory/SslContextFactoryTest.java @@ -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); -- cgit 1.2.3-korg