Improve Resource Validator (#4566)

This will resolve the specific problem raised in

Signed-off-by: Dave Kelsey <[email protected]>
#4562

Author: davek-at-ibm

packages/composer-common/lib/serializer/resourcevalidator.js

@@ -301,7 +301,7 @@ class ResourceValidator {
             case 'Double':
             case 'Long':
             case 'Integer':
-                if(dataType !== 'number') {
+                if(dataType !== 'number' || isNaN(obj)) {
                     invalid = true;
                 }
                 break;
@@ -425,6 +425,9 @@ class ResourceValidator {
                 catch(err) {
                     value = value.toString();
                 }
+            } else if (typeOfValue === 'number' && isNaN(value)) {
+                value = 'Non-numeric';
+                typeOfValue = 'unknown';
             }
         }
 

packages/composer-common/test/serializer/resourcevalidator.js

@@ -470,6 +470,43 @@ describe('ResourceValidator', function () {
             }).should.throw(/Model violation in instance identifier field propName/);
         });
 
+        it('should throw if type of data is not a number for a numeric field', () => {
+            let mockField = sinon.createStubInstance(Field);
+            mockField.getName.returns('propName');
+            mockField.getType.returns('Integer');
+            mockField.isPrimitive.returns(true);
+            (() => {
+                resourceValidator.checkItem('fred', mockField, {rootResourceIdentifier: 'identifier'});
+            }).should.throw(/Model violation in instance identifier field propName.*fred.*string.*Integer/);
+        });
+
+        it('should throw if number is actually NaN', () => {
+            let mockField = sinon.createStubInstance(Field);
+            mockField.getName.returns('propName');
+            mockField.getType.returns('Integer');
+            mockField.isPrimitive.returns(true);
+            (() => {
+                resourceValidator.checkItem(parseInt('fred') /* creates NaN object*/, mockField, {rootResourceIdentifier: 'identifier'});
+            }).should.throw(/Model violation in instance identifier field propName.*Non-numeric.*unknown.*Integer/);
+
+            mockField = sinon.createStubInstance(Field);
+            mockField.getName.returns('propName');
+            mockField.getType.returns('Double');
+            mockField.isPrimitive.returns(true);
+            (() => {
+                resourceValidator.checkItem(parseInt('fred') /* creates NaN object*/, mockField, {rootResourceIdentifier: 'identifier'});
+            }).should.throw(/Model violation in instance identifier field propName.*Non-numeric.*unknown.*Double/);
+
+            mockField = sinon.createStubInstance(Field);
+            mockField.getName.returns('propName');
+            mockField.getType.returns('Long');
+            mockField.isPrimitive.returns(true);
+            (() => {
+                resourceValidator.checkItem(parseInt('fred') /* creates NaN object*/, mockField, {rootResourceIdentifier: 'identifier'});
+            }).should.throw(/Model violation in instance identifier field propName.*Non-numeric.*unknown.*Long/);
+
+        });
+
         it('should throw if class declaration is not found', () => {
             let mockField = sinon.createStubInstance(Field);
             mockField.isPrimitive.returns(false);