From: vempo <vitaliy.emporopulo@amdocs.com>
Date: Mon, 23 Oct 2017 14:03:24 +0000 (+0300)
Subject: Run static analysis on Onboarding production code
X-Git-Tag: v1.2.0~798^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=e0c1d8c61904a4f6491964949b193eb604a206be;p=sdc.git

Run static analysis on Onboarding production code

Enable static analysis to run by default, and fail the
build if violations are found in the Onboarding code.
For now check only the production code and only for
high-severity issues (e.g. not closing files).

Change-Id: I2567eb8c704310109b01cb8b6e2a345522870e2b
Issue-ID: SDC-497
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
---

diff --git a/onboarding/pom.xml b/onboarding/pom.xml
index 0582c764af..50f4649c11 100644
--- a/onboarding/pom.xml
+++ b/onboarding/pom.xml
@@ -2,7 +2,6 @@
          xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
     <modelVersion>4.0.0</modelVersion>
 
-    <groupId>org.openecomp.sdc</groupId>
     <artifactId>sdc-onboarding</artifactId>
     <packaging>pom</packaging>
 
@@ -108,7 +107,7 @@
                 <artifactId>maven-jxr-plugin</artifactId>
                 <version>2.5</version>
                 <configuration>
-                    <skip>${skipSA}</skip>
+                    <skip>${pmd.skip}</skip>
                 </configuration>
                 <executions>
                     <execution>
@@ -126,9 +125,10 @@
                 <artifactId>maven-pmd-plugin</artifactId>
                 <version>3.8</version>
                 <configuration>
-                    <skip>${skipSA}</skip>
+                    <skip>${pmd.skip}</skip>
                     <aggregate>false</aggregate>
-                    <targetJdk>${maven.compiler.target}</targetJdk>
+                    <includeTests>${pmd.include.tests}</includeTests>
+                    <printFailingErrors>true</printFailingErrors>
                     <rulesets>
                         <ruleset>build-pmd-ruleset.xml</ruleset>
                     </rulesets>