From 5afc1ab68f9eb20232d02f6b463758637cdda540 Mon Sep 17 00:00:00 2001 From: Kanagaraj Manickam k00365106 Date: Mon, 23 Oct 2017 17:12:43 +0530 Subject: Remove aspect based logging Issue-Id: CLI-66 Change-Id: Ib0d9365af04710aac812e1413972354c132ebb47 Signed-off-by: Kanagaraj Manickam k00365106 --- framework/pom.xml | 10 -- .../org/onap/cli/fw/log/OnapCommandLogger.java | 53 -------- .../org/onap/cli/fw/log/OnapCommandLoggerTest.java | 134 --------------------- 3 files changed, 197 deletions(-) delete mode 100644 framework/src/main/java/org/onap/cli/fw/log/OnapCommandLogger.java delete mode 100644 framework/src/test/java/org/onap/cli/fw/log/OnapCommandLoggerTest.java (limited to 'framework') diff --git a/framework/pom.xml b/framework/pom.xml index 2b7aece2..ce49164b 100644 --- a/framework/pom.xml +++ b/framework/pom.xml @@ -36,16 +36,6 @@ snakeyaml 1.17 - - org.aspectj - aspectjrt - 1.8.10 - - - org.aspectj - aspectjweaver - 1.8.10 - org.slf4j slf4j-log4j12 diff --git a/framework/src/main/java/org/onap/cli/fw/log/OnapCommandLogger.java b/framework/src/main/java/org/onap/cli/fw/log/OnapCommandLogger.java deleted file mode 100644 index a29d3597..00000000 --- a/framework/src/main/java/org/onap/cli/fw/log/OnapCommandLogger.java +++ /dev/null @@ -1,53 +0,0 @@ -/* - * Copyright 2017 Huawei Technologies Co., Ltd. - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -package org.onap.cli.fw.log; - -import org.aspectj.lang.ProceedingJoinPoint; -import org.aspectj.lang.annotation.Around; -import org.aspectj.lang.annotation.Aspect; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -/** - * Helps to log the command method boundary calls. - * - */ -@Aspect -public class OnapCommandLogger { - private static final Logger LOGGER = LoggerFactory.getLogger(OnapCommandLogger.class); - - //(mrkanag) verify that it logs for all classes in this project. - /** - * Logging intercepter. - * - * @param joinPoint - * joinpoint - * @return object - * @throws Throwable - * exception - */ - @Around("execution(* org.onap.cli.fw*(..))") - public Object log(ProceedingJoinPoint joinPoint) throws Throwable { // NOSONAR - LOGGER.info(joinPoint.getThis().toString() + "->" + joinPoint.getSignature().getName() + "(" - + joinPoint.getArgs() + ")"); - - Object response = joinPoint.proceed(); - LOGGER.info(response.toString()); - - return response; - } -} diff --git a/framework/src/test/java/org/onap/cli/fw/log/OnapCommandLoggerTest.java b/framework/src/test/java/org/onap/cli/fw/log/OnapCommandLoggerTest.java deleted file mode 100644 index 4b675edc..00000000 --- a/framework/src/test/java/org/onap/cli/fw/log/OnapCommandLoggerTest.java +++ /dev/null @@ -1,134 +0,0 @@ -/* - * Copyright 2016-17 Huawei Technologies Co., Ltd. - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -package org.onap.cli.fw.log; - -import org.aspectj.lang.ProceedingJoinPoint; -import org.aspectj.lang.Signature; -import org.aspectj.lang.reflect.SourceLocation; -import org.aspectj.runtime.internal.AroundClosure; -import org.junit.Test; - -public class OnapCommandLoggerTest { - - @Test - public void logTest() throws Throwable { - - - ProceedingJoinPoint point = new ProceedingJoinPoint() { - @Override - public String toShortString() { - return null; - } - - @Override - public String toLongString() { - return null; - } - - @Override - public Object getThis() { - return new Object(); - } - - @Override - public Object getTarget() { - return null; - } - - @Override - public StaticPart getStaticPart() { - return null; - } - - @Override - public SourceLocation getSourceLocation() { - return null; - } - - @Override - public Signature getSignature() { - return new Signature(){ - - @Override - public Class getDeclaringType() { - // TODO Auto-generated method stub - return null; - } - - @Override - public String getDeclaringTypeName() { - // TODO Auto-generated method stub - return null; - } - - @Override - public int getModifiers() { - // TODO Auto-generated method stub - return 0; - } - - @Override - public String getName() { - // TODO Auto-generated method stub - return null; - } - - @Override - public String toLongString() { - // TODO Auto-generated method stub - return null; - } - - @Override - public String toShortString() { - // TODO Auto-generated method stub - return null; - } - - }; - } - - @Override - public String getKind() { - return null; - } - - @Override - public Object[] getArgs() { - return null; - } - - @Override - public void set$AroundClosure(AroundClosure arg0) { - } - - @Override - public Object proceed(Object[] arg0) throws Throwable { - return null; - } - - @Override - public Object proceed() throws Throwable { - return new Object(); - } - }; - - OnapCommandLogger log = new OnapCommandLogger(); - log.log(point); - } - -} -- cgit 1.2.3-korg