feat(alias-import): added test case for introspector

packages/concerto-core/changelog.txt

@@ -25,7 +25,7 @@
 #
 
 Version 3.17.1 {ddc91ebd1ff660b421b60302d1e92271} 2024-06-21
-- Added 'enableAliasedType' option to BaseModelManager
+- Added 'importAliasing' option to BaseModelManager
 - Aliased types mapped to FQN in modelfile
 
 Version 3.16.8 {a23d37a4a92071314ff821f879943364} 2024-07-09

packages/concerto-core/lib/basemodelmanager.js

@@ -82,7 +82,7 @@
      * @param {boolean} [options.metamodelValidation] - When true, modelfiles will be validated
      * @param {boolean} [options.addMetamodel] - When true, the Concerto metamodel is added to the model manager
      * @param {boolean} [options.enableMapType] - When true, the Concerto Map Type feature is enabled
-     * @param {boolean} [options.enableAliasedType] - When true, the Concerto Aliasing feature is enabled
+     * @param {boolean} [options.importAliasing] - When true, the Concerto Aliasing feature is enabled
      * @param {*} [processFile] - how to obtain a concerto AST from an input to the model manager
      */
     constructor(options, processFile) {
@@ -98,7 +98,7 @@
         // TODO Remove on release of MapType
         // Supports both env var and property based flag
         this.enableMapType = !!options?.enableMapType;
-        this.enableAliasedType = !!options?.enableAliasedType;
+        this.importAliasing = !!options?.importAliasing;
 
         // Cache a copy of the Metamodel ModelFile for use when validating the structure of ModelFiles later.
         this.metamodelModelFile = new ModelFile(this, MetaModelUtil.metaModelAst, undefined, MetaModelNamespace);
@@ -126,10 +126,10 @@
 
     /**
      * Checks if the import aliasing feature is enabled.
-     * @returns {boolean} true if the enableAliasedType has been set
+     * @returns {boolean} true if the importAliasing has been set
      */
     isAliasedTypeEnabled() {
-        return this.enableAliasedType;
+        return this.importAliasing;
     }
 
     /**
@@ -269,7 +269,7 @@
             if (this.isStrict()) {
                 throw new MetamodelException(err.message);
             } else {
                 console.warn('Invalid metamodel found. This will throw an exception in a future release. ', err.message);
             }
         }
 

packages/concerto-core/test/introspect/introspector.js

@@ -24,6 +24,7 @@ const chai = require('chai');
 chai.use(require('chai-things'));
 const sinon = require('sinon');
 require('chai').should();
+const ParserUtil = require('./parserutility');
 
 describe('Introspector', () => {
 
@@ -80,6 +81,31 @@ describe('Introspector', () => {
             const introspector = new Introspector(modelManager);
             introspector.getClassDeclaration('org.acme.base.Person').should.not.be.null;
         });
+
+        it('should be able to handle the aliased imported types', () => {
+            // create and populate the ModelManager with a model file
+            const modelManager = new ModelManager({ importAliasing: true });
+            Util.addComposerModel(modelManager);
+            modelManager.should.not.be.null;
+
+            const model1 = `
+            namespace org.example.ext
+            asset MyAsset2 identified by assetId {
+                o String assetId
+            }`;
+            const model2 = `
+            namespace org.acme
+            import org.example.ext.{MyAsset2 as m}
+            asset MyAsset identified by assetId {
+                o String assetId
+                o m[] arr
+            }`;
+            let modelFile1 = ParserUtil.newModelFile(modelManager, model1);
+            modelManager.addModelFile(modelFile1);
+            ParserUtil.newModelFile(modelManager, model2);
+            const introspector = new Introspector(modelManager);
+            introspector.getClassDeclaration('org.example.ext.MyAsset2').should.not.be.null;
+        });
     });
 
     describe('#getModelManager', () => {

packages/concerto-core/test/introspect/modelfile.js

@@ -611,7 +611,7 @@ describe('ModelFile', () => {
     describe('#aliasedImport', () => {
 
         beforeEach(()=>{
-            modelManager.enableAliasedType=true;
+            modelManager.importAliasing=true;
         });
         it('should resolve aliased name of import type', () => {
             const model = `