@abaplint/transpiler 2.5.38 → 2.5.39

package/build/src/types.d.ts

@@ -1,58 +1,58 @@
-import { Chunk } from "./chunk";
-import * as abaplint from "@abaplint/core";
-import { TestMethodList } from "./unit_test";
-import { DatabaseSetupResult } from "./db/database_setup_result";
-export interface IFile {
-    filename: string;
-    relative?: string;
-    contents: string;
-}
-export interface IObjectIdentifier {
-    name: string;
-    type: string;
-}
-export interface IOutput {
-    objects: IOutputFile[];
-    reg: abaplint.IRegistry;
-    unitTestScript: string;
-    unitTestScriptOpen: string;
-    initializationScript: string;
-    initializationScript2: string;
-    databaseSetup: DatabaseSetupResult;
-}
-export interface IRequire {
-    name: string | undefined;
-    filename: string;
-}
-export interface IProgress {
-    set(total: number, text: string): void;
-    tick(text: string): Promise<void>;
-}
-/** one javascript output file for each object */
-export interface IOutputFile {
-    object: IObjectIdentifier;
-    filename: string;
-    chunk: Chunk;
-    requires: readonly IRequire[];
-    exports: readonly string[];
-}
-export interface ITranspilerOptions {
-    /** ignore syntax check, used for internal testing */
-    ignoreSyntaxCheck?: boolean;
-    /** adds common js modules */
-    addCommonJS?: boolean;
-    /** adds filenames as comments in the output js */
-    addFilenames?: boolean;
-    /** skip outputing constants, used for internal testing */
-    skipConstants?: boolean;
-    /** sets behavior for unknown types, either fail at compile- or run-time */
-    unknownTypes?: "compileError" | "runtimeError";
-    skip?: TestMethodList;
-    only?: TestMethodList;
-    /** extra setup script to be executed during initialization */
-    extraSetup?: string;
-    /** list of keywords to rename, if not supplied default will be used */
-    keywords?: string[];
-    /** dont insert into REPOSRC */
-    skipReposrc?: boolean;
-}
+import { Chunk } from "./chunk";
+import * as abaplint from "@abaplint/core";
+import { TestMethodList } from "./unit_test";
+import { DatabaseSetupResult } from "./db/database_setup_result";
+export interface IFile {
+    filename: string;
+    relative?: string;
+    contents: string;
+}
+export interface IObjectIdentifier {
+    name: string;
+    type: string;
+}
+export interface IOutput {
+    objects: IOutputFile[];
+    reg: abaplint.IRegistry;
+    unitTestScript: string;
+    unitTestScriptOpen: string;
+    initializationScript: string;
+    initializationScript2: string;
+    databaseSetup: DatabaseSetupResult;
+}
+export interface IRequire {
+    name: string | undefined;
+    filename: string;
+}
+export interface IProgress {
+    set(total: number, text: string): void;
+    tick(text: string): Promise<void>;
+}
+/** one javascript output file for each object */
+export interface IOutputFile {
+    object: IObjectIdentifier;
+    filename: string;
+    chunk: Chunk;
+    requires: readonly IRequire[];
+    exports: readonly string[];
+}
+export interface ITranspilerOptions {
+    /** ignore syntax check, used for internal testing */
+    ignoreSyntaxCheck?: boolean;
+    /** adds common js modules */
+    addCommonJS?: boolean;
+    /** adds filenames as comments in the output js */
+    addFilenames?: boolean;
+    /** skip outputing constants, used for internal testing */
+    skipConstants?: boolean;
+    /** sets behavior for unknown types, either fail at compile- or run-time */
+    unknownTypes?: "compileError" | "runtimeError";
+    skip?: TestMethodList;
+    only?: TestMethodList;
+    /** extra setup script to be executed during initialization */
+    extraSetup?: string;
+    /** list of keywords to rename, if not supplied default will be used */
+    keywords?: string[];
+    /** dont insert into REPOSRC */
+    skipReposrc?: boolean;
+}

package/build/src/types.js

@@ -1,3 +1,3 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
 //# sourceMappingURL=types.js.map

package/build/src/unique_identifier.d.ts

