diff options
author | Kanagaraj Manickam k00365106 <kanagaraj.manickam@huawei.com> | 2017-10-23 17:12:43 +0530 |
---|---|---|
committer | Kanagaraj Manickam k00365106 <kanagaraj.manickam@huawei.com> | 2017-11-21 18:07:43 +0530 |
commit | 5afc1ab68f9eb20232d02f6b463758637cdda540 (patch) | |
tree | 3c49b4a50ea136c933335cc10cdb662d83596707 /framework/src | |
parent | 8aa599260569b9f063fb85680c95116b00609c31 (diff) |
Remove aspect based logging
Issue-Id: CLI-66
Change-Id: Ib0d9365af04710aac812e1413972354c132ebb47
Signed-off-by: Kanagaraj Manickam k00365106 <kanagaraj.manickam@huawei.com>
Diffstat (limited to 'framework/src')
-rw-r--r-- | framework/src/main/java/org/onap/cli/fw/log/OnapCommandLogger.java | 53 | ||||
-rw-r--r-- | framework/src/test/java/org/onap/cli/fw/log/OnapCommandLoggerTest.java | 134 |
2 files changed, 0 insertions, 187 deletions
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); - } - -} |