Merge pull request #494 from braydonf/frameslimit

Limit query for frames

lib/server/routes/frames.js

@@ -404,7 +404,7 @@ FramesRouter.prototype.destroyFrameById = function(req, res, next) {
 FramesRouter.prototype.getFrames = function(req, res, next) {
   const Frame = this.storage.models.Frame;
 
-  Frame.find({ user: req.user._id }, function(err, frames) {
+  Frame.find({ user: req.user._id }).limit(10).exec(function(err, frames) {
     if (err) {
       return next(new errors.InternalError(err.message));
     }

package.json

@@ -82,7 +82,7 @@
     "storj-service-error-types": "^1.2.0",
     "storj-service-mailer": "^1.0.0",
     "storj-service-middleware": "^1.3.1",
-    "storj-service-storage-models": "^8.22.2",
+    "storj-service-storage-models": "^8.22.3",
     "through": "^2.3.8"
   }
 }

test/server/routes/frames.unit.js

@@ -2005,7 +2005,11 @@ describe('FramesRouter', function() {
       var _frameFind = sinon.stub(
         framesRouter.storage.models.Frame,
         'find'
-      ).callsArgWith(1, new Error('Panic!'));
+      ).returns({
+        limit: sinon.stub().returns({
+          exec: sinon.stub().callsArgWith(0, new Error('Panic!'))
+        })
+      });
       framesRouter.getFrames(request, response, function(err) {
         _frameFind.restore();
         expect(err.message).to.equal('Panic!');
@@ -2032,7 +2036,11 @@ describe('FramesRouter', function() {
       var _frameFind = sinon.stub(
         framesRouter.storage.models.Frame,
         'find'
-      ).callsArgWith(1, null, [frame1, frame2]);
+      ).returns({
+        limit: sinon.stub().returns({
+          exec: sinon.stub().callsArgWith(0, null, [frame1, frame2])
+        })
+      });
       response.on('end', function() {
         _frameFind.restore();
         expect(response._getData()).to.have.lengthOf(2);