@abaplint/cli 2.113.232 → 2.113.233

package/build/cli.js

@@ -23979,9 +23979,30 @@ class TypeUtils {
                 return true;
             }
             else if (source instanceof basic_1.StructureType) {
-                if (this.structureContainsString(target) && !this.structureContainsString(source)) {
+                const targetDeep = this.structureContainsString(target);
+                const sourceDeep = this.structureContainsString(source);
+                if (targetDeep && !sourceDeep) {
                     return false;
                 }
+                const targetComponents = target.getComponents();
+                const sourceComponents = source.getComponents();
+                if (targetComponents.length !== sourceComponents.length) {
+                    if (targetDeep === true || sourceDeep === true) {
+                        return false;
+                    }
+                }
+                for (let i = 0; i < targetComponents.length; i++) {
+                    if (sourceComponents[i] === undefined) {
+                        continue;
+                    }
+                    // hmm
+                    if (sourceComponents[i].type instanceof basic_1.StringType && !(targetComponents[i].type instanceof basic_1.StringType)) {
+                        return false;
+                    }
+                    else if (!(sourceComponents[i].type instanceof basic_1.StringType) && targetComponents[i].type instanceof basic_1.StringType) {
+                        return false;
+                    }
+                }
                 return true;
             }
             else if (target.containsVoid() === true) {
@@ -25977,7 +25998,7 @@ class FieldAssignment {
                 const text = c.concatTokens();
                 if (text !== "-" && context instanceof basic_1.StructureType) {
                     context = context.getComponentByName(text);
-                    if (context === undefined && targetType.containsVoid() === false) {
+                    if (context === undefined) {
                         const message = `field ${text} does not exist in structure`;
                         input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                         return;
@@ -33131,6 +33152,7 @@ const _object_oriented_1 = __webpack_require__(/*! ../_object_oriented */ "./nod
 const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
+const visibility_1 = __webpack_require__(/*! ../../4_file_information/visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
 class MethodImplementation {
     runSyntax(node, input) {
         const helper = new _object_oriented_1.ObjectOriented(input.scope);
@@ -33144,6 +33166,13 @@ class MethodImplementation {
             return;
         }
         const { method: methodDefinition } = helper.searchMethodName(classDefinition, methodName);
+        if (classDefinition.isForTesting()
+            && (methodDefinition === null || methodDefinition === void 0 ? void 0 : methodDefinition.getVisibility()) !== visibility_1.Visibility.Private
+            && ["SETUP", "TEARDOWN", "CLASS_SETUP", "CLASS_TEARDOWN"].includes(methodName.toUpperCase())) {
+            const message = "Special test method \"" + methodName + "\" must be private";
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+            return;
+        }
         const start = node.getFirstToken().getStart();
         if ((methodDefinition === null || methodDefinition === void 0 ? void 0 : methodDefinition.isStatic()) === false) {
             input.scope.push(_scope_type_1.ScopeType.MethodInstance, methodName, start, input.filename);
@@ -33984,6 +34013,11 @@ class ReadTable {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
                 return;
             }
+            else if (sourceType instanceof basic_1.TableType && sourceType.getAccessType() === basic_1.TableAccessType.hashed) {
+                const message = "INDEX on hashed table not possible";
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                return;
+            }
         }
         const fromSource = node.findExpressionAfterToken("FROM");
         if (fromSource) {
@@ -54737,7 +54771,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.232";
+        return "2.113.233";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.232",
+    "version": "2.113.233",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,16 +38,16 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.232",
+        "@abaplint/core": "^2.113.233",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.7.2",
+        "@types/node": "^24.9.1",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",
         "chalk": "^5.6.2",
-        "eslint": "^9.37.0",
+        "eslint": "^9.38.0",
         "glob": "^11.0.3",
         "json5": "^2.2.3",
         "memfs": "^4.49.0",