From d4e3a1b394715c6386f963130e4e081d421ecd1b Mon Sep 17 00:00:00 2001 From: liamfallon Date: Fri, 15 Nov 2019 13:35:42 +0000 Subject: Unit/SONAR/Checkstyle in ONAP-REST Fifth and final batch of JPA pojos (Q-Z), with JUnit added and SONAR/Checkstyle issues addressed. In cases where a class name change caused an update in another package, the license header on files for those knock on changes are not updated. Issue-ID: POLICY-2131 Change-Id: I1cc536a5b8a0cd3c6beff068dd72381f7ae10d12 Signed-off-by: liamfallon --- .../org/onap/policy/conf/HibernateSession.java | 4 ++-- .../CreateClosedLoopFaultController.java | 2 +- .../controller/CreateFirewallController.java | 22 +++++++++++----------- .../java/org/onap/policy/dao/SystemLogDbDao.java | 6 +++--- .../org/onap/policy/daoImp/SystemLogDbDaoImpl.java | 18 +++++++++--------- 5 files changed, 26 insertions(+), 26 deletions(-) (limited to 'POLICY-SDK-APP/src/main/java/org/onap') diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/conf/HibernateSession.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/conf/HibernateSession.java index 82583c5f5..ef6b98803 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/conf/HibernateSession.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/conf/HibernateSession.java @@ -32,7 +32,7 @@ import org.hibernate.cfg.Configuration; import org.onap.policy.common.logging.flexlogger.FlexLogger; import org.onap.policy.common.logging.flexlogger.Logger; import org.onap.policy.controller.PolicyController; -import org.onap.policy.rest.jpa.SystemLogDB; +import org.onap.policy.rest.jpa.SystemLogDb; @SuppressWarnings("deprecation") public class HibernateSession { @@ -51,7 +51,7 @@ public class HibernateSession { prop.setProperty("hibernate.connection.driver_class", PolicyController.getLogdbDriver()); prop.setProperty("show_sql", "false"); logSessionFactory = new Configuration().addPackage("org.onap.policy.*").addProperties(prop) - .addAnnotatedClass(SystemLogDB.class).buildSessionFactory(); + .addAnnotatedClass(SystemLogDb.class).buildSessionFactory(); } catch (Exception ex) { LOGGER.error("Exception Occured while creating Log database Hibernate session" + ex); } diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java index a7a03c021..58765ded1 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java @@ -366,7 +366,7 @@ public class CreateClosedLoopFaultController extends RestrictedBaseController { try { varbindId = (VarbindDictionary) commonclassdao.getEntityItem(VarbindDictionary.class, "varbindName", attrib); - return varbindId.getVarbindOID(); + return varbindId.getVarbindOid(); } catch (Exception e) { policyLogger.error("Error during retrieving varbindName " + attrib, e); return attrib; diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java index 446e124ed..3da703ab2 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java @@ -321,9 +321,9 @@ public class CreateFirewallController extends RestrictedBaseController { List tmList = commonClassDao.getDataById(TermList.class, "termName", id); jpaTermList = (TermList) tmList.get(0); if (jpaTermList != null) { - ruleSrcList = jpaTermList.getSrcIPList(); + ruleSrcList = jpaTermList.getSrcIpList(); if ((ruleSrcList != null) && (!ruleSrcList.isEmpty()) && !"null".equals(ruleSrcList)) { - displayString.append("Source IP List: " + jpaTermList.getSrcIPList()); + displayString.append("Source IP List: " + jpaTermList.getSrcIpList()); displayString.append(" ; \t\n"); for (String srcList : ruleSrcList.split(",")) { if (srcList.startsWith(GROUP)) { @@ -354,9 +354,9 @@ public class CreateFirewallController extends RestrictedBaseController { } displayString.append("\n"); } - ruleDestList = jpaTermList.getDestIPList(); + ruleDestList = jpaTermList.getDestIpList(); if (ruleDestList != null && (!ruleDestList.isEmpty()) && !"null".equals(ruleDestList)) { - displayString.append("Destination IP List: " + jpaTermList.getDestIPList()); + displayString.append("Destination IP List: " + jpaTermList.getDestIpList()); displayString.append(" ; \t\n"); for (String destList : ruleDestList.split(",")) { if (destList.startsWith(GROUP)) { @@ -409,7 +409,7 @@ public class CreateFirewallController extends RestrictedBaseController { displayString.append("\n\t" + "Name: " + sl.getServiceName()); displayString.append("\n\t" + "Description: " + sl.getServiceDescription()); displayString.append( - "\n\t" + "Transport-Protocol: " + sl.getServiceTransProtocol()); + "\n\t" + "Transport-Protocol: " + sl.getServiceTransportProtocol()); displayString.append("\n\t" + "Ports: " + sl.getServicePorts()); displayString.append("\n"); } @@ -421,7 +421,7 @@ public class CreateFirewallController extends RestrictedBaseController { displayString.append("\n\t" + "Name: " + sl.getServiceName()); displayString.append("\n\t" + "Description: " + sl.getServiceDescription()); displayString - .append("\n\t" + "Transport-Protocol: " + sl.getServiceTransProtocol()); + .append("\n\t" + "Transport-Protocol: " + sl.getServiceTransportProtocol()); displayString.append("\n\t" + "Ports: " + sl.getServicePorts()); displayString.append("\n"); } @@ -529,7 +529,7 @@ public class CreateFirewallController extends RestrictedBaseController { for (int j = 0; j < termListData.size(); j++) { jpaTermList = (TermList) termListData.get(j); if (jpaTermList.getTermName().equals(termCollectorList.get(tl))) { - ruleDesc = jpaTermList.getTermDescription(); + ruleDesc = jpaTermList.getDescription(); if ((ruleDesc != null) && (!ruleDesc.isEmpty())) { targetTerm.setDescription(ruleDesc); } @@ -545,14 +545,14 @@ public class CreateFirewallController extends RestrictedBaseController { mapToZone = new HashMap<>(); mapToZone.put(tl, ruleToZone); } - ruleSrcPrefixList = jpaTermList.getSrcIPList(); + ruleSrcPrefixList = jpaTermList.getSrcIpList(); if ((ruleSrcPrefixList != null) && (!ruleSrcPrefixList.isEmpty())) { mapSrcIp = new HashMap<>(); mapSrcIp.put(tl, ruleSrcPrefixList); } - ruleDestPrefixList = jpaTermList.getDestIPList(); + ruleDestPrefixList = jpaTermList.getDestIpList(); if ((ruleDestPrefixList != null) && (!ruleDestPrefixList.isEmpty())) { mapDestIP = new HashMap<>(); mapDestIP.put(tl, ruleDestPrefixList); @@ -741,7 +741,7 @@ public class CreateFirewallController extends RestrictedBaseController { sl = mappingServiceList(t); targetSl.setName(sl.getServiceName()); targetSl.setDescription(sl.getServiceDescription()); - targetSl.setTransportProtocol(sl.getServiceTransProtocol()); + targetSl.setTransportProtocol(sl.getServiceTransportProtocol()); targetSl.setType(sl.getServiceType()); targetSl.setPorts(sl.getServicePorts()); servListArray.add(targetSl); @@ -794,7 +794,7 @@ public class CreateFirewallController extends RestrictedBaseController { targetSl.setName(expandGroupSl.getServiceName()); targetSl.setDescription(expandGroupSl.getServiceDescription()); - targetSl.setTransportProtocol(expandGroupSl.getServiceTransProtocol()); + targetSl.setTransportProtocol(expandGroupSl.getServiceTransportProtocol()); targetSl.setType(expandGroupSl.getServiceType()); targetSl.setPorts(expandGroupSl.getServicePorts()); servListArray.add(targetSl); diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/dao/SystemLogDbDao.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/dao/SystemLogDbDao.java index 3c3f4d397..8883475b8 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/dao/SystemLogDbDao.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/dao/SystemLogDbDao.java @@ -22,10 +22,10 @@ package org.onap.policy.dao; import java.util.List; -import org.onap.policy.rest.jpa.SystemLogDB; +import org.onap.policy.rest.jpa.SystemLogDb; public abstract interface SystemLogDbDao { - public abstract List getLoggingData(); + public abstract List getLoggingData(); - public abstract List getSystemAlertData(); + public abstract List getSystemAlertData(); } diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/daoImp/SystemLogDbDaoImpl.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/daoImp/SystemLogDbDaoImpl.java index 768a7a41c..46aed213f 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/daoImp/SystemLogDbDaoImpl.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/daoImp/SystemLogDbDaoImpl.java @@ -31,7 +31,7 @@ import org.onap.policy.common.logging.flexlogger.Logger; import org.onap.policy.conf.HibernateSession; import org.onap.policy.controller.PolicyController; import org.onap.policy.dao.SystemLogDbDao; -import org.onap.policy.rest.jpa.SystemLogDB; +import org.onap.policy.rest.jpa.SystemLogDb; import org.onap.policy.xacml.api.XACMLErrorConstants; import org.springframework.stereotype.Service; @@ -41,10 +41,10 @@ public class SystemLogDbDaoImpl implements SystemLogDbDao { @SuppressWarnings("unchecked") @Override - public List getLoggingData() { + public List getLoggingData() { Session session = HibernateSession.getSession(); Transaction tx = session.beginTransaction(); - List system = null; + List system = null; try { String sqlWhere = null; if (PolicyController.isjUnit()) { @@ -53,12 +53,12 @@ public class SystemLogDbDaoImpl implements SystemLogDbDao { sqlWhere = "date > DATE_SUB(curdate(), INTERVAL 5 DAY) ORDER BY date DESC limit " + PolicyController.getLogTableLimit() + ""; } - Criteria cr = session.createCriteria(SystemLogDB.class); + Criteria cr = session.createCriteria(SystemLogDb.class); cr.add(Restrictions.sqlRestriction(sqlWhere)); system = cr.list(); tx.commit(); } catch (Exception e) { - logger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW + "Error While Querying SystemLogDB Table" + e); + logger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW + "Error While Querying SystemLogDb Table" + e); } finally { try { session.close(); @@ -71,10 +71,10 @@ public class SystemLogDbDaoImpl implements SystemLogDbDao { @SuppressWarnings("unchecked") @Override - public List getSystemAlertData() { + public List getSystemAlertData() { Session session = HibernateSession.getSession(); Transaction tx = session.beginTransaction(); - List system = null; + List system = null; try { String sqlWhere = null; if (PolicyController.isjUnit()) { @@ -83,12 +83,12 @@ public class SystemLogDbDaoImpl implements SystemLogDbDao { sqlWhere = "date > DATE_SUB(curdate(), INTERVAL 5 DAY) and logtype = 'error' ORDER BY date DESC limit " + PolicyController.getSystemAlertTableLimit() + ""; } - Criteria cr = session.createCriteria(SystemLogDB.class); + Criteria cr = session.createCriteria(SystemLogDb.class); cr.add(Restrictions.sqlRestriction(sqlWhere)); system = cr.list(); tx.commit(); } catch (Exception e) { - logger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW + "Error While Querying SystemLogDB Table" + e); + logger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW + "Error While Querying SystemLogDb Table" + e); } finally { try { session.close(); -- cgit 1.2.3-korg