@contrail/flexplm 1.8.0-alpha.562c8dd → 1.8.0-alpha.95a3ea2

package/README.md

@@ -5,15 +5,10 @@ Core model for contrail, which establishes the base peristence services (without
 
 | Item | Default |
 |---|---|
-| VibeIQ type path | `custom-entity:country` (case-insensitive) |
+| VibeIQ type path | `custom-entity:country` (exact, lowercase; matched case-sensitively in both directions) |
 | FlexPLM class / type path | `LCSCountry` / `Country` |
 | Identifier property | `vibeIQIdentifier` (same key in both systems; no rekey runs without a mapping) |
 | Informational property | `name` |
 
-A mapping section for `LCSCountry` overrides any of these. The mapping section must also declare which system creates Countries, with the existing creation gates (there is no default; if it is not declared, the app treats Country as read-only):
+A mapping section for `LCSCountry` overrides these defaults when selected by the type-conversion routing. If your mapping defines `typeConversion.vibe2flex['custom-entity'].getMapKey`, make it return `'LCSCountry'` for Country entities to apply that section's outbound overrides; an empty key bypasses the section.
 
-```ts
-LCSCountry: { isInboundCreatable: () => true,  isOutboundCreatable: () => false }, // FlexPLM creates
-LCSCountry: { isInboundCreatable: () => false, isOutboundCreatable: () => true  }, // VibeIQ creates
-```
-Read the declared origin with `TypeConversionUtils.getOriginSystem(...)`.

package/lib/interfaces/mapping-file.d.ts

@@ -340,13 +340,6 @@ export interface MappingSection {
      *    update existing entities.
      *  - Conditional — inspect the row and allow / disallow per record.
      *
-     * On the **`LCSCountry`** section this pair declares which system creates
-     * Countries: `isInboundCreatable: () => true` together with
-     * `isOutboundCreatable: () => false` means FlexPLM creates them (the
-     * reverse pair means VibeIQ creates them). Both functions must be present
-     * and opposite; otherwise the origin is undeclared and the connector treats
-     * Country as read-only. See `TypeConversionUtils.getOriginSystem`.
-     *
      * On the **season-link** sections (`LCSProductSeasonLink` /
      * `LCSSKUSeasonLink`), `() => true` enables the add-to-project
      * flow: FlexPLM-driven inbound events can add an item to a Vibe
@@ -369,6 +362,10 @@ export interface MappingSection {
      * }
      *
      * @example
+     * // FlexPLM creates Countries (VibeIQ creates them by default).
+     * LCSCountry: { isInboundCreatable: () => true }
+     *
+     * @example
      * // add-to-project flow — Flex drives project-item creation.
      * // Paired with isOutboundCreatable: () => false to make
      * // outbound update-only.
@@ -714,7 +711,7 @@ interface MappingFileBase {
  * | `item`               | `LCSProduct` / `LCSSKU`          | Syncs once `lifecycleStage` is out of `concept`. Sends Primary Viewable as the thumbnail (no other images). Creates `LCSProduct` / `LCSSKU` if missing; auto-retries `LCSSKU` creation when the parent `LCSProduct` isn't ready yet. |
  * | `color`              | `LCSColor`                       | Syncs on create / update. Sends Primary Viewable as the thumbnail. Creates `LCSColor` if missing.          |
  * | `custom-entity`      | `LCSRevisableEntity`             | Syncs on create / update. Creates `LCSRevisableEntity` if missing. Workflows should gate on the custom-entity subtype so only the intended subtypes publish. |
- * | `custom-entity` (`custom-entity:country`) | `LCSCountry` | Built-in defaults for type path (`Country`), identifier (`vibeIQIdentifier`, same key in both systems) and name; the mapping section overrides each default and **must** declare the origin. Which system creates Countries is declared in the `LCSCountry` mapping section with the `isInboundCreatable` / `isOutboundCreatable` pair: `(true, false)` = FlexPLM creates, `(false, true)` = VibeIQ creates; anything else is undeclared. See `TypeConversionUtils.getOriginSystem`. |
+ * | `custom-entity` (`custom-entity:country`) | `LCSCountry` | Built-in defaults for type path (`Country`), identifier (`vibeIQIdentifier`, same key in both systems) and name; the mapping section overrides each default. Syncs on create / update. Creates `LCSCountry` if missing. Workflows should gate on the custom-entity subtype so only the intended subtypes publish. |
  * | `project-item`       | `LCSProductSeasonLink` / `LCSSKUSeasonLink` | Syncs on plan publish. Requires the assortment to have `publishToFlexPLM=true` and `flexPLMSeasonName` set. Adds the Product / SKU to the `LCSSeason` if not already linked. Only project-items changed since the last publish are sent (configurable to reach further back). Processing on the FlexPLM side is async via a Windchill Queue. |
  * | `item:material`      | `LCSMaterial`                    | Supported via custom mapping section + `typeConversion`. Route `item:material` typePath to a mapping section that declares `getClass: () => 'LCSMaterial'` and `getSoftType: () => 'Material'`. See {@link TypeConversion} for the `LCSMaterial.processAsItem` config flag that pairs with this. |
  * | `custom-entity`      | `LCSLast` / `LCSLifecycleManaged` | **Not yet supported.**                                                                                     |
@@ -732,7 +729,7 @@ interface MappingFileBase {
  * | `LCSLifecycleManaged`   | `custom-entity`            | Configurable to create new custom-entities; updates existing.                                              |
  * | `LCSMaterial`           | `custom-entity` (default) or `item:material` | Default: custom-entity. To route to `item:material` instead, set `{ "LCSMaterial": { "processAsItem": true } }` in the connector app config and write a mapping section with `getClass: () => 'item'` + `getSoftType: () => 'item:material'`. Wire it via `flex2vibe.LCSMaterial.getMapKey` returning that section's name. |
  * | `LCSRevisableEntity`    | `custom-entity`            | Configurable to create new custom-entities; updates existing.                                              |
- * | `LCSCountry`            | `custom-entity` (`custom-entity:country`) | Built-in defaults for type path and identifier; the mapping section must declare the origin. Identifier `vibeIQIdentifier` (same key in both systems). Creation is allowed only when the `LCSCountry` section declares FlexPLM as the origin: `isInboundCreatable: () => true` and `isOutboundCreatable: () => false`. |
+ * | `LCSCountry`            | `custom-entity` (`custom-entity:country`) | Built-in defaults for type path and identifier (`vibeIQIdentifier`, same key in both systems). Updates existing Countries; configurable to create new ones with `isInboundCreatable` (for example `LCSCountry: { isInboundCreatable: () => true }`). |
  * | (any)                   | `item` (direct, no link)   | **Not yet supported.**                                                                                     |
  *
  * @example

package/lib/util/type-conversion-utils.d.ts

@@ -1,6 +1,4 @@
 import { MapFileUtil } from '@contrail/transform-data';
-/** The system that originates (creates) an entity type: 'flexplm' or 'vibeiq'. */
-export type OriginSystem = 'flexplm' | 'vibeiq';
 /** This class is helper functions to get the data for converting
  * VibeIQ entities to / from FlexPLM Objects
 */
@@ -149,23 +147,6 @@ export declare class TypeConversionUtils {
     static getMapKeyFromObject(fileId: any, mapFileUtil: MapFileUtil, object: any, direction: string): Promise<string | undefined>;
     static isInboundCreatableFromObject(fileId: string, mapFileUtil: MapFileUtil, object: any, context?: any): Promise<boolean>;
     static isOutboundCreatableFromEntity(fileId: string, mapFileUtil: MapFileUtil, entity: any, context?: any): Promise<boolean>;
-    /** Returns which system originates (creates) an entity type, declared in its mapping section
-     * with the isInboundCreatable / isOutboundCreatable pair:
-     *  - isInboundCreatable true  and isOutboundCreatable false -> 'flexplm' (FlexPLM creates)
-     *  - isInboundCreatable false and isOutboundCreatable true  -> 'vibeiq'  (VibeIQ creates)
-     * Anything else is undeclared and returns undefined: no mapping file, no section, either function
-     * missing, both true, both false, or an error while resolving. Unlike isInboundCreatableFromObject /
-     * isOutboundCreatableFromEntity, there are NO defaults: the origin must be stated explicitly,
-     * so callers can fail safe (treat the entity as read-only) when it is not.
-     *
-     * @param fileId id for mapFile
-     * @param mapFileUtil class to get mapfile
-     * @param entityOrObject VibeIQ entity (vibe2flex) or FlexPLM object (flex2vibe)
-     * @param direction VIBE2FLEX_DIRECTION or FLEX2VIBE_DIRECTION; selects how the map section key is resolved
-     * @param context optional context object passed to the mapping functions
-     * @returns Promise<'flexplm' | 'vibeiq' | undefined>
-     */
-    static getOriginSystem(fileId: string, mapFileUtil: MapFileUtil, entityOrObject: any, direction: string, context?: any): Promise<OriginSystem | undefined>;
     /** Takes in a FlexPLM object and determines whether inbound
      * images should be synced. In most cases, the creation owning system
      * will also control image syncing. Defaults to false if no mapping exists.

package/lib/util/type-conversion-utils.js

@@ -143,11 +143,6 @@ class TypeConversionUtils {
             const mappingData = await mapFileUtil.getMappingSection(transformMapFile, 'typeConversion', direction);
             if (mappingData && mappingData[type] && mappingData[type]['getMapKey']) {
                 const mapKey = await mappingData[type]['getMapKey'](entity);
-                // An org's custom-entity routing usually returns '' for the subtypes it does not list.
-                // Country has a built-in section key, so its section is found without a typeConversion entry.
-                if (!mapKey && type_defaults_1.TypeDefaults.isCountryEntity(entity)) {
-                    return type_defaults_1.TypeDefaults.COUNTRY_OBJECT_CLASS;
-                }
                 return mapKey;
             }
         }
@@ -365,51 +360,6 @@ class TypeConversionUtils {
         }
         return isOutboundCreatable;
     }
-    /** Returns which system originates (creates) an entity type, declared in its mapping section
-     * with the isInboundCreatable / isOutboundCreatable pair:
-     *  - isInboundCreatable true  and isOutboundCreatable false -> 'flexplm' (FlexPLM creates)
-     *  - isInboundCreatable false and isOutboundCreatable true  -> 'vibeiq'  (VibeIQ creates)
-     * Anything else is undeclared and returns undefined: no mapping file, no section, either function
-     * missing, both true, both false, or an error while resolving. Unlike isInboundCreatableFromObject /
-     * isOutboundCreatableFromEntity, there are NO defaults: the origin must be stated explicitly,
-     * so callers can fail safe (treat the entity as read-only) when it is not.
-     *
-     * @param fileId id for mapFile
-     * @param mapFileUtil class to get mapfile
-     * @param entityOrObject VibeIQ entity (vibe2flex) or FlexPLM object (flex2vibe)
-     * @param direction VIBE2FLEX_DIRECTION or FLEX2VIBE_DIRECTION; selects how the map section key is resolved
-     * @param context optional context object passed to the mapping functions
-     * @returns Promise<'flexplm' | 'vibeiq' | undefined>
-     */
-    static async getOriginSystem(fileId, mapFileUtil, entityOrObject, direction, context) {
-        if (!fileId || !entityOrObject) {
-            return undefined;
-        }
-        try {
-            const mapKey = (TypeConversionUtils.VIBE2FLEX_DIRECTION === direction)
-                ? await this.getMapKey(fileId, mapFileUtil, entityOrObject, direction)
-                : await this.getMapKeyFromObject(fileId, mapFileUtil, entityOrObject, direction);
-            if (!mapKey) {
-                return undefined;
-            }
-            const mapData = await map_utils_1.MapUtil.getFullMapSection(fileId, mapFileUtil, mapKey);
-            if (!mapData || typeof mapData['isInboundCreatable'] !== 'function' || typeof mapData['isOutboundCreatable'] !== 'function') {
-                return undefined;
-            }
-            const inbound = !!(await mapData['isInboundCreatable'](entityOrObject, context));
-            const outbound = !!(await mapData['isOutboundCreatable'](entityOrObject, context));
-            if (inbound && !outbound) {
-                return 'flexplm';
-            }
-            if (!inbound && outbound) {
-                return 'vibeiq';
-            }
-        }
-        catch (e) {
-            console.error('TypeConversionUtils.getOriginSystem: unable to read the origin from the mapping file: ' + e?.message);
-        }
-        return undefined;
-    }
     /** Takes in a FlexPLM object and determines whether inbound
      * images should be synced. In most cases, the creation owning system
      * will also control image syncing. Defaults to false if no mapping exists.

package/lib/util/type-conversion-utils.spec.js

@@ -963,87 +963,22 @@ describe('conversion-utils', () => {
             const entity = { entityType: 'custom-entity', typePath: 'custom-entity:country' };
             expect(await type_conversion_utils_1.TypeConversionUtils.getIdentifierProperties(TRANSFORM_MAP_FILE, mapFileUtil, entity)).toEqual(['isoCode']);
         });
-    });
-    describe('getOriginSystem (isInboundCreatable / isOutboundCreatable pair)', () => {
-        const mapFileUtil = new transform_data_1.MapFileUtil(new sdk_1.Entities());
-        const countryEntity = { entityType: 'custom-entity', typePath: 'custom-entity:country' };
-        const countryObject = { flexPLMObjectClass: 'LCSCountry', flexPLMTypePath: 'Country' };
-        let spy;
-        // Org-style routing: custom-entity subtypes that are not listed return ''
-        const typeConversion = {
-            vibe2flex: { 'custom-entity': { getMapKey: () => '' } },
-            flex2vibe: {},
-        };
-        function useMap(countrySection) {
-            const map = { typeConversion };
-            if (countrySection) {
-                map.LCSCountry = countrySection;
-            }
-            spy.mockImplementation(async () => map);
-        }
-        beforeEach(() => {
-            spy = jest.spyOn(mapFileUtil, 'getMapFile');
-        });
-        afterEach(() => {
-            spy.mockRestore();
-        });
-        it('should return flexplm when inbound is creatable and outbound is not, in both directions', async () => {
-            useMap({ isInboundCreatable: () => true, isOutboundCreatable: () => false });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryEntity, type_conversion_utils_1.TypeConversionUtils.VIBE2FLEX_DIRECTION)).toBe('flexplm');
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryObject, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBe('flexplm');
-        });
-        it('should return vibeiq when outbound is creatable and inbound is not, in both directions', async () => {
-            useMap({ isInboundCreatable: () => false, isOutboundCreatable: () => true });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryEntity, type_conversion_utils_1.TypeConversionUtils.VIBE2FLEX_DIRECTION)).toBe('vibeiq');
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryObject, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBe('vibeiq');
-        });
-        it('should support async mapping functions', async () => {
-            useMap({ isInboundCreatable: async () => true, isOutboundCreatable: async () => false });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryObject, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBe('flexplm');
+        it('should leave the section key of a Country unchanged when the org custom entity routing returns an empty key', async () => {
+            const country = { entityType: 'custom-entity', typePath: 'custom-entity:country' };
+            expect(await type_conversion_utils_1.TypeConversionUtils.getMapKey(TRANSFORM_MAP_FILE, mapFileUtil, country, type_conversion_utils_1.TypeConversionUtils.VIBE2FLEX_DIRECTION)).toEqual('');
         });
-        it('should return undefined when both gates are true', async () => {
-            useMap({ isInboundCreatable: () => true, isOutboundCreatable: () => true });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryObject, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBeUndefined();
-        });
-        it('should return undefined when both gates are false', async () => {
-            useMap({ isInboundCreatable: () => false, isOutboundCreatable: () => false });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryEntity, type_conversion_utils_1.TypeConversionUtils.VIBE2FLEX_DIRECTION)).toBeUndefined();
-        });
-        it('should return undefined when either gate function is missing', async () => {
-            useMap({ isInboundCreatable: () => true });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryObject, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBeUndefined();
-            useMap({ isOutboundCreatable: () => true });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryEntity, type_conversion_utils_1.TypeConversionUtils.VIBE2FLEX_DIRECTION)).toBeUndefined();
-        });
-        it('should return undefined when there is no LCSCountry section', async () => {
-            useMap();
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryEntity, type_conversion_utils_1.TypeConversionUtils.VIBE2FLEX_DIRECTION)).toBeUndefined();
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryObject, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBeUndefined();
-        });
-        it('should return undefined when there is no mapping file id or no entity', async () => {
-            useMap({ isInboundCreatable: () => true, isOutboundCreatable: () => false });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem('', mapFileUtil, countryObject, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBeUndefined();
-            expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, undefined, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBeUndefined();
-        });
-        it('should return undefined and log when a mapping function throws', async () => {
-            useMap({ isInboundCreatable: () => { throw new Error('boom'); }, isOutboundCreatable: () => false });
-            const errorSpy = jest.spyOn(console, 'error').mockImplementation(() => undefined);
-            try {
-                expect(await type_conversion_utils_1.TypeConversionUtils.getOriginSystem(TRANSFORM_MAP_FILE, mapFileUtil, countryObject, type_conversion_utils_1.TypeConversionUtils.FLEX2VIBE_DIRECTION)).toBeUndefined();
-                expect(errorSpy).toHaveBeenCalled();
-            }
-            finally {
-                errorSpy.mockRestore();
-            }
+        it('should not allow inbound creation of a Country when the mapping has no isInboundCreatable gate', async () => {
+            const object = { flexPLMObjectClass: 'LCSCountry', flexPLMTypePath: 'Country' };
+            expect(await type_conversion_utils_1.TypeConversionUtils.isInboundCreatableFromObject(TRANSFORM_MAP_FILE, mapFileUtil, object)).toBe(false);
         });
-        it('should not change the section key of other custom entity subtypes', async () => {
-            useMap({ isInboundCreatable: () => true, isOutboundCreatable: () => false });
-            const other = { entityType: 'custom-entity', typePath: 'custom-entity:exchangeRate' };
-            expect(await type_conversion_utils_1.TypeConversionUtils.getMapKey(TRANSFORM_MAP_FILE, mapFileUtil, other, type_conversion_utils_1.TypeConversionUtils.VIBE2FLEX_DIRECTION)).toEqual('');
+        it('should allow inbound creation of a Country when the LCSCountry section sets isInboundCreatable', async () => {
+            spy.mockImplementation(async () => ({ ...mapWithoutCountry, LCSCountry: { isInboundCreatable: () => true } }));
+            const object = { flexPLMObjectClass: 'LCSCountry', flexPLMTypePath: 'Country' };
+            expect(await type_conversion_utils_1.TypeConversionUtils.isInboundCreatableFromObject(TRANSFORM_MAP_FILE, mapFileUtil, object)).toBe(true);
         });
-        it('should return the LCSCountry section key when the org custom entity routing returns an empty key', async () => {
-            useMap({ isInboundCreatable: () => true, isOutboundCreatable: () => false });
-            expect(await type_conversion_utils_1.TypeConversionUtils.getMapKey(TRANSFORM_MAP_FILE, mapFileUtil, countryEntity, type_conversion_utils_1.TypeConversionUtils.VIBE2FLEX_DIRECTION)).toEqual('LCSCountry');
+        it('should allow outbound creation of a Country by default', async () => {
+            const entity = { entityType: 'custom-entity', typePath: 'custom-entity:country' };
+            expect(await type_conversion_utils_1.TypeConversionUtils.isOutboundCreatableFromEntity(TRANSFORM_MAP_FILE, mapFileUtil, entity)).toBe(true);
         });
     });
 });

package/lib/util/type-defaults.d.ts

@@ -20,7 +20,7 @@ export declare class TypeDefaults {
     static applyConfig(config: any): void;
     static isPropertyTrue(value: any): boolean;
     /** True when the VibeIQ entity is a Country: a custom-entity whose typePath is
-     *  'custom-entity:country' (compared case-insensitively).
+     *  exactly 'custom-entity:country' (compared case-sensitively, as on inbound).
      *
      * @param entity VibeIQ entity
      * @returns boolean

package/lib/util/type-defaults.js

@@ -19,14 +19,14 @@ class TypeDefaults {
         return value === true || (typeof value === 'string' && value.toLowerCase() === 'true');
     }
     /** True when the VibeIQ entity is a Country: a custom-entity whose typePath is
-     *  'custom-entity:country' (compared case-insensitively).
+     *  exactly 'custom-entity:country' (compared case-sensitively, as on inbound).
      *
      * @param entity VibeIQ entity
      * @returns boolean
      */
     static isCountryEntity(entity) {
         const typePath = entity?.['typePath'];
-        return typeof typePath === 'string' && typePath.toLowerCase() === TypeDefaults.COUNTRY_VIBE_TYPE_PATH;
+        return typeof typePath === 'string' && typePath === TypeDefaults.COUNTRY_VIBE_TYPE_PATH;
     }
     /**Takes in full entity and returs the default FlexPLM
      * object class.

package/lib/util/type-defaults.spec.js

@@ -50,12 +50,12 @@ describe('Type Defaults', () => {
             const objectClass = type_defaults_1.TypeDefaults.getDefaultObjectClass(newData);
             expect(objectClass).toEqual('LCSCountry');
         });
-        it('should return LCSCountry when the Country typePath uses different casing', () => {
+        it('should return LCSRevisableEntity when the Country typePath uses different casing', () => {
             const newData = {
                 entityType: 'custom-entity',
                 typePath: 'custom-entity:Country'
             };
-            expect(type_defaults_1.TypeDefaults.getDefaultObjectClass(newData)).toEqual('LCSCountry');
+            expect(type_defaults_1.TypeDefaults.getDefaultObjectClass(newData)).toEqual('LCSRevisableEntity');
         });
         it('should return LCSRevisableEntity for a custom entity that is not a Country', () => {
             const newData = {
@@ -347,6 +347,9 @@ describe('Type Defaults', () => {
         it('should return true for the custom-entity:country type path', () => {
             expect(type_defaults_1.TypeDefaults.isCountryEntity({ typePath: 'custom-entity:country' })).toBe(true);
         });
+        it('should return false when the typePath differs only in casing', () => {
+            expect(type_defaults_1.TypeDefaults.isCountryEntity({ typePath: 'custom-entity:Country' })).toBe(false);
+        });
         it('should return false for another subtype, a missing typePath, or an undefined entity', () => {
             expect(type_defaults_1.TypeDefaults.isCountryEntity({ typePath: 'custom-entity:other' })).toBe(false);
             expect(type_defaults_1.TypeDefaults.isCountryEntity({})).toBe(false);

package/lib/util/type-utils.js

@@ -15,6 +15,11 @@ class TypeUtils {
             id,
             relations: ['typeProperties', 'typeInterfaces']
         });
+        // The API answers 200 with an empty body for a missing/deleted type, which the SDK returns as null.
+        if (!type) {
+            throw new Error('getTypeById: type not found for typeId: ' + id
+                + '. The entity references a type that no longer exists.');
+        }
         return type;
     }
     async getByRootAndPath(options) {

package/lib/util/type-utils.spec.js

@@ -215,3 +215,16 @@ describe('filterProperties-color', () => {
         expect(filteredProps.length).toEqual(len);
     });
 });
+describe('TypeUtils.getTypeById() tests', () => {
+    it('should throw an error containing the missing type id when the type does not exist', async () => {
+        const typeUtils = new type_utils_1.TypeUtils();
+        typeUtils['typesObj'] = { getType: jest.fn().mockResolvedValue(null) };
+        await expect(typeUtils.getTypeById('missingTypeId')).rejects.toThrow('type not found for typeId: missingTypeId');
+    });
+    it('should return the same type object when the type exists', async () => {
+        const typeUtils = new type_utils_1.TypeUtils();
+        const type = { id: 'typeId', typeProperties: [] };
+        typeUtils['typesObj'] = { getType: jest.fn().mockResolvedValue(type) };
+        expect(await typeUtils.getTypeById('typeId')).toBe(type);
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/flexplm",
-  "version": "1.8.0-alpha.562c8dd",
+  "version": "1.8.0-alpha.95a3ea2",
   "description": "Library used for integration with flexplm.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",