fix: remove all unnecessary items (#10)

* Fixed bug where not all required items were deleted from array and replacd deepequal with faster version

* Added proper tests

* chore: better test

* test: add

* prefer clone deep

Co-authored-by: Ivan Malkov <[email protected]>

Author: XVincentX

.gitignore

@@ -1,2 +1,3 @@
 *.swp
 node_modules
+.idea

index.js

@@ -1,6 +1,6 @@
-var deepEqual = require('deep-equal');
+var deepEqual = require('fast-deep-equal');
 var convert = require('./lib/convert');
-var clone = require('clone');
+var clone = require('lodash.clonedeep');
 
 module.exports = openapiSchemaToJsonSchema;
 module.exports.fromSchema = openapiSchemaToJsonSchema;

lib/converters/schema.js

@@ -232,16 +232,15 @@ function convertPatternProperties (schema, handler) {
 }
 
 function cleanRequired (required, properties) {
-	var i = 0
-
 	required = required || []
 	properties = properties || {}
+	let remainedRequired = []
 
-	for (i; i < required.length; i++) {
-		if (properties[required[i]] === undefined) {
-			required.splice(i, 1)
-		}
+	for (let i = 0; i < required.length; i++) {
+    if (properties[required[i]]) {
+      remainedRequired.push(required[i])
+    }
 	}
 
-	return required
+	return remainedRequired
 }