change: additionalProperties are set to false instead of removing it when dealing with pattern properties

Author: Mika Kunnas

index.js

@@ -167,7 +167,7 @@ function patternPropertiesHandler(schema) {
 		type = patternsObj[pattern].type;
 
 		if ((type === additProps.type) && type !== 'object') {
-			delete schema.additionalProperties;
+			schema.additionalProperties = false;
 			break;
 		}
 	}

test/pattern_properties.js

@@ -27,14 +27,15 @@ test('handling additional properties of the same type: string', function(assert)
 	expected = {
 		$schema: 'http://json-schema.org/draft-04/schema#',
 		type: 'object',
+		additionalProperties: false,
 		patternProperties: {
 			'^[a-z]*$': {
 				type: 'string'
 			}
 		}
 	};
 
-	assert.deepEqual(result, expected, 'additionalProperties deleted');
+	assert.deepEqual(result, expected, 'additionalProperties set to false');
 });
 
 test('handling additional properties of the same type: number', function(assert) {
@@ -62,14 +63,15 @@ test('handling additional properties of the same type: number', function(assert)
 	expected = {
 		$schema: 'http://json-schema.org/draft-04/schema#',
 		type: 'object',
+		additionalProperties: false,
 		patternProperties: {
 			'^[a-z]*$': {
 				type: 'number'
 			}
 		}
 	};
 
-	assert.deepEqual(result, expected, 'additionalProperties deleted');
+	assert.deepEqual(result, expected, 'additionalProperties set to false');
 });
 
 test('handling additional properties with one of patternProperty types', function(assert) {
@@ -100,6 +102,7 @@ test('handling additional properties with one of patternProperty types', functio
 	expected = {
 		$schema: 'http://json-schema.org/draft-04/schema#',
 		type: 'object',
+		additionalProperties: false,
 		patternProperties: {
 			'^[a-z]*$': {
 				type: 'string'
@@ -110,7 +113,7 @@ test('handling additional properties with one of patternProperty types', functio
 		}
 	};
 
-	assert.deepEqual(result, expected, 'additionalProperties deleted');
+	assert.deepEqual(result, expected, 'additionalProperties set to false');
 });
 
 test('keeping additionalProperties with object type', function(assert) {
@@ -154,7 +157,7 @@ test('keeping additionalProperties with object type', function(assert) {
 		}
 	};
 
-	assert.deepEqual(result, expected, 'additionalProperties kept');
+	assert.deepEqual(result, expected, 'additionalProperties kept unchanged');
 });
 
 test('not supporting patternProperties', function(assert) {