fixed updateMany bug on sql models

Author: iolufemi

TODO.md

@@ -3,6 +3,7 @@
 |:------|:------:|:------
 | app.js | 61 | Write a complete Documentation for this project
 | config/index.js | 12 | Test for production and development senarios
+| models/Testthrees.js | 95 | Test transactions
 | models/index.js | 14 | Automatically generate tests with the schema structure
 | routes/index.js | 260 | Test API versioning
 | routes/index.js | 261 | Test rate limiting

services/queue/jobs.js

@@ -32,7 +32,7 @@ jobs.updateRequestLog = function(response, done){
         delete response.requestId;
     }
 
-    models.RequestLogs.updateMany({RequestId: requestId},response)
+    models.RequestLogs.update({RequestId: requestId},response)
     .then(function(res){
         return done(false, res);
     })
@@ -92,12 +92,12 @@ split = _.flattenDeep(split);
 var task;
 if(model){
     if(isSQL){
-        task = models[model].updateMany({ tags: split.join(', ')}, {where: dataClone} );
+        task = models[model].update({ tags: split.join(', ')}, {where: dataClone} );
     }else{
         if(update){
-            task = models[model].updateMany(dataClone,{ updatedAt: new Date(Date.now()).toISOString(), tags: split});
+            task = models[model].update(dataClone,{ updatedAt: new Date(Date.now()).toISOString(), tags: split});
         }else{
-            task = models[model].updateMany(dataClone,{ tags: split});
+            task = models[model].update(dataClone,{ tags: split});
         }
     }
 

services/request/index.js

@@ -54,7 +54,7 @@ module.exports = function(service, requestId, uri, method, data, headers){
             return request(options);
         })
         .then(function(resp){
-            return [Model.updateMany({RequestId: requestId, service: service}, {response: resp, responseStatusCode: 200, updatedAt: Date.now()}), resp];
+            return [Model.update({RequestId: requestId, service: service}, {response: resp, responseStatusCode: 200, updatedAt: Date.now()}), resp];
         })
         .spread(function(update, resp){
             return resolve(resp);
@@ -65,7 +65,7 @@ module.exports = function(service, requestId, uri, method, data, headers){
             }else{
                 var updateddd;
                 if(existss){
-                    updateddd = Model.updateMany({RequestId: requestId, service: service}, {response: (err.response && err.response.body) ? err.response.body : { type: 'internal error', message: err.message}, responseStatusCode: err.statusCode ? err.statusCode : 500, updatedAt: Date.now()});
+                    updateddd = Model.update({RequestId: requestId, service: service}, {response: (err.response && err.response.body) ? err.response.body : { type: 'internal error', message: err.message}, responseStatusCode: err.statusCode ? err.statusCode : 500, updatedAt: Date.now()});
                 }else{
                     options.RequestId = requestId;
                     options.service = service;

template/controller_sql.tmpl

@@ -139,8 +139,8 @@ var <%= service %>sController = {};
             _query.limit = limit;
         }
         _query.where = query;
-        var totalResult = <%= service %>s.estimatedDocumentCount(_query);
-        var total = <%= service %>s.estimatedDocumentCount({});
+        var totalResult = <%= service %>s.count(_query);
+        var total = <%= service %>s.count({});
 
         if(sort){
             _query.order = [];
@@ -294,7 +294,7 @@ var <%= service %>sController = {};
     }
 
     _query.where = query;
-    <%= service %>s.updateMany(data, _query)
+    <%= service %>s.update(data, _query)
     .then(function(resp){
         res.ok(resp);
     })
@@ -311,7 +311,7 @@ var <%= service %>sController = {};
         delete data.secure;
     }
     _query.where = <%= service %>s.where = {_id:id};
-    <%= service %>s.updateMany(data,_query)
+    <%= service %>s.update(data,_query)
     .then(function(resp){
         if(!resp){
             next();

template/controller_sql_test.tmpl

@@ -313,7 +313,7 @@ it('should update documents', function(done){
     var req = {};
     req.query = {name: 'Femi'};
     req.body = {name: 'Femi Updated'};
-    <%= object %>s.updateMany(req, res, next);
+    <%= object %>s.update(req, res, next);
 });
 it('should update a document', function(done){
     var next = function(err){

template/controller_test.tmpl

@@ -305,7 +305,7 @@ it('should update documents', function(done){
     var req = {};
     req.query = {name: 'Femi'};
     req.body = {name: 'Femi Updated'};
-    <%= object %>s.updateMany(req, res, next);
+    <%= object %>s.update(req, res, next);
 });
 it('should update a document', function(done){
     var next = function(err){

template/model_sql_test.tmpl

@@ -64,7 +64,7 @@ describe('<%= service %> Model',function(){
 
         it('should update data', function(done){
             var cb = sinon.spy();
-            var my<%= object %> = <%= service %>.updateMany({name: 'Olaoluwa'}, { where: {name: 'femi'}});
+            var my<%= object %> = <%= service %>.update({name: 'Olaoluwa'}, { where: {name: 'femi'}});
 
             my<%= object %>.then(function(res){
                 cb();
@@ -122,7 +122,7 @@ describe('<%= service %> Model',function(){
             var my<%= object %> = <%= service %>.create({name: 'i am a demigod!'});
             my<%= object %>.then(function(res){
                 id2 = res._id;
-                return <%= service %>.updateMany({name: 'This is the titan'}, {where: {_id: id}});
+                return <%= service %>.update({name: 'This is the titan'}, {where: {_id: id}});
             })
             .then(function(res){
                 return <%= service %>.findOne({where: {name: 'This is the titan'}});
@@ -162,7 +162,7 @@ describe('<%= service %> Model',function(){
         });
 
         it('should count returned records', function(done){
-            var my<%= object %> = <%= service %>.estimatedDocumentCount({where: {name: 'This is the titan'}});
+            var my<%= object %> = <%= service %>.count({where: {name: 'This is the titan'}});
 
             my<%= object %>.then(function(res){
                 res.should.be.a.number; /* jslint ignore:line */