implement "sugar" wrapper method for findAll() followed by count()

lib/dao-factory.js

@@ -249,6 +249,48 @@ module.exports = (function() {
     return this.QueryInterface.rawSelect(this.tableName, options, 'count')
   }
 
+  DAOFactory.prototype.findAndCount = function(options) {
+    var self  = this
+      , copts = Utils._.extend({}, options || {})
+
+    // no limit or offset for the options given to count()
+    copts.offset = 0;
+    copts.limit  = 0;
+
+    return new Utils.CustomEventEmitter(function (emitter) {
+      var emit = {
+          err  : function(e) {        // emit error
+            emitter.emit('error', e);
+          }
+        , okay : function(c, r) {     // emit success
+            emitter.emit('success', {
+              count: c || 0,
+              rows : (r && Array.isArray(r) ? r : [])
+            });
+          }
+        , sql  : function(s) {        // emit SQL
+            emitter.emit('sql', s);
+          }
+      };
+
+      self.count(copts)
+          .on('sql', emit.sql)
+          .error(emit.err)
+          .success(function(cnt) {
+            if (cnt === 0)
+              return emit.okay(cnt); // no records, no need for another query
+
+            self.findAll(options)
+                .on('sql', emit.sql)
+                .error(emit.err)
+                .success(function(rows) {
+                   emit.okay(cnt, rows)
+                })
+          });
+
+    }).run();
+  }
+
   DAOFactory.prototype.max = function(field, options) {
     options = Utils._.extend({ attributes: [] }, options || {})
     options.attributes.push(['max(' + field + ')', 'max'])

spec-jasmine/dao-factory.spec.js

@@ -227,6 +227,63 @@ describe('DAOFactory', function() {
         })
       })
 
+      describe('findAndCount', function() {
+        var users = [], fullcount = 3
+
+        beforeEach(function() {
+          Helpers.Factories.User({name: 'user', bio: 'foobar'}, function(_users) {
+            users = _users
+          }, fullcount)
+        })
+
+        it("handles where clause [only]", function() {
+          Helpers.async(function(done) {
+            User.findAndCount({where: "id != " + users[0].id}).success(function(info) {
+              expect(info.count).toEqual(fullcount - 1)
+              expect(Array.isArray(info.rows)).toBeTruthy()
+              expect(info.rows.length).toEqual(fullcount - 1)
+              done()
+            })
+          })
+        })
+
+/*
+// at time of writing (v1.6.0) Sequelize does not seem to support 'offset' on it's own consistently (goes wrong for PostGRES and SQLite)
+        it("handles offset", function() {
+          Helpers.async(function(done) {
+            User.findAndCount({offset: 1}).success(function(info) {
+              expect(info.count).toEqual(fullcount)
+              expect(Array.isArray(info.rows)).toBeTruthy()
+              expect(info.rows.length).toEqual(fullcount - 1)
+              done()
+            })
+          })
+        })
+*/
+        it("handles limit", function() {
+          Helpers.async(function(done) {
+            User.findAndCount({limit: 1})/*.on('sql', console.log)*/.success(function(info) {
+              expect(info.count).toEqual(fullcount)
+              expect(Array.isArray(info.rows)).toBeTruthy()
+              expect(info.rows.length).toEqual(1)
+              done()
+            })
+          })
+        })
+
+        it("handles offset and limit", function() {
+          Helpers.async(function(done) {
+            User.findAndCount({offset: 1, limit: 1}).success(function(info) {
+              expect(info.count).toEqual(fullcount)
+              expect(Array.isArray(info.rows)).toBeTruthy()
+              expect(info.rows.length).toEqual(1)
+              done()
+            })
+          })
+        })
+      })
+
+
       describe('all', function() {
         beforeEach(function() {
           Helpers.Factories.User({name: 'user', bio: 'foobar'}, null, 2)