@abaplint/cli 2.113.90 → 2.113.91

package/build/cli.js

@@ -19243,15 +19243,11 @@ exports.Do = Do;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DynproLogic = void 0;
-const Statements = __webpack_require__(/*! ../../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Structures = __webpack_require__(/*! ./ */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/index.js");
 const _combi_1 = __webpack_require__(/*! ./_combi */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/_combi.js");
-const chain_1 = __webpack_require__(/*! ./chain */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/chain.js");
 class DynproLogic {
     getMatcher() {
-        const pai = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen), (0, _combi_1.sub)(chain_1.Chain)));
-        const pbo = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen)));
-        const pov = (0, _combi_1.star)((0, _combi_1.sta)(Statements.Field));
-        return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessBeforeOutput), pbo, (0, _combi_1.sta)(Statements.ProcessAfterInput), pai, (0, _combi_1.opt)((0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessOnValueRequest), pov)));
+        return (0, _combi_1.seq)((0, _combi_1.sub)(Structures.ProcessBeforeOutput), (0, _combi_1.sub)(Structures.ProcessAfterInput), (0, _combi_1.opt)((0, _combi_1.sub)(Structures.ProcessOnValueRequest)));
     }
 }
 exports.DynproLogic = DynproLogic;
@@ -19482,10 +19478,10 @@ __exportStar(__webpack_require__(/*! ./case_type */ "./node_modules/@abaplint/co
 __exportStar(__webpack_require__(/*! ./case */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/case.js"), exports);
 __exportStar(__webpack_require__(/*! ./catch_system_exceptions */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/catch_system_exceptions.js"), exports);
 __exportStar(__webpack_require__(/*! ./catch */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/catch.js"), exports);
+__exportStar(__webpack_require__(/*! ./chain */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/chain.js"), exports);
 __exportStar(__webpack_require__(/*! ./class_data */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/class_data.js"), exports);
 __exportStar(__webpack_require__(/*! ./class_definition */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/class_definition.js"), exports);
 __exportStar(__webpack_require__(/*! ./class_global */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/class_global.js"), exports);
-__exportStar(__webpack_require__(/*! ./chain */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/chain.js"), exports);
 __exportStar(__webpack_require__(/*! ./class_implementation */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/class_implementation.js"), exports);
 __exportStar(__webpack_require__(/*! ./cleanup */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/cleanup.js"), exports);
 __exportStar(__webpack_require__(/*! ./constants */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/constants.js"), exports);
@@ -19510,6 +19506,9 @@ __exportStar(__webpack_require__(/*! ./module */ "./node_modules/@abaplint/core/
 __exportStar(__webpack_require__(/*! ./normal */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/normal.js"), exports);
 __exportStar(__webpack_require__(/*! ./on_change */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/on_change.js"), exports);
 __exportStar(__webpack_require__(/*! ./private_section */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/private_section.js"), exports);
+__exportStar(__webpack_require__(/*! ./process_after_input */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_after_input.js"), exports);
+__exportStar(__webpack_require__(/*! ./process_before_output */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_before_output.js"), exports);
+__exportStar(__webpack_require__(/*! ./process_on_value_request */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_on_value_request.js"), exports);
 __exportStar(__webpack_require__(/*! ./protected_section */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/protected_section.js"), exports);
 __exportStar(__webpack_require__(/*! ./provide */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/provide.js"), exports);
 __exportStar(__webpack_require__(/*! ./public_section */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/public_section.js"), exports);
@@ -19742,6 +19741,76 @@ exports.PrivateSection = PrivateSection;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_after_input.js":
+/*!***************************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_after_input.js ***!
+  \***************************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.ProcessAfterInput = void 0;
+const Statements = __webpack_require__(/*! ../../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const _combi_1 = __webpack_require__(/*! ./_combi */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/_combi.js");
+const chain_1 = __webpack_require__(/*! ./chain */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/chain.js");
+class ProcessAfterInput {
+    getMatcher() {
+        const pai = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen), (0, _combi_1.sub)(chain_1.Chain)));
+        return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessAfterInput), pai);
+    }
+}
+exports.ProcessAfterInput = ProcessAfterInput;
+//# sourceMappingURL=process_after_input.js.map
+
+/***/ }),
+
+/***/ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_before_output.js":
+/*!*****************************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_before_output.js ***!
+  \*****************************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.ProcessBeforeOutput = void 0;
+const Statements = __webpack_require__(/*! ../../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const _combi_1 = __webpack_require__(/*! ./_combi */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/_combi.js");
+class ProcessBeforeOutput {
+    getMatcher() {
+        const pbo = (0, _combi_1.star)((0, _combi_1.alt)((0, _combi_1.sta)(Statements.Module), (0, _combi_1.sta)(Statements.Field), (0, _combi_1.sta)(Statements.CallSubscreen)));
+        return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessBeforeOutput), pbo);
+    }
+}
+exports.ProcessBeforeOutput = ProcessBeforeOutput;
+//# sourceMappingURL=process_before_output.js.map
+
+/***/ }),
+
+/***/ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_on_value_request.js":
+/*!********************************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/3_structures/structures/process_on_value_request.js ***!
+  \********************************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.ProcessOnValueRequest = void 0;
+const Statements = __webpack_require__(/*! ../../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const _combi_1 = __webpack_require__(/*! ./_combi */ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/_combi.js");
+class ProcessOnValueRequest {
+    getMatcher() {
+        const pov = (0, _combi_1.star)((0, _combi_1.sta)(Statements.Field));
+        return (0, _combi_1.seq)((0, _combi_1.sta)(Statements.ProcessOnValueRequest), pov);
+    }
+}
+exports.ProcessOnValueRequest = ProcessOnValueRequest;
+//# sourceMappingURL=process_on_value_request.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/3_structures/structures/protected_section.js":
 /*!*************************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/3_structures/structures/protected_section.js ***!
@@ -40318,6 +40387,13 @@ class CDSDetermineTypes {
                         continue;
                     }
                     if ((source === null || source === void 0 ? void 0 : source.name) === undefined) {
+                        if (prefixUpper.startsWith("_")) {
+                            components.push({
+                                name: f.name,
+                                type: new basic_1.VoidType("DDLS:association"),
+                            });
+                            continue;
+                        }
                         components.push({
                             name: f.name,
                             type: new basic_1.UnknownType("CDS parser error, unknown source, " + ddlsName),
@@ -44174,15 +44250,22 @@ const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modu
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
 const dump_scope_1 = __webpack_require__(/*! ./dump_scope */ "./node_modules/@abaplint/core/build/src/lsp/dump_scope.js");
 const virtual_position_1 = __webpack_require__(/*! ../virtual_position */ "./node_modules/@abaplint/core/build/src/virtual_position.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 class Help {
     static find(reg, textDocument, position) {
-        const file = _lsp_utils_1.LSPUtils.getABAPFile(reg, textDocument.uri);
+        const abapFile = _lsp_utils_1.LSPUtils.getABAPFile(reg, textDocument.uri);
+        if (abapFile !== undefined) {
+            return this.dumpABAP(abapFile, reg, textDocument, position);
+        }
+        const file = reg.getFileByName(textDocument.uri);
         if (file === undefined) {
-            return "file not found";
+            return "File not found: " + textDocument.uri;
         }
-        else {
-            return this.dumpABAP(file, reg, textDocument, position);
+        const obj = reg.findObjectForFile(file);
+        if (obj instanceof objects_1.DataDefinition) {
+            return "Data definition, dump todo";
         }
+        return "Unhandled object type: " + obj.getType();
     }
     /////////////////////////////////////////////////
     static dumpABAP(file, reg, textDocument, position) {
@@ -53674,7 +53757,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.90";
+        return "2.113.91";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -66107,7 +66190,7 @@ class LocalTestclassConsistency extends _abap_rule_1.ABAPRule {
         this.conf = conf;
     }
     runParsed(file, obj) {
-        var _a;
+        var _a, _b, _c;
         const issues = [];
         if (this.reg.getConfig().getVersion() === version_1.Version.v700) {
             // 700 does not have testclass includes
@@ -66133,6 +66216,16 @@ class LocalTestclassConsistency extends _abap_rule_1.ABAPRule {
                 issues.push(issue);
             }
         }
+        if (file.getFilename() === ((_b = obj.getMainABAPFile()) === null || _b === void 0 ? void 0 : _b.getFilename())
+            && obj.getTestclassFile() === undefined
+            && ((_c = obj.getXML()) === null || _c === void 0 ? void 0 : _c.includes("<WITH_UNIT_TESTS>X</WITH_UNIT_TESTS>")) === true) {
+            const id = obj.getIdentifier();
+            if (id) {
+                const message = "Has <WITH_UNIT_TESTS> set in XML, but no testclasses";
+                const issue = issue_1.Issue.atIdentifier(id, message, this.getMetadata().key, this.conf.severity);
+                issues.push(issue);
+            }
+        }
         return issues;
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.90",
+    "version": "2.113.91",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,12 +38,12 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.90",
+        "@abaplint/core": "^2.113.91",
         "@types/chai": "^4.3.20",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^22.10.6",
+        "@types/node": "^22.10.7",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "chalk": "^5.4.1",