feat: throw exception if schema id is not a string and report error, c…

…loses #380

lib/ajv.js

@@ -136,7 +136,8 @@ function addSchema(schema, key, _skipValidation, _meta) {
     for (var i=0; i<schema.length; i++) this.addSchema(schema[i], undefined, _skipValidation, _meta);
     return;
   }
-  // can key/id have # inside?
+  if (schema.id !== undefined && typeof schema.id != 'string')
+    throw new Error('schema id must be string');
   key = resolve.normalizeId(key || schema.id);
   checkUnique(this, key);
   this._schemas[key] = this._addSchema(schema, _skipValidation, _meta, true);

spec/ajv.spec.js

@@ -208,6 +208,15 @@ describe('Ajv', function () {
     it('should throw if schema is not an object', function() {
       should.throw(function() { ajv.addSchema('foo'); });
     });
+
+    it('should throw if schema id is not a string', function() {
+      try {
+        ajv.addSchema({ id: 1, type: 'integer' });
+        throw new Error('should have throw exception');
+      } catch(e) {
+        e.message .should.equal('schema id must be string');
+      }
+    });
   });