@@ -1,9 +1,9 @@
-export declare class UniqueIdentifier {
-    private static counter;
-    private static indexBackup;
-    static reset(): void;
-    static get(): string;
-    static resetIndexBackup(): void;
-    static getIndexBackup(): string;
-    static getIndexBackup1(): string;
-}
+export declare class UniqueIdentifier {
+    private static counter;
+    private static indexBackup;
+    static reset(): void;
+    static get(): string;
+    static resetIndexBackup(): void;
+    static getIndexBackup(): string;
+    static getIndexBackup1(): string;
+}

package/build/src/unique_identifier.js

@@ -1,28 +1,28 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.UniqueIdentifier = void 0;
-class UniqueIdentifier {
-    static reset() {
-        this.counter = 0;
-    }
-    static get() {
-        // as part of the validation, it is ensured that no identifiers exists
-        // in the input source with ^unique\d+$
-        this.counter++;
-        return "unique" + this.counter;
-    }
-    static resetIndexBackup() {
-        this.indexBackup = 0;
-    }
-    static getIndexBackup() {
-        this.indexBackup++;
-        return "indexBackup" + this.indexBackup;
-    }
-    static getIndexBackup1() {
-        return "indexBackup1";
-    }
-}
-exports.UniqueIdentifier = UniqueIdentifier;
-UniqueIdentifier.counter = 0;
-UniqueIdentifier.indexBackup = 0;
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.UniqueIdentifier = void 0;
+class UniqueIdentifier {
+    static reset() {
+        this.counter = 0;
+    }
+    static get() {
+        // as part of the validation, it is ensured that no identifiers exists
+        // in the input source with ^unique\d+$
+        this.counter++;
+        return "unique" + this.counter;
+    }
+    static resetIndexBackup() {
+        this.indexBackup = 0;
+    }
+    static getIndexBackup() {
+        this.indexBackup++;
+        return "indexBackup" + this.indexBackup;
+    }
+    static getIndexBackup1() {
+        return "indexBackup1";
+    }
+}
+UniqueIdentifier.counter = 0;
+UniqueIdentifier.indexBackup = 0;
+exports.UniqueIdentifier = UniqueIdentifier;
 //# sourceMappingURL=unique_identifier.js.map

package/build/src/unit_test.d.ts

@@ -1,15 +1,15 @@
-import * as abaplint from "@abaplint/core";
-import { DatabaseSetupResult } from "./db/database_setup_result";
-export declare type TestMethodList = {
-    object: string;
-    class: string;
-    method: string;
-}[];
-export declare class UnitTest {
-    initializationScript(reg: abaplint.IRegistry, dbSetup: DatabaseSetupResult, extraSetup?: string, useImport?: boolean): string;
-    private escapeNamespace;
-    unitTestScriptOpen(reg: abaplint.IRegistry, _skip?: TestMethodList, _only?: TestMethodList): string;
-    unitTestScript(reg: abaplint.IRegistry, skip?: TestMethodList, _only?: TestMethodList): string;
-    private buildImports;
-    private hasClassConstructor;
-}
+import * as abaplint from "@abaplint/core";
+import { DatabaseSetupResult } from "./db/database_setup_result";
+export type TestMethodList = {
+    object: string;
+    class: string;
+    method: string;
+}[];
+export declare class UnitTest {
+    initializationScript(reg: abaplint.IRegistry, dbSetup: DatabaseSetupResult, extraSetup?: string, useImport?: boolean): string;
+    private escapeNamespace;
+    unitTestScriptOpen(reg: abaplint.IRegistry, _skip?: TestMethodList, _only?: TestMethodList): string;
+    unitTestScript(reg: abaplint.IRegistry, skip?: TestMethodList, _only?: TestMethodList): string;
+    private buildImports;
+    private hasClassConstructor;
+}

package/build/src/unit_test.js

