abapgit-agent 1.17.6 → 1.17.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "abapgit-agent",
-  "version": "1.17.6",
+  "version": "1.17.8",
   "description": "ABAP Git Agent - Pull and activate ABAP code via abapGit from any git repository",
   "files": [
     "bin/",

package/src/commands/unit.js

@@ -327,7 +327,48 @@ Examples:
       }
     }
 
-    // JUnit output mode — write XML, then continue to normal output
+    // Coverage threshold enforcement — per file, runs BEFORE JUnit output
+    // so the injected failure testcase lands in the correct class's testsuite.
+    if (coverage && coverageThreshold > 0) {
+      let anyData = false;
+      for (const result of results) {
+        const stats = result.COVERAGE_STATS || result.coverage_stats;
+        if (!stats) continue;
+        anyData = true;
+        const totalLines   = stats.TOTAL_LINES   || stats.total_lines   || 0;
+        const coveredLines = stats.COVERED_LINES  || stats.covered_lines  || 0;
+        if (totalLines === 0) continue;
+        const rate = Math.round((coveredLines / totalLines) * 100);
+        const className = result._className || 'UNKNOWN';
+        if (rate < coverageThreshold) {
+          const msg = `${className}: coverage ${rate}% is below threshold ${coverageThreshold}%`;
+          if (coverageMode === 'warn') {
+            if (!jsonOutput) console.warn(`⚠️  ${msg}`);
+          } else {
+            if (!jsonOutput) console.error(`❌ ${msg}`);
+            hasErrors = true;
+            // Inject failure into this class's own testsuite so Jenkins shows
+            // which class failed the gate and what its actual coverage was.
+            const errors = result.ERRORS || result.errors || [];
+            errors.push({
+              CLASS_NAME:  className,
+              METHOD_NAME: 'coverage_threshold',
+              ERROR_KIND:  'FAILURE',
+              ERROR_TEXT:  msg
+            });
+            result.ERRORS = errors;
+            result.FAILED_COUNT = (result.FAILED_COUNT || result.failed_count || 0) + 1;
+          }
+        } else {
+          if (!jsonOutput) console.log(`✅ ${className}: coverage ${rate}% meets threshold ${coverageThreshold}%`);
+        }
+      }
+      if (!anyData) {
+        if (!jsonOutput) console.warn('⚠️  Coverage data unavailable — threshold not enforced');
+      }
+    }
+
+    // JUnit output mode — write XML after threshold check so synthetic failure is included
     if (junitOutput) {
       const xml = buildUnitJUnit(results);
       const outputPath = pathModule.isAbsolute(junitOutput)
@@ -343,29 +384,6 @@ Examples:
       }
     }
 
-    // Coverage threshold enforcement — aggregated across all files
-    if (coverage && coverageThreshold > 0) {
-      const totalLines   = results.reduce((s, r) => s + ((r.COVERAGE_STATS || r.coverage_stats)?.TOTAL_LINES   || (r.COVERAGE_STATS || r.coverage_stats)?.total_lines   || 0), 0);
-      const coveredLines = results.reduce((s, r) => s + ((r.COVERAGE_STATS || r.coverage_stats)?.COVERED_LINES || (r.COVERAGE_STATS || r.coverage_stats)?.covered_lines || 0), 0);
-
-      if (totalLines === 0) {
-        if (!jsonOutput) console.warn('⚠️  Coverage data unavailable — threshold not enforced');
-      } else {
-        const rate = Math.round((coveredLines / totalLines) * 100);
-        if (rate < coverageThreshold) {
-          const msg = `Coverage ${rate}% is below threshold ${coverageThreshold}%`;
-          if (coverageMode === 'warn') {
-            if (!jsonOutput) console.warn(`⚠️  ${msg}`);
-          } else {
-            if (!jsonOutput) console.error(`❌ ${msg}`);
-            hasErrors = true;
-          }
-        } else {
-          if (!jsonOutput) console.log(`✅ Coverage ${rate}% meets threshold ${coverageThreshold}%`);
-        }
-      }
-    }
-
     // JSON output mode
     if (jsonOutput) {
       console.log(JSON.stringify(results, null, 2));

package/src/config.js

@@ -233,6 +233,20 @@ function getScratchWorkspace() {
   };
 }
 
+/**
+ * Get coverage policy from project-level config (.abapgit-agent.json)
+ * @returns {{ threshold: number, mode: string, excludes: string[] }}
+ */
+function getCoverageConfig() {
+  const projectConfig = loadProjectConfig();
+  const coverage = projectConfig?.coverage || {};
+  return {
+    threshold: coverage.threshold || 0,
+    mode:      coverage.mode      || 'fail',
+    excludes:  Array.isArray(coverage.excludes) ? coverage.excludes : []
+  };
+}
+
 module.exports = {
   loadConfig,
   getAbapConfig,
@@ -246,5 +260,6 @@ module.exports = {
   loadProjectConfig,
   getTransportHookConfig,
   getTransportSettings,
-  getScratchWorkspace
+  getScratchWorkspace,
+  getCoverageConfig
 };