Stop iteration on Request.eachitem returning false

This echoes the behaviour of eachPage.

Author: James Haggerty

.changes/next-release/feature-Request-fcc76298.json

@@ -0,0 +1,5 @@
+{
+  "type": "feature",
+  "category": "Request",
+  "description": "eachItem method stops iteration on returning false (like eachPage)"
+}

lib/request.js

@@ -513,7 +513,14 @@ AWS.Request = inherit({
       var resultKey = config.resultKey;
       if (Array.isArray(resultKey)) resultKey = resultKey[0];
       var items = jmespath.search(data, resultKey);
-      AWS.util.arrayEach(items, function(item) { callback(null, item); });
+      var continueIteration = true;
+      AWS.util.arrayEach(items, function(item) {
+        continueIteration = callback(null, item);
+        if (continueIteration === false) {
+          return AWS.util.abort;
+        }
+      });
+      return continueIteration;
     }
 
     this.eachPage(wrappedCallback);

test/request.spec.coffee

@@ -192,6 +192,23 @@ describe 'AWS.Request', ->
           ]
           done()
 
+    it 'supports stopping responses if false is returned', ->
+      helpers.mockResponses [
+        {data: {Results: [{Value: 1}, {Value: 2}], NextToken: 'a'}},
+        {data: {Results: [{Value: 3}], NextToken: 'b'}},
+        {data: Results: [{Value: 4}, {Value: 5}]}
+      ]
+      resps = []
+      service.mockMethod().eachItem (err, data) ->
+        if resps.length == 2
+          return false
+        resps.push([err, data])
+
+      expect(resps).to.eql [
+        [null, {Value: 1}],
+        [null, {Value: 2}]
+      ]
+
     it 'passes error to callback', (done) ->
       helpers.mockResponses [
         {data: {Results: [{Value: 1}, {Value: 2}], NextToken: 'a'}},