aboutsummaryrefslogtreecommitdiffstats
path: root/framework
diff options
context:
space:
mode:
authorKanagaraj Manickam k00365106 <kanagaraj.manickam@huawei.com>2017-10-23 17:12:43 +0530
committerKanagaraj Manickam k00365106 <kanagaraj.manickam@huawei.com>2017-11-21 18:07:43 +0530
commit5afc1ab68f9eb20232d02f6b463758637cdda540 (patch)
tree3c49b4a50ea136c933335cc10cdb662d83596707 /framework
parent8aa599260569b9f063fb85680c95116b00609c31 (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')
-rw-r--r--framework/pom.xml10
-rw-r--r--framework/src/main/java/org/onap/cli/fw/log/OnapCommandLogger.java53
-rw-r--r--framework/src/test/java/org/onap/cli/fw/log/OnapCommandLoggerTest.java134
3 files changed, 0 insertions, 197 deletions
diff --git a/framework/pom.xml b/framework/pom.xml
index 2b7aece2..ce49164b 100644
--- a/framework/pom.xml
+++ b/framework/pom.xml
@@ -37,16 +37,6 @@
<version>1.17</version>
</dependency>
<dependency>
- <groupId>org.aspectj</groupId>
- <artifactId>aspectjrt</artifactId>
- <version>1.8.10</version>
- </dependency>
- <dependency>
- <groupId>org.aspectj</groupId>
- <artifactId>aspectjweaver</artifactId>
- <version>1.8.10</version>
- </dependency>
- <dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-log4j12</artifactId>
<version>1.6.1</version>
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);
- }
-
-}