summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-03-02Reduce Technical Debt on drools-applicationsliamfallon18-193/+199
This commit mops up the technical debt issues in drools-applications that are reaistic to fix witout major surgery on the code. note: warnings on non compliant Java class names that are used in yaml serializaiton and deserailization are ignored. note: warnings on complexity figures for equals() methods are ignored. Issue-ID: POLICY-455 Change-Id: I99ccda4bd9932ceb5ccbcfd2778730234b55d7a5 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-21Support vDNS scale out for multiple timesSwapnilPathak1-1/+34
Generation of Unique values for attribute vf-module-name for every scale out operation Change-Id: I5274361c2f5a4136020a384dbe4a4009607f589a Issue-ID: POLICY-610 Signed-off-by: SwapnilPathak <SP00494485@techmahindra.com>
2018-02-19Rename maven modules to mirror directory structureliamfallon85-5937/+5976
The naming of Maven modules in drools-applications was not aligned with the directory structure in the git repository of drools-applications. Therefore it was difficult to see the strucutre of the repository in Eclipse and other IDEs. This change amends the Maven module IDs to reflect the repository directory structure. This patch reset fixes the previos patch set, where many references to maven modules internally in drools-applciations were missed. See also changes in engine and docker repos. Updated to reflect repo directory structure in maven artifact groups. Issue-ID: POLICY-238 Change-Id: I8ab1a7ecdb664045222bbbfda269135e3e449109 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-16Modify sonar settings to include all sub-modulesPamela Dragosh1-4/+79
We are now including all the sub-modules coverage into one file. This will now reflect correct statistics for any sub-module that tests code located in another sub-module. Issue-ID: POLICY-634 Change-Id: Id96eb9e9f637770932067bd803cb5ccdb571c05a Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
2018-02-12Merge "Fix technical debt/JUnit on policy-yaml"Jorge Hernandez31-686/+2359
2018-02-12Fix technical debt/JUnit on policy-yamlmmis31-686/+2359
Unit tests added and technical debt removed Issue-ID: POLICY-455 Change-Id: I31566f9386cb94d78e9af3d6261e31b9cad1cb53 Signed-off-by: mmis <michael.morris@ericsson.com>
2018-02-11Tidy dependencies and scopes in POMsliamfallon8-31/+6
There were some duplicate dependencies in the POMs that were causing warnings. These have been removed. In cases where the revision of the duplicate dependencies were different,the latest revision was used. The scope of the "junit" dependency was "provided" in some POMs. These have been corrected with the scope of "test" being used for junit dependencies. Issue-ID: POLICY-238 Change-Id: Ib1cc6f040b0c4d7f2b39ae02bc6fb4590d395809 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-09Merge "Upgrade XACML jar"Jorge Hernandez5-6/+6
2018-02-09Merge "Remove Technical Debt from MSB code"Pamela Dragosh2-113/+113
2018-02-09Upgrade XACML jarPamela Dragosh5-6/+6
XACML was released with minor fixes for XML files with comments. Mainly it contained upgrade to dependencies that had fixed security issues. One still remains but will not be able to be fixed. Issue-ID: POLICY-616 Change-Id: Ifaf4f92d8da878a1c49dff5fd028744d01999ff5 Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
2018-02-09Remove Technical Debt from MSB codeliamfallon2-113/+113
Some minor warnings showed up in Eclipse on the MSB module. This change removes those warnings. Issue-ID: POLICY-455 Change-Id: Ibdd8c563120178d73a55adfd0d8853b6bc82b820 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-08JUint/Technical Debt for eventmanagerliamfallon29-942/+2551
Unit test expanded for SO POJOs, technical debt removed in eventmanager. Change-Id: I7045f5af88fd2c15019befe29a0b4fcd69801c49 Signed-off-by: liamfallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-07Merge "Resolve security issues SONATYPE-2017-0359"Jorge Hernandez3-3/+3
2018-02-07Resolve security issues SONATYPE-2017-0359Pamela Dragosh3-3/+3
Upgrade httpclient to 4.5.5 - the latest version. Issue-ID: POLICY-612 Change-Id: I5e79918bc447889b1dbcd1bf897c3324f6a0cc0d Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
2018-02-07Merge "Fix technical debt/JUnit on MSB"Jorge Hernandez7-12/+147
2018-02-07Fix technical debt/JUnit on MSBeeimmis7-12/+147
Unit tests added and technical debt removed Issue-ID: POLICY-455 Change-Id: Ib10613ff7cac64bb576046e038b2acbd66152648 Signed-off-by: mmis <michael.morris@ericsson.com>
2018-02-07Log thrown exception in PIPEngineGetHistoryliamfallon1-1/+1
Sonar Critical error highlighted, with thrown execption not logged. The exception is now logged. Issue-ID: POLICY-455 Change-Id: If2969a4fe118d22dbd1fb11879ba028487848eb8 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-06Add JUnit and Remove Technical Debt in Guardliamfallon25-762/+1788
Unit test for guard added and technical debt removed. Issue-ID: POLICY-455 Change-Id: I440766660ddae078013e21b1991ee49c8e488bb3 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-02-05Fix package directory naming to lower caseLiam Fallon5-0/+0
In macOS (and windows) directory names are case insensitive, but in Linux they are case sensitive. Therefore, when the "actorServiceProvider" directory name was renamed to "actorserviceprovider", the change did not propogate into git when a Macbook was used for development. This error was discovered when working on a Ubuntu machine and is now fixed. Signed-off-by: liamfallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455 Change-Id: Ifebe3d37d42e79fff8da2370369967a25c371b11 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-31Merge "Fix technical debt/JUnit on SO/VFC/SO ACTOR"Jorge Hernandez15-235/+703
2018-01-31Merge "Fix technical debt/JUnit on actor service provider"Jorge Hernandez12-64/+140
2018-01-30Merge "Do not enforce hostname validation"Jorge Hernandez1-2/+11
2018-01-30Fix technical debt/JUnit on actor service providerliamfallon12-64/+140
Unit test for actor service provider added and technical debt removed. Change-Id: I9573f1c3ff0252b166e06caaa88eb679a1fc7347 Signed-off-by: liamfallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-30Fix technical debt/JUnit on SO/VFC/SO ACTORliamfallon15-235/+703
Unit test expanded for SO POJOs, technical debt removed in SO actor and VFC pojos. Change-Id: I23b886c40c1ac6ac8dc2ebbaade315b71cca9dd0 Signed-off-by: liamfallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455
2018-01-29Do not enforce hostname validationAlexis de Talhouët1-2/+11
Change-Id: I1c4e5d58126e35a64140c91c0c3afda5e9b3a5bd Issue-ID: POLICY-510 Signed-off-by: Alexis de Talhouët <adetalhouet89@gmail.com>
2018-01-28Fix Tech Debt/JUnit on SO actorliamfallon3-565/+338
Unit test for SO actor added. SO actor service provider class refactored to remove technical debt. Change-Id: I6186ede5661ce483917184f03ee44d57a124b6a4 Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-23Implement JUnit tests for actor.vfcliamfallon2-6/+108
JUnit tests for the VFC actor implemented. Change-Id: Ifcd433a4ce6f46cad46d50f4ce4c7c03e2abec67 Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-22Fix Tech Debt/JUnit on VFC actorliamfallon3-107/+126
Unit test for VFC actor added. Some minor chages to actor service provider class to remove technical debt. Change-Id: I54ee24bcfa1c82f4a18c0035bdff186d4d3ebdf7 Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-22Merge "Fix Tech Debt/JUnit on VFC POJOs"Jorge Hernandez20-196/+996
2018-01-21Fix Tech Debt/JUnit on APPC and APPCLCM actorsliamfallon5-321/+358
Unit test updated to increase coverage on getter methods. Constants added for some string literals to reduce technical debt warnings. Change-Id: I3b175de7c65aa9d342e71adc951d1ee08c9d781b Issue-ID: POLICY-455 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-20Fix Tech Debt/JUnit on VFC POJOsliamfallon20-196/+996
Making fields private, fixing field and methods to follow Java guidelines, and adding getter and setter methods. Unit test added for all classes in org.onap.policy.vfc Issue-ID: POLICY-455 Change-Id: I262337a816706cd5243849cd51e57689275545bf Signed-off-by: liamfallon <liam.fallon@ericsson.com>
2018-01-19Merge "Fix Technical Debt, add Unit tests for SO"Jorge Hernandez42-829/+2390
2018-01-18Merge "Fix Technical Debt, Unit Test for APPCLCM POJOs"Jorge Hernandez14-158/+835
2018-01-18Merge "Fix Tech Debt/JUnit on control loop event POJOs"Jorge Hernandez25-669/+989
2018-01-18Fix Technical Debt, Unit Test for APPCLCM POJOsLiam Fallon14-158/+835
Mainly making fields private, fixing field and methods to follow Java guidelines, and adding getter and setter methods. Unit test added for all classes in org.onap.policy.appclcm Issue-ID: POLICY-455 Change-Id: Iae873f94a236cc9faeac5542a9633ca840b696ed Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
2018-01-18Merge "Fix Technical Debt, basic JUnit tests for Rest"Jorge Hernandez4-85/+233
2018-01-18Fix Technical Debt, add Unit tests for SOvdmeer42-829/+2390
Converted to POJOs, fixed sona lint errors, added JUnit tests POJOs now have only private members and getters/setters This implicated actors, simulators, and existing tests Issue-ID: POLICY-455 Change-Id: I4b80f729565e8675822a890892b15676908f73b4 Signed-off-by: vdmeer.sven <sven.van.der.meer@ericsson.com>
2018-01-18Merge "Fix Technical Debt for TrafficGenerator"Pamela Dragosh5-55/+52
2018-01-18Fix Tech Debt/JUnit on control loop event POJOsLiam Fallon25-669/+989
Mainly making fields private, fixing field and methods to follow Java guidelines, and adding getter and setter methods. Unit test added for all classes in org.onap.policy.controlloop Issue-ID: POLICY-455 Change-Id: I445b7cfaf9eb921a230bbb72b06ff4455fe003ce Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
2018-01-17Fix Technical Debt, basic JUnit tests for Restvdmeer4-85/+233
Basic tests for get and post (not testing full user/pw/body) Issue-ID: POLICY-455 Change-Id: I186ebf969abc239e5056199c2fd8da9f3253d2fc Signed-off-by: vdmeer.sven <sven.van.der.meer@ericsson.com>
2018-01-17Fix Technical Debt for TrafficGeneratorvdmeer5-55/+52
Changed formatting for checkstyle, changed sequence of statements Left empty constructors as requested Issue-ID: POLICY-455 Change-Id: I0e443bba69cede4c006a841d8d34c82391ff1040 Signed-off-by: vdmeer.sven <sven.van.der.meer@ericsson.com>
2018-01-17Added all J Unit tests SDCvdmeer5-0/+622
Issue-ID: POLICY-455 Change-Id: I8192e8a595642e0d36608591d95f21f3ee574d13 Signed-off-by: vdmeer.sven <sven.van.der.meer@ericsson.com>
2018-01-16Fix Sonar Technical Debt, Unit Test for APPC POJOsLiam Fallon19-183/+907
Mainly making fields private, fixing field and methods to follow Java guidelines, and adding getter and setter methods. Unit test added for all classes in org.onap.policy.appc Change-Id: If3db740bc146a09f8f7387f02c12b048ad00b201 Signed-off-by: Liam Fallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455 Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
2018-01-15Restore getters and setters to AAINQResponse classLiam Fallon1-0/+12
Some get and set methods were deleted from this class in error. This patch restores them. Change-Id: I4dfaf1215e56f30a1d62d19624fe4da570e99171 Signed-off-by: Liam Fallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455
2018-01-15Merge "Cleanup of A&AI message model module."Jorge Hernandez74-917/+2067
2018-01-10Add IntelliJ files to .gitignoreAlexis de Talhouët1-0/+4
Change-Id: I45dd03d4295322ecc0f10c4673848c1fe85d5e98 Issue-ID: POLICY-509 Signed-off-by: Alexis de Talhouët <adetalhouet89@gmail.com>
2017-12-15Add guard junitsTemoc Rodriguez12-18/+555
Added a few junit files which test most functionality in the guard submodule. Some minor bugs were found by the junits and fixed those newly found bugs. Added persistence.xml for use by junits in guard. Issue-ID: POLICY-490 Change-Id: Ied7e276cfd417e1f396b5a6685a2e9acc9efd109 Signed-off-by: Temoc Rodriguez <cr056n@att.com>
2017-12-14Cleanup of A&AI message model module.Liam Fallon74-917/+2067
Fields have been made fields private in POJOs, and wrote JUnit unit tests to bring up code coverage. AAIManager also changed to use non-static methods so that it can be stubbed with Mockito. Change-Id: I871427ced5db8a1dfd6495fef4e6d84adf264e65 Signed-off-by: Liam Fallon <liam.fallon@ericsson.com> Issue-ID: POLICY-455 Signed-off-by: Liam Fallon <liam.fallon@ericsson.com>
2017-12-11Merge "Technical Debt in AAI."Pamela Dragosh48-866/+890
2017-12-08remove synchronized calls to avoid race conditionsJorge Hernandez1-26/+0
This is to prevent potential problems where multiple jar versions are updated one after the other. Change-Id: I78835b485e57125efb213cfdefba60351d815916 Issue-ID: POLICY-487 Signed-off-by: Jorge Hernandez <jh1730@att.com> (cherry picked from commit f2715bc0f272e5e657ed8a82c0f4e520a7c7706d)