abapgit-agent 1.19.0 → 1.19.1

package/package.json

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

package/src/commands/inspect.js

@@ -48,8 +48,10 @@ function buildInspectJUnit(results) {
 
     const testcases = [];
 
+    const inspectClass = `${escapeXml(objectName)}.Inspect`;
+
     if (errorCount === 0 && warnCount === 0) {
-      testcases.push(`    <testcase name="Inspect / Syntax check" classname="${escapeXml(objectName)}"/>`);
+      testcases.push(`    <testcase name="Syntax check" classname="${inspectClass}"/>`);
     }
 
     for (const err of errors) {
@@ -67,9 +69,9 @@ function buildInspectJUnit(results) {
         text
       ].filter(Boolean).join('\n');
       const checkId  = checkClass && checkCode ? `${checkClass}/${checkCode}` : null;
-      const caseName = checkId ? `Inspect / ${checkId}` : (methodName ? `${methodName} line ${line}` : `Line ${line}`);
+      const caseName = checkId ? checkId : (methodName ? `${methodName} line ${line}` : `Line ${line}`);
       testcases.push(
-        `    <testcase name="${escapeXml(caseName)}" classname="${escapeXml(objectName)}">\n` +
+        `    <testcase name="${escapeXml(caseName)}" classname="${inspectClass}">\n` +
         `      <failure type="SyntaxError" message="${escapeXml(text)}">${escapeXml(detail)}</failure>\n` +
         `    </testcase>`
       );
@@ -89,9 +91,9 @@ function buildInspectJUnit(results) {
         text
       ].filter(Boolean).join('\n');
       const checkId  = checkClass && checkCode ? `${checkClass}/${checkCode}` : null;
-      const caseName = checkId ? `Inspect / ${checkId}` : (methodName ? `${methodName} line ${line}` : `Line ${line}`);
+      const caseName = checkId ? checkId : (methodName ? `${methodName} line ${line}` : `Line ${line}`);
       testcases.push(
-        `    <testcase name="${escapeXml(caseName)}" classname="${escapeXml(objectName)}">\n` +
+        `    <testcase name="${escapeXml(caseName)}" classname="${inspectClass}">\n` +
         `      <failure type="Warning" message="${escapeXml(text)}">${escapeXml(detail)}</failure>\n` +
         `    </testcase>`
       );

package/src/commands/unit.js

@@ -158,6 +158,9 @@ function buildUnitJUnit(results) {
     const failedMethods = methods.filter(m => !m.passed);
     const syntheticFailures = thresholdFailure ? 1 : 0;
     const totalFailures = failedMethods.length + syntheticFailures;
+    // classname "ZCL_MY_TEST.LTCL_UNIT_TEST" groups under the same ABAP class node
+    // as "ZCL_MY_TEST.Inspect" from the inspect report — one node per ABAP class.
+    const suiteLabel = className;
 
     const lines = [];
 
@@ -170,13 +173,12 @@ function buildUnitJUnit(results) {
     }
 
     if (testCount === 0) {
-      lines.push(`    <testcase name="(no tests)" classname="${escapeXml(className)}"/>`);
+      lines.push(`    <testcase name="(no tests)" classname="${escapeXml(suiteLabel)}"/>`);
     } else {
       for (const m of methods) {
         const timeAttr = m.executionTime != null ? ` time="${m.executionTime}"` : '';
-        // Use "ZCL_MY_TEST.LTCL_UNIT_TEST" so Jenkins groups by ABAP class first,
-        // then local test class — giving a clear package hierarchy in the test report.
-        const testClass = m.testClassName ? `${className}.${m.testClassName}` : className;
+        // "Unit Tests / ZCL_MY_TEST.LTCL_UNIT_TEST" groups by namespace → class → local test class.
+        const testClass = m.testClassName ? `${suiteLabel}.${m.testClassName}` : suiteLabel;
         if (m.passed) {
           lines.push(`    <testcase name="${escapeXml(m.name)}" classname="${escapeXml(testClass)}"${timeAttr}/>`);
         } else {
@@ -195,14 +197,14 @@ function buildUnitJUnit(results) {
 
     if (thresholdFailure) {
       lines.push(
-        `    <testcase name="coverage_threshold" classname="${escapeXml(className)}">\n` +
+        `    <testcase name="coverage_threshold" classname="${escapeXml(suiteLabel)}">\n` +
         `      <failure type="FAILURE" message="${escapeXml(thresholdFailure)}">${escapeXml(thresholdFailure)}</failure>\n` +
         `    </testcase>`
       );
     }
 
     return (
-      `  <testsuite name="${escapeXml(className)}" ` +
+      `  <testsuite name="${escapeXml(suiteLabel)}" ` +
       `tests="${Math.max(testCount + syntheticFailures, 1)}" failures="${totalFailures}" errors="0">\n` +
       lines.join('\n') + '\n' +
       `  </testsuite>`