use Array.forEach instead of Array.map where applicable for better performance

ahmadnassri · ahmadnassri · commit 1320389a065d · 2015-04-11T18:53:16.000-04:00
diff --git a/test/requests.js b/test/requests.js
@@ -48,7 +48,7 @@ async.each(fixtures.cli, function (cli) {
 
             // make an exception for multipart/form-data
             if (fixtures.requests[request].headers) {
-              fixtures.requests[request].headers.map(function (header, index) {
+              fixtures.requests[request].headers.forEach(function (header, index) {
                 if (header.name === 'content-type' && header.value === 'multipart/form-data') {
                   delete fixtures.requests[request].headers[index]
                 }
diff --git a/test/targets.js b/test/targets.js
@@ -66,7 +66,7 @@ var itShouldGenerateOutput = function (request, path, target, client) {
 describe('Available Targets', function () {
   var targets = HTTPSnippet.availableTargets()
 
-  targets.map(function (target) {
+  targets.forEach(function (target) {
     it('available-targets.json should include ' + target.title, function () {
       fixtures['available-targets'].should.containEql(target)
     })

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ async.each(fixtures.cli, function (cli) {`
`48`	`48`
`49`	`49`	`// make an exception for multipart/form-data`
`50`	`50`	`if (fixtures.requests[request].headers) {`
`51`		`- fixtures.requests[request].headers.map(function (header, index) {`
	`51`	`+ fixtures.requests[request].headers.forEach(function (header, index) {`
`52`	`52`	`if (header.name === 'content-type' && header.value === 'multipart/form-data') {`
`53`	`53`	`delete fixtures.requests[request].headers[index]`
`54`	`54`	`}`