make traverse safe for delete operations (#640)

Author: shlomiassaf

src/lib/models/reflections/container.ts

@@ -46,7 +46,7 @@ export class ContainerReflection extends Reflection {
      */
     traverse(callback: TraverseCallback) {
         if (this.children) {
-            this.children.forEach((child: DeclarationReflection) => {
+            this.children.slice().forEach((child: DeclarationReflection) => {
                 callback(child, TraverseProperty.Children);
             });
         }

src/lib/models/reflections/declaration.ts

@@ -153,15 +153,15 @@ export class DeclarationReflection extends ContainerReflection implements Defaul
      */
     traverse(callback: TraverseCallback) {
         if (this.typeParameters) {
-            this.typeParameters.forEach((parameter) => callback(parameter, TraverseProperty.TypeParameter));
+            this.typeParameters.slice().forEach((parameter) => callback(parameter, TraverseProperty.TypeParameter));
         }
 
         if (this.type instanceof ReflectionType) {
             callback((<ReflectionType> this.type).declaration, TraverseProperty.TypeLiteral);
         }
 
         if (this.signatures) {
-            this.signatures.forEach((signature) => callback(signature, TraverseProperty.Signatures));
+            this.signatures.slice().forEach((signature) => callback(signature, TraverseProperty.Signatures));
         }
 
         if (this.indexSignature) {

src/lib/models/reflections/signature.ts

@@ -58,11 +58,11 @@ export class SignatureReflection extends Reflection implements TypeContainer, Ty
         }
 
         if (this.typeParameters) {
-            this.typeParameters.forEach((parameter) => callback(parameter, TraverseProperty.TypeParameter));
+            this.typeParameters.slice().forEach((parameter) => callback(parameter, TraverseProperty.TypeParameter));
         }
 
         if (this.parameters) {
-            this.parameters.forEach((parameter) => callback(parameter, TraverseProperty.Parameters));
+            this.parameters.slice().forEach((parameter) => callback(parameter, TraverseProperty.Parameters));
         }
 
         super.traverse(callback);