feat(find_many): adds posibility to include additional parameters in …

…populate request
platanus · Sep 24, 2014 · bf05802 · bf05802
1 parent dcfd37b
commit bf05802
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/plugins/find-many.js b/src/plugins/find-many.js
@@ -55,12 +55,12 @@ angular.module('restmod').factory('restmod.FindMany', ['restmod', 'RMPackerCache
      * @param {array} _records Records to resolve.
      * @return {Resource} Resource holding the populate promise.
      */
-    .define('Scope.$populate', function(_records) {
+    .define('Scope.$populate', function(_records, _params) {
 
       // Extract record pks for non resolved records and build a record map
       var pks = [],
           recordMap = {},
-          params = {},
+          params = _params || {},
           model = this.$type,
           dummy = model.dummy(true),
           record, request;

diff --git a/test/plugins/find-many-spec.js b/test/plugins/find-many-spec.js
@@ -27,6 +27,14 @@ describe('Plugin: Find Many plugin', function() {
       expect(bikes[1].brand).toEqual('Yetti');
     });
 
+    it('should include additional parameters in request', function() {
+      var bikes = [ Bike.$new(1) ];
+
+      Bike.$populate(bikes, { include: 'parts' });
+      $httpBackend.expectGET('/api/bikes?ids=1&include=parts').respond(200, []);
+      $httpBackend.flush();
+    });
+
     it('should not repeat ids in request, but should populate repeated records separatedly', function() {
       var bikes = [
         Bike.$new(1),