fix sonar issue: pmd:AvoidThrowingNullPointerException
authorm.dalakov <m.dalakov@samsung.com>
Mon, 25 Sep 2017 09:13:17 +0000 (12:13 +0300)
committerm.dalakov <m.dalakov@samsung.com>
Mon, 25 Sep 2017 09:18:31 +0000 (12:18 +0300)
servers/api-integration-tests/src/main/java/com/samsung/ci/basic/utils/Cbor.java
servers/api-integration-tests/src/main/java/org/apache/maven/plugin/surefire/log/api/ConsoleLoggerDecorator.java
servers/api-integration-tests/src/test/java/com/samsung/ci/test/useractions/UserDSMActionTest.java
servers/mq/src/main/java/com/samsung/servermq/utils/Cbor.java
servers/mq/src/test/java/com/samsung/servermq/utils/CborTest.java

index 255bfa9..d704f6c 100644 (file)
@@ -55,7 +55,7 @@ public class Cbor<T> {
     public T parsePayloadFromCbor(byte[] cborPayload, Class<? extends Object> class1) {
         T payload = null;
         if (cborPayload == null) {
-            throw new NullPointerException("cborPayload is null");
+            throw new IllegalArgumentException("cborPayload is null");
         }
         try {
             payload = (T) mapper.readValue(cborPayload, class1);
@@ -69,7 +69,7 @@ public class Cbor<T> {
     public byte[] encodingPayloadToCbor(Object payload) {
         byte[] cborData = null;
         if (payload == null) {
-            throw new NullPointerException("payload must be initialized");
+            throw new IllegalArgumentException("payload must be initialized");
         }
         try {
             cborData = mapper.writeValueAsBytes(payload);
index 0d662ac..81cfdcf 100644 (file)
@@ -1 +1 @@
-package org.apache.maven.plugin.surefire.log.api;\r\r/*\r * Licensed to the Apache Software Foundation (ASF) under one\r * or more contributor license agreements.  See the NOTICE file\r * distributed with this work for additional information\r * regarding copyright ownership.  The ASF licenses this file\r * to you under the Apache License, Version 2.0 (the\r * "License"); you may not use this file except in compliance\r * with the License.  You may obtain a copy of the License at\r *\r *     http://www.apache.org/licenses/LICENSE-2.0\r *\r * Unless required by applicable law or agreed to in writing,\r * software distributed under the License is distributed on an\r * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY\r * KIND, either express or implied.  See the License for the\r * specific language governing permissions and limitations\r * under the License.\r */\r\r/**\r * Decorator around {@link ConsoleLogger}.\r * This class is loaded in the isolated ClassLoader and the child logger in the\r * in-plugin ClassLoader.\r *\r * @author <a href="mailto:tibordigana@apache.org">Tibor Digana (tibor17)</a>\r * @since 2.20\r */\rpublic final class ConsoleLoggerDecorator implements ConsoleLogger {\r\r    private static final String ERROR = "error";\r    private final Object logger;\r\r    public ConsoleLoggerDecorator(Object logger) {\r        if (logger == null) {\r            throw new NullPointerException("logger argument is null in " + ConsoleLoggerDecorator.class);\r        }\r        this.logger = logger;\r    }\r\r    @Override\r    public void debug(String message) {\r        try {\r            logger.getClass().getMethod("debug", String.class).invoke(logger, message);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void info(String message) {\r        try {\r            logger.getClass().getMethod("info", String.class).invoke(logger, message);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void warning(String message) {\r        try {\r            logger.getClass().getMethod("warning", String.class).invoke(logger, message);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void error(String message) {\r        try {\r            logger.getClass().getMethod(ERROR, String.class).invoke(logger, message);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void error(String message, Throwable t) {\r        try {\r            logger.getClass().getMethod(ERROR, String.class, Throwable.class).invoke(logger, message, t);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void error(Throwable t) {\r        try {\r            logger.getClass().getMethod(ERROR, Throwable.class).invoke(logger, t);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r}\r
\ No newline at end of file
+package org.apache.maven.plugin.surefire.log.api;\r\r/*\r * Licensed to the Apache Software Foundation (ASF) under one\r * or more contributor license agreements.  See the NOTICE file\r * distributed with this work for additional information\r * regarding copyright ownership.  The ASF licenses this file\r * to you under the Apache License, Version 2.0 (the\r * "License"); you may not use this file except in compliance\r * with the License.  You may obtain a copy of the License at\r *\r *     http://www.apache.org/licenses/LICENSE-2.0\r *\r * Unless required by applicable law or agreed to in writing,\r * software distributed under the License is distributed on an\r * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY\r * KIND, either express or implied.  See the License for the\r * specific language governing permissions and limitations\r * under the License.\r */\r\r/**\r * Decorator around {@link ConsoleLogger}.\r * This class is loaded in the isolated ClassLoader and the child logger in the\r * in-plugin ClassLoader.\r *\r * @author <a href="mailto:tibordigana@apache.org">Tibor Digana (tibor17)</a>\r * @since 2.20\r */\rpublic final class ConsoleLoggerDecorator implements ConsoleLogger {\r\r    private static final String ERROR = "error";\r    private final Object logger;\r\r    public ConsoleLoggerDecorator(Object logger) {\r        if (logger == null) {\r            throw new IllegalArgumentException("logger argument is null in " + ConsoleLoggerDecorator.class);\r        }\r        this.logger = logger;\r    }\r\r    @Override\r    public void debug(String message) {\r        try {\r            logger.getClass().getMethod("debug", String.class).invoke(logger, message);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void info(String message) {\r        try {\r            logger.getClass().getMethod("info", String.class).invoke(logger, message);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void warning(String message) {\r        try {\r            logger.getClass().getMethod("warning", String.class).invoke(logger, message);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void error(String message) {\r        try {\r            logger.getClass().getMethod(ERROR, String.class).invoke(logger, message);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void error(String message, Throwable t) {\r        try {\r            logger.getClass().getMethod(ERROR, String.class, Throwable.class).invoke(logger, message, t);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r\r    @Override\r    public void error(Throwable t) {\r        try {\r            logger.getClass().getMethod(ERROR, Throwable.class).invoke(logger, t);\r        } catch (Exception e) {\r            throw new IllegalStateException(e.getLocalizedMessage(), e);\r        }\r    }\r}\r
\ No newline at end of file
index 5f2efc2..3fb2a50 100644 (file)
@@ -69,7 +69,7 @@ public class UserDSMActionTest extends BasicUserAction {
      * 3. Logout
      */
     @Test
-    public void AtestUserRegistration() {
+    public void atestUserRegistration() {
         log("userRegistration");
         createNewUser();
         loginNewUser();
@@ -83,7 +83,7 @@ public class UserDSMActionTest extends BasicUserAction {
      * 3. Logout
      */
     @Test
-    public void BtestDeleteUserIfNotAdmin() {
+    public void btestDeleteUserIfNotAdmin() {
         log("deleteUserIfNotAdmin");
         Response resp = loginNewUser();
         deleteNewUserIfUser(resp.getCookie("JSESSIONID"));
@@ -97,7 +97,7 @@ public class UserDSMActionTest extends BasicUserAction {
      * 3. Logout
      */
     @Test
-    public void CtestDeleteUserIfAdmin() {
+    public void ctestDeleteUserIfAdmin() {
         log("deleteUserIfAdmin");
         Response resp = loginAdmin();
         deleteNewUserIfAdmin(resp.getCookie("JSESSIONID"));
index 39a293e..adeea0d 100644 (file)
@@ -50,7 +50,7 @@ public class Cbor<T> {
     public T parsePayloadFromCbor(byte[] cborPayload, Class<? extends Object> class1) {
         T payload = null;
         if (cborPayload == null) {
-            throw new NullPointerException("cborPayload is null");
+            throw new IllegalArgumentException("cborPayload is null");
         }
         try {
             payload = (T) mapper.readValue(cborPayload, class1);
@@ -64,7 +64,7 @@ public class Cbor<T> {
     public byte[] encodingPayloadToCbor(Object payload) {
         byte[] cborData = null;
         if (payload == null) {
-            throw new NullPointerException("payload must be initialized");
+            throw new IllegalArgumentException("payload must be initialized");
         }
         try {
             cborData = mapper.writeValueAsBytes(payload);
index 7f73cb3..63edb8a 100644 (file)
@@ -19,7 +19,7 @@ public class CborTest {
         assertTrue(result.equals(data));
     }
 
-    @Test(expected = NullPointerException.class)
+    @Test(expected = IllegalArgumentException.class)
     public void testparsePayloadFromCborWithNullValue() {
         Cbor<String> cbor = new Cbor<>();
         cbor.parsePayloadFromCbor(null, String.class);
@@ -43,7 +43,7 @@ public class CborTest {
         assertFalse(result.equals(data));
     }
 
-    @Test(expected = NullPointerException.class)
+    @Test(expected = IllegalArgumentException.class)
     public void testEncodingPayloadToCborWithNull() {
         Cbor<String> cbor = new Cbor<>();
         cbor.encodingPayloadToCbor(null);