@@ -1,50 +1,50 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.UnitTest = void 0;
-/* eslint-disable max-len */
-const abaplint = require("@abaplint/core");
-class UnitTest {
-    // todo, move this method somewhere else, its much more than just unit test relevant
-    initializationScript(reg, dbSetup, extraSetup, useImport) {
-        let ret = "";
-        if (useImport === true) {
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.UnitTest = void 0;
+/* eslint-disable max-len */
+const abaplint = require("@abaplint/core");
+class UnitTest {
+    // todo, move this method somewhere else, its much more than just unit test relevant
+    initializationScript(reg, dbSetup, extraSetup, useImport) {
+        let ret = "";
+        if (useImport === true) {
             ret = `/* eslint-disable import/newline-after-import */
-import "./_top.mjs";\n`;
-        }
-        else {
+import "./_top.mjs";\n`;
+        }
+        else {
             ret = `/* eslint-disable import/newline-after-import */
 import runtime from "@abaplint/runtime";
-globalThis.abap = new runtime.ABAP();\n`;
-        }
+globalThis.abap = new runtime.ABAP();\n`;
+        }
         ret += `${this.buildImports(reg, useImport)}
 
-export async function initializeABAP() {\n`;
-        ret += `  const sqlite = [];\n`;
-        for (const i of dbSetup.schemas.sqlite) {
-            ret += `sqlite.push(\`${i}\`);\n`;
-        }
-        ret += `  const hdb = \`${dbSetup.schemas.hdb}\`;\n`;
-        ret += `  const pg = \`${dbSetup.schemas.pg}\`;\n`;
-        ret += `  const schemas = {sqlite, hdb, pg};\n`;
-        ret += `  const insert = [];\n`;
-        for (const i of dbSetup.insert) {
-            ret += `  insert.push(\`${i}\`);\n`;
-        }
-        if (extraSetup === undefined) {
-            ret += `// no setup logic specified in config\n`;
-        }
-        else {
-            ret += `  const {setup} = await import("${extraSetup}");\n` +
-                `  await setup(globalThis.abap, schemas, insert);\n`;
-        }
-        ret += `}`;
-        return ret;
-    }
-    escapeNamespace(filename) {
-        // ES modules are resolved and cached as URLs. This means that special characters must be percent-encoded, such as # with %23 and ? with %3F.
-        return filename.replace(/\//g, "%23");
-    }
-    unitTestScriptOpen(reg, _skip, _only) {
+export async function initializeABAP() {\n`;
+        ret += `  const sqlite = [];\n`;
+        for (const i of dbSetup.schemas.sqlite) {
+            ret += `sqlite.push(\`${i}\`);\n`;
+        }
+        ret += `  const hdb = \`${dbSetup.schemas.hdb}\`;\n`;
+        ret += `  const pg = \`${dbSetup.schemas.pg}\`;\n`;
+        ret += `  const schemas = {sqlite, hdb, pg};\n`;
+        ret += `  const insert = [];\n`;
+        for (const i of dbSetup.insert) {
+            ret += `  insert.push(\`${i}\`);\n`;
+        }
+        if (extraSetup === undefined) {
+            ret += `// no setup logic specified in config\n`;
+        }
+        else {
+            ret += `  const {setup} = await import("${extraSetup}");\n` +
+                `  await setup(globalThis.abap, schemas, insert);\n`;
+        }
+        ret += `}`;
+        return ret;
+    }
+    escapeNamespace(filename) {
+        // ES modules are resolved and cached as URLs. This means that special characters must be percent-encoded, such as # with %23 and ? with %3F.
+        return filename.replace(/\//g, "%23");
+    }
+    unitTestScriptOpen(reg, _skip, _only) {
         let ret = `/* eslint-disable curly */
 import fs from "fs";
 import path from "path";
@@ -58,33 +58,33 @@ async function run() {
   let lt_input = new abap.types.Table(new abap.types.Structure({class_name: new abap.types.Character(30), testclass_name: new abap.types.Character(30), method_name: new abap.types.Character(30)}), {"withHeader":false,"type":"STANDARD","isUnique":false,"keyFields":[]});
   let ls_input = new abap.types.Structure({class_name: new abap.types.Character(30), testclass_name: new abap.types.Character(30), method_name: new abap.types.Character(30)});
   let ls_result = new abap.types.Structure({list: new abap.types.Table(new abap.types.Structure({class_name: new abap.types.Character(30), testclass_name: new abap.types.Character(30), method_name: new abap.types.Character(30), expected: new abap.types.String(), actual: new abap.types.String(), status: new abap.types.String(), runtime: new abap.types.Integer(), message: new abap.types.String(), js_location: new abap.types.String(), console: new abap.types.String()}), {"withHeader":false,"type":"STANDARD","isUnique":false,"keyFields":[]}), json: new abap.types.String()});
-`;
-        for (const obj of reg.getObjects()) {
-            if (reg.isDependency(obj) || !(obj instanceof abaplint.Objects.Class)) {
-                continue;
-            }
-            const hasTestFile = obj.getFiles().some(f => { return f.getFilename().includes(".testclasses."); });
-            if (hasTestFile === true) {
-                ret += `  await import("./${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}.testclasses.mjs");\n`;
-            }
-            for (const file of obj.getABAPFiles()) {
-                for (const def of file.getInfo().listClassDefinitions()) {
-                    if (def.isForTesting === false || def.isGlobal === true || def.methods.length === 0) {
-                        // todo, fix, there might be global test methods
-                        continue;
-                    }
-                    for (const m of def.methods) {
-                        if (m.isForTesting === false) {
-                            continue;
-                        }
+`;
+        for (const obj of reg.getObjects()) {
+            if (reg.isDependency(obj) || !(obj instanceof abaplint.Objects.Class)) {
+                continue;
+            }
+            const hasTestFile = obj.getFiles().some(f => { return f.getFilename().includes(".testclasses."); });
+            if (hasTestFile === true) {
+                ret += `  await import("./${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}.testclasses.mjs");\n`;
+            }
+            for (const file of obj.getABAPFiles()) {
+                for (const def of file.getInfo().listClassDefinitions()) {
+                    if (def.isForTesting === false || def.isGlobal === true || def.methods.length === 0) {
+                        // todo, fix, there might be global test methods
+                        continue;
+                    }
+                    for (const m of def.methods) {
+                        if (m.isForTesting === false) {
+                            continue;
+                        }
                         ret += `  ls_input.get().class_name.set("${obj.getName()}");
   ls_input.get().testclass_name.set("${def.name.toUpperCase()}");
   ls_input.get().method_name.set("${m.name.toUpperCase()}");
-  abap.statements.append({source: ls_input, target: lt_input});\n`;
-                    }
-                }
-            }
-        }
+  abap.statements.append({source: ls_input, target: lt_input});\n`;
+                    }
+                }
+            }
+        }
         ret += `
 
   ls_result.set(await abap.Classes["KERNEL_UNIT_RUNNER"].run({it_input: lt_input}));
@@ -96,10 +96,10 @@ run().then(() => {
 }).catch((err) => {
   console.log(err);
   process.exit(1);
-});`;
-        return ret;
-    }
-    unitTestScript(reg, skip, _only) {
+});`;
+        return ret;
+    }
+    unitTestScript(reg, skip, _only) {
         let ret = `/* eslint-disable curly */
 import fs from "fs";
 import path from "path";
@@ -115,55 +115,55 @@ async function run() {
   let clas;
   let locl;
   let meth;
-  try {\n`;
-        for (const obj of reg.getObjects()) {
-            if (reg.isDependency(obj) || !(obj instanceof abaplint.Objects.Class)) {
-                continue;
-            }
-            ret += `// --------------------------------------------\n`;
-            ret += `    clas = unit.addObject("${obj.getName()}");\n`;
-            for (const file of obj.getABAPFiles()) {
-                for (const def of file.getInfo().listClassDefinitions()) {
-                    if (def.isForTesting === false
-                        || def.isGlobal === true
-                        || def.methods.length === 0
-                        || def.isAbstract === true) {
-                        // todo, fix, there might be global test methods
-                        continue;
-                    }
-                    const hasTestFile = obj.getFiles().some(f => { return f.getFilename().includes(".testclasses."); });
-                    if (hasTestFile === false) {
-                        break;
-                    }
+  try {\n`;
+        for (const obj of reg.getObjects()) {
+            if (reg.isDependency(obj) || !(obj instanceof abaplint.Objects.Class)) {
+                continue;
+            }
+            ret += `// --------------------------------------------\n`;
+            ret += `    clas = unit.addObject("${obj.getName()}");\n`;
+            for (const file of obj.getABAPFiles()) {
+                for (const def of file.getInfo().listClassDefinitions()) {
+                    if (def.isForTesting === false
+                        || def.isGlobal === true
+                        || def.methods.length === 0
+                        || def.isAbstract === true) {
+                        // todo, fix, there might be global test methods
+                        continue;
+                    }
+                    const hasTestFile = obj.getFiles().some(f => { return f.getFilename().includes(".testclasses."); });
+                    if (hasTestFile === false) {
+                        break;
+                    }
                     ret += `    {
       const {${def.name}} = await import("./${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}.testclasses.mjs");
       locl = clas.addTestClass("${def.name}");
-      if (${def.name}.class_setup) await ${def.name}.class_setup();\n`;
-                    for (const m of def.methods) {
-                        if (m.isForTesting === false) {
-                            continue;
-                        }
-                        const skipThis = (skip || []).some(a => a.object === obj.getName() && a.class === def.name && a.method === m.name);
-                        if (skipThis) {
-                            ret += `  console.log('${obj.getName()}: running ${def.name}->${m.name}, skipped');\n`;
-                            ret += `  meth = locl.addMethod("${m.name}");\n`;
-                            ret += `  meth.skip();\n`;
-                            continue;
-                        }
-                        ret += `      {\n        const test = await (new ${def.name}()).constructor_();\n`;
-                        ret += `        if (test.setup) await test.setup();\n`;
-                        ret += `        console.log("${obj.getName()}: running ${def.name}->${m.name}");\n`;
-                        ret += `        meth = locl.addMethod("${m.name}");\n`;
-                        ret += `        await test.${m.name}();\n`;
-                        ret += `        meth.pass();\n`;
-                        ret += `        if (test.teardown) await test.teardown();\n`;
-                        ret += `      }\n`;
-                    }
-                    ret += `      if (${def.name}.class_teardown) await ${def.name}.class_teardown();\n`;
-                    ret += `    }\n`;
-                }
-            }
-        }
+      if (${def.name}.class_setup) await ${def.name}.class_setup();\n`;
+                    for (const m of def.methods) {
+                        if (m.isForTesting === false) {
+                            continue;
+                        }
+                        const skipThis = (skip || []).some(a => a.object === obj.getName() && a.class === def.name && a.method === m.name);
+                        if (skipThis) {
+                            ret += `  console.log('${obj.getName()}: running ${def.name}->${m.name}, skipped');\n`;
+                            ret += `  meth = locl.addMethod("${m.name}");\n`;
+                            ret += `  meth.skip();\n`;
+                            continue;
+                        }
+                        ret += `      {\n        const test = await (new ${def.name}()).constructor_();\n`;
+                        ret += `        if (test.setup) await test.setup();\n`;
+                        ret += `        console.log("${obj.getName()}: running ${def.name}->${m.name}");\n`;
+                        ret += `        meth = locl.addMethod("${m.name}");\n`;
+                        ret += `        await test.${m.name}();\n`;
+                        ret += `        meth.pass();\n`;
+                        ret += `        if (test.teardown) await test.teardown();\n`;
+                        ret += `      }\n`;
+                    }
+                    ret += `      if (${def.name}.class_teardown) await ${def.name}.class_teardown();\n`;
+                    ret += `    }\n`;
+                }
+            }
+        }
         ret += `// -------------------END-------------------
     console.log(abap.console.get());
     fs.writeFileSync(__dirname + path.sep + "_output.xml", unit.xUnitXML());
@@ -182,71 +182,71 @@ run().then(() => {
 }).catch((err) => {
   console.log(err);
   process.exit(1);
-});`;
-        return ret;
-    }
-    buildImports(reg, useImport) {
-        // note: ES modules are hoised, so use the dynamic import(), due to setting of globalThis.abap
-        // some sorting required: eg. a class constructor using constant from interface
-        const list = [];
-        const late = [];
-        const imp = (filename) => {
-            if (useImport === true) {
-                return `import "./${filename}.mjs";`;
-            }
-            else {
-                return `await import("./${filename}.mjs");`;
-            }
-        };
-        for (const obj of reg.getObjects()) {
-            if (obj instanceof abaplint.Objects.Table
-                || obj instanceof abaplint.Objects.DataElement
-                || obj instanceof abaplint.Objects.LockObject
-                || obj instanceof abaplint.Objects.MIMEObject
-                || obj instanceof abaplint.Objects.WebMIME
-                || obj instanceof abaplint.Objects.TypePool
-                || obj instanceof abaplint.Objects.TableType) {
-                list.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}`));
-            }
-        }
-        for (const obj of reg.getObjects()) {
-            if (obj instanceof abaplint.Objects.FunctionGroup) {
-                for (const m of obj.getModules()) {
-                    list.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.fugr.${this.escapeNamespace(m.getName().toLowerCase())}`));
-                }
-            }
-            else if (obj instanceof abaplint.Objects.Class) {
-                if (obj.getName().toUpperCase() !== "CL_ABAP_CHAR_UTILITIES"
-                    && this.hasClassConstructor(reg, obj)) {
-                    // this will not solve all problems with class constors 100%, but probably good enough
-                    late.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}`));
-                }
-                else {
-                    list.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}`));
-                }
-            }
-            else if (obj instanceof abaplint.Objects.Interface) {
-                list.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}`));
-            }
-        }
-        return [...list.sort(), ...late].join("\n");
-    }
-    // class constructors might make early use of eg. constants from interfaces
-    // sub classes will import() super classes and trigger a class constructor of the super
-    hasClassConstructor(reg, clas) {
-        var _a, _b;
-        if (((_a = clas.getDefinition()) === null || _a === void 0 ? void 0 : _a.getMethodDefinitions().getByName("CLASS_CONSTRUCTOR")) !== undefined) {
-            return true;
-        }
-        const sup = (_b = clas.getDefinition()) === null || _b === void 0 ? void 0 : _b.getSuperClass();
-        if (sup !== undefined) {
-            const superClass = reg.getObject("CLAS", sup);
-            if (superClass) {
-                return this.hasClassConstructor(reg, superClass);
-            }
-        }
-        return false;
-    }
-}
-exports.UnitTest = UnitTest;
+});`;
+        return ret;
+    }
+    buildImports(reg, useImport) {
+        // note: ES modules are hoised, so use the dynamic import(), due to setting of globalThis.abap
+        // some sorting required: eg. a class constructor using constant from interface
+        const list = [];
+        const late = [];
+        const imp = (filename) => {
+            if (useImport === true) {
+                return `import "./${filename}.mjs";`;
+            }
+            else {
+                return `await import("./${filename}.mjs");`;
+            }
+        };
+        for (const obj of reg.getObjects()) {
+            if (obj instanceof abaplint.Objects.Table
+                || obj instanceof abaplint.Objects.DataElement
+                || obj instanceof abaplint.Objects.LockObject
+                || obj instanceof abaplint.Objects.MIMEObject
+                || obj instanceof abaplint.Objects.WebMIME
+                || obj instanceof abaplint.Objects.TypePool
+                || obj instanceof abaplint.Objects.TableType) {
+                list.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}`));
+            }
+        }
+        for (const obj of reg.getObjects()) {
+            if (obj instanceof abaplint.Objects.FunctionGroup) {
+                for (const m of obj.getModules()) {
+                    list.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.fugr.${this.escapeNamespace(m.getName().toLowerCase())}`));
+                }
+            }
+            else if (obj instanceof abaplint.Objects.Class) {
+                if (obj.getName().toUpperCase() !== "CL_ABAP_CHAR_UTILITIES"
+                    && this.hasClassConstructor(reg, obj)) {
+                    // this will not solve all problems with class constors 100%, but probably good enough
+                    late.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}`));
+                }
+                else {
+                    list.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}`));
+                }
+            }
+            else if (obj instanceof abaplint.Objects.Interface) {
+                list.push(imp(`${this.escapeNamespace(obj.getName().toLowerCase())}.${obj.getType().toLowerCase()}`));
+            }
+        }
+        return [...list.sort(), ...late].join("\n");
+    }
+    // class constructors might make early use of eg. constants from interfaces
+    // sub classes will import() super classes and trigger a class constructor of the super
+    hasClassConstructor(reg, clas) {
+        var _a, _b;
+        if (((_a = clas.getDefinition()) === null || _a === void 0 ? void 0 : _a.getMethodDefinitions().getByName("CLASS_CONSTRUCTOR")) !== undefined) {
+            return true;
+        }
+        const sup = (_b = clas.getDefinition()) === null || _b === void 0 ? void 0 : _b.getSuperClass();
+        if (sup !== undefined) {
+            const superClass = reg.getObject("CLAS", sup);
+            if (superClass) {
+                return this.hasClassConstructor(reg, superClass);
+            }
+        }
+        return false;
+    }
+}
+exports.UnitTest = UnitTest;
 //# sourceMappingURL=unit_test.js.map

package/build/src/validation.d.ts

@@ -1,8 +1,8 @@
-import { Issue, IRegistry, IConfig } from "@abaplint/core";
-import { ITranspilerOptions } from "./types";
-export declare const config: IConfig;
-export declare class Validation {
-    private readonly options;
-    constructor(options?: ITranspilerOptions);
-    run(reg: IRegistry): readonly Issue[];
-}
+import { Issue, IRegistry, IConfig } from "@abaplint/core";
+import { ITranspilerOptions } from "./types";
+export declare const config: IConfig;
+export declare class Validation {
+    private readonly options;
+    constructor(options?: ITranspilerOptions);
+    run(reg: IRegistry): readonly Issue[];
+}