@abaplint/cli 2.115.20 → 2.115.22

package/build/cli.js

@@ -3967,7 +3967,7 @@ const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/co
 const method_name_1 = __webpack_require__(/*! ./method_name */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/method_name.js");
 class AbstractMethods extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.seq)("ABSTRACT METHODS", (0, combi_1.plusPrio)(method_name_1.MethodName));
+        return (0, combi_1.seq)("ABSTRACT METHODS", (0, combi_1.plus)(method_name_1.MethodName));
     }
 }
 exports.AbstractMethods = AbstractMethods;
@@ -14854,7 +14854,7 @@ class InterfaceDef {
     getMatcher() {
         const val = (0, combi_1.seq)(expressions_1.AttributeName, "=", expressions_1.Source);
         const dataValues = (0, combi_1.seq)("DATA VALUES", (0, combi_1.plus)(val));
-        const options = (0, combi_1.alt)(expressions_1.AbstractMethods, expressions_1.FinalMethods, "ALL METHODS ABSTRACT", "ALL METHODS FINAL", (0, combi_1.ver)(version_1.Version.v740sp02, "PARTIALLY IMPLEMENTED", version_1.Version.OpenABAP));
+        const options = (0, combi_1.alt)((0, combi_1.seq)(expressions_1.AbstractMethods, (0, combi_1.opt)(expressions_1.FinalMethods)), expressions_1.FinalMethods, "ALL METHODS ABSTRACT", "ALL METHODS FINAL", (0, combi_1.ver)(version_1.Version.v740sp02, "PARTIALLY IMPLEMENTED", version_1.Version.OpenABAP));
         return (0, combi_1.seq)("INTERFACES", expressions_1.InterfaceName, (0, combi_1.opt)(options), (0, combi_1.opt)(dataValues));
     }
 }
@@ -15420,7 +15420,8 @@ class Move {
         const equals = (0, combi_1.altPrio)((0, combi_1.altPrio)(chained, "?="), calcAssign);
         // todo, move "?=" to CAST?
         const eq = (0, combi_1.seq)(expressions_1.Target, equals, expressions_1.Source);
-        return (0, combi_1.altPrio)(move, eq);
+        const eqd = (0, combi_1.seq)(expressions_1.Target, equals, expressions_1.Source, expressions_1.Dereference);
+        return (0, combi_1.alt)(move, eq, eqd);
     }
 }
 exports.Move = Move;
@@ -33639,6 +33640,7 @@ const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_module
 const inline_data_1 = __webpack_require__(/*! ../expressions/inline_data */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/inline_data.js");
 const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+const dereference_1 = __webpack_require__(/*! ../expressions/dereference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/dereference.js");
 class Move {
     runSyntax(node, input) {
         const targets = node.findDirectExpressions(Expressions.Target);
@@ -33655,12 +33657,15 @@ class Move {
             }
         }
         const source = node.findDirectExpression(Expressions.Source);
-        const sourceType = source ? source_1.Source.runSyntax(source, input, targetType) : undefined;
+        let sourceType = source ? source_1.Source.runSyntax(source, input, targetType) : undefined;
         if (sourceType === undefined) {
             const message = "No source type determined";
             input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
             return;
         }
+        if (node.findDirectExpression(Expressions.Dereference)) {
+            sourceType = dereference_1.Dereference.runSyntax(node, sourceType, input);
+        }
         if (inline) {
             inline_data_1.InlineData.runSyntax(inline, input, sourceType);
             targetType = sourceType;
@@ -55480,7 +55485,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.115.20";
+        return "2.115.22";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -65424,6 +65429,7 @@ const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@ab
 const Objects = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
 class GlobalClassConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.GlobalClassConf = GlobalClassConf;
@@ -65445,7 +65451,9 @@ class GlobalClass extends _abap_rule_1.ABAPRule {
 
 * global classes must be global definitions
 
-* global interfaces must be global definitions`,
+* global interfaces must be global definitions
+
+* global FOR TESTING, must have CATEGORY = 05 in the XML`,
             tags: [_irule_1.RuleTag.Syntax],
         };
     }
@@ -65470,6 +65478,18 @@ class GlobalClass extends _abap_rule_1.ABAPRule {
                 const issue = issue_1.Issue.atIdentifier(definition.identifier, "Class must be local", this.getMetadata().key, this.conf.severity);
                 output.push(issue);
             }
+            if (definition.isGlobal && obj instanceof Objects.Class) {
+                if (definition.isForTesting === true && obj instanceof Objects.Class && obj.getCategory() !== objects_1.ClassCategory.Test) {
+                    const message = "Class is marked as FOR TESTING, but CATEGORY is not 05 in the XML";
+                    const issue = issue_1.Issue.atIdentifier(definition.identifier, message, this.getMetadata().key, this.conf.severity);
+                    output.push(issue);
+                }
+                else if (definition.isForTesting === false && obj instanceof Objects.Class && obj.getCategory() === objects_1.ClassCategory.Test) {
+                    const message = "Class has CATEGORY 05 in the XML, but is not marked as FOR TESTING";
+                    const issue = issue_1.Issue.atIdentifier(definition.identifier, message, this.getMetadata().key, this.conf.severity);
+                    output.push(issue);
+                }
+            }
         }
         for (const impl of file.getInfo().listClassImplementations()) {
             if (file.getFilename().match(/\.clas\.abap$/)

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.115.20",
+    "version": "2.115.22",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,11 +38,11 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.115.20",
+        "@abaplint/core": "^2.115.22",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.10.10",
+        "@types/node": "^24.10.12",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",