@contrail/flexplm 1.7.1-alpha.a7e0d3c → 1.7.1-alpha.e5225c7

package/lib/entity-processor/base-entity-processor.js

@@ -184,6 +184,9 @@ class BaseEntityProcessor {
         }
         const entityReference = identityEntity.entityReference;
         const firstColonIndex = entityReference.indexOf(':');
+        if (firstColonIndex === -1 || firstColonIndex === 0 || firstColonIndex === entityReference.length - 1) {
+            throw new Error('Invalid entityReference: ' + entityReference);
+        }
         const entityName = entityReference.slice(0, firstColonIndex);
         const id = entityReference.slice(firstColonIndex + 1);
         const entity = await this.entities.get({

package/lib/entity-processor/base-entity-processor.spec.js

@@ -617,5 +617,32 @@ describe('BaseEntityProcessor', () => {
                 id: 'e5g8ugusNWOuCvdw:nju12Jw2jGFvcQSf'
             });
         });
+        it('should throw error when entityReference has no colon', async () => {
+            mockEntitiesGet.mockResolvedValue([{ entityReference: 'item1' }]);
+            await expect(btep.getEntityUsingIdentityService({
+                poolKey: 'item',
+                propertyName: 'itemNumber',
+                propertyValue: '12345'
+            })).rejects.toThrow('Invalid entityReference: item1');
+            expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
+        });
+        it('should throw error when entityReference starts with a colon (missing entityName)', async () => {
+            mockEntitiesGet.mockResolvedValue([{ entityReference: ':1' }]);
+            await expect(btep.getEntityUsingIdentityService({
+                poolKey: 'item',
+                propertyName: 'itemNumber',
+                propertyValue: '12345'
+            })).rejects.toThrow('Invalid entityReference: :1');
+            expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
+        });
+        it('should throw error when entityReference ends with a colon (missing id)', async () => {
+            mockEntitiesGet.mockResolvedValue([{ entityReference: 'item:' }]);
+            await expect(btep.getEntityUsingIdentityService({
+                poolKey: 'item',
+                propertyName: 'itemNumber',
+                propertyValue: '12345'
+            })).rejects.toThrow('Invalid entityReference: item:');
+            expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
+        });
     });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/flexplm",
-  "version": "1.7.1-alpha.a7e0d3c",
+  "version": "1.7.1-alpha.e5225c7",
   "description": "Library used for integration with flexplm.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",