[BUGFIX beta] pass DS.SnapshotRecordArray to urlForFindAll

(cherry picked from commit 3868436)

Author: trevorrjohn

addon/-private/adapters/build-url-mixin.js

@@ -56,7 +56,7 @@ export default Ember.Mixin.create({
       case 'findRecord':
         return this.urlForFindRecord(id, modelName, snapshot);
       case 'findAll':
-        return this.urlForFindAll(modelName);
+        return this.urlForFindAll(modelName, snapshot);
       case 'query':
         return this.urlForQuery(query, modelName);
       case 'queryRecord':
@@ -121,9 +121,10 @@ export default Ember.Mixin.create({
   /**
    * @method urlForFindAll
    * @param {String} modelName
+   * @param {DS.SnapshotRecordArray} snapshot
    * @return {String} url
    */
-  urlForFindAll(modelName) {
+  urlForFindAll(modelName, snapshot) {
     return this._buildURL(modelName);
   },
 

addon/adapters/rest.js

@@ -432,13 +432,9 @@ export default Adapter.extend(BuildURLMixin, {
     @return {Promise} promise
   */
   findAll(store, type, sinceToken, snapshotRecordArray) {
-    const url = this.buildURL(type.modelName, null, null, 'findAll');
+    const url = this.buildURL(type.modelName, null, snapshotRecordArray, 'findAll');
     const query = this.buildQuery(snapshotRecordArray);
 
-    if (sinceToken) {
-      query.since = sinceToken;
-    }
-
     return this.ajax(url, 'GET', { data: query });
   },
 

tests/integration/adapter/rest-adapter-test.js

@@ -1020,8 +1020,11 @@ test("findAll - returning an array populates the array", function(assert) {
 });
 
 
-test("findAll - passes buildURL the requestType", function(assert) {
+test("findAll - passes buildURL the requestType and snapshot", function(assert) {
+  assert.expect(2);
+  let adapterOptionsStub = { stub: true };
   adapter.buildURL = function(type, id, snapshot, requestType) {
+    assert.equal(snapshot.adapterOptions, adapterOptionsStub);
     return "/" + requestType + "/posts";
   };
 
@@ -1032,7 +1035,7 @@ test("findAll - passes buildURL the requestType", function(assert) {
     ]
   });
 
-  store.findAll('post').then(assert.wait(function(posts) {
+  store.findAll('post', { adapterOptions: adapterOptionsStub }).then(assert.wait(function(posts) {
     assert.equal(passedUrl, "/findAll/posts");
   }));
 });

tests/unit/adapters/build-url-mixin/build-url-test.js

@@ -42,13 +42,15 @@ test('buildURL - find requestType delegates to urlForFindRecord', function(asser
 });
 
 test('buildURL - findAll requestType delegates to urlForFindAll', function(assert) {
-  assert.expect(2);
+  assert.expect(3);
   let originalMethod = adapter.urlForFindAll;
-  adapter.urlForFindAll = function(type) {
+  let snapshotStub = { snapshot: true };
+  adapter.urlForFindAll = function(type, snapshot) {
     assert.equal(type, 'super-user');
+    assert.equal(snapshot, snapshotStub);
     return originalMethod.apply(this, arguments);
   };
-  assert.equal(adapter.buildURL('super-user', null, null, 'findAll'), '/superUsers');
+  assert.equal(adapter.buildURL('super-user', null, snapshotStub, 'findAll'), '/superUsers');
 });
 
 test('buildURL - query requestType delegates to urlForQuery', function(assert) {