fix(posts): Pull results out of the cache in descending order.

randytarampi · Aug 19, 2018 · 48ab8e2 · 48ab8e2
1 parent 8cd22de
commit 48ab8e2
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 12 deletions.
diff --git a/packages/posts/lib/searchParams.js b/packages/posts/lib/searchParams.js
@@ -61,7 +61,9 @@ class SearchParams extends Record({
     }
 
     get Dynamoose() {
-        const options = {};
+        const options = {
+            descending: true
+        };
 
         if (this.perPage) {
             options.limit = this.perPage;

diff --git a/packages/posts/test/unit/lib/cachedDataSource.js b/packages/posts/test/unit/lib/cachedDataSource.js
@@ -128,7 +128,8 @@ describe("CachedDataSource", function () {
                         hash: {type: {eq: "Photo"}},
                         options: {
                             indexName: "type-source-index",
-                            limit: 100
+                            limit: 100,
+                            descending: true
                         },
                         range: {source: {eq: stubType}}
                     });
@@ -291,7 +292,7 @@ describe("CachedDataSource", function () {
                     expect(stubGetPost.calledOnce).to.eql(true);
                     sinon.assert.calledWith(stubGetPost, {
                         uid: {eq: stubPost.uid},
-                        options: {limit: 100}
+                        options: {limit: 100, descending: true}
                     });
                 });
         });
@@ -307,7 +308,7 @@ describe("CachedDataSource", function () {
                     expect(stubGetPost.calledOnce).to.eql(true);
                     sinon.assert.calledWith(stubGetPost, {
                         uid: {eq: stubPost.uid},
-                        options: {limit: 100}
+                        options: {limit: 100, descending: true}
                     });
                 });
         });

diff --git a/packages/posts/test/unit/lib/searchParams.js b/packages/posts/test/unit/lib/searchParams.js
@@ -99,7 +99,7 @@ describe("SearchParams", function () {
 
             expect(searchParams.Dynamoose).to.eql({
                 uid: {eq: "woof"},
-                options: {limit: 100}
+                options: {limit: 100, descending: true}
             });
         });
 
@@ -108,7 +108,7 @@ describe("SearchParams", function () {
 
             expect(searchParams.Dynamoose).to.eql({
                 type: {eq: "woof"},
-                options: {limit: 100}
+                options: {limit: 100, descending: true}
             });
         });
 
@@ -117,7 +117,7 @@ describe("SearchParams", function () {
 
             expect(searchParams.Dynamoose).to.eql({
                 source: {eq: "meow"},
-                options: {limit: 100}
+                options: {limit: 100, descending: true}
             });
         });
 
@@ -127,7 +127,7 @@ describe("SearchParams", function () {
             expect(searchParams.Dynamoose).to.eql({
                 hash: {type: {eq: "woof"}},
                 range: {source: {eq: "meow"}},
-                options: {indexName: "type-source-index", limit: 100}
+                options: {indexName: "type-source-index", limit: 100, descending: true}
             });
         });
 
@@ -142,7 +142,7 @@ describe("SearchParams", function () {
             expect(searchParams.Dynamoose).to.eql({
                 hash: {type: {eq: "woof"}},
                 range: {meow: {lt: "grr"}},
-                options: {indexName: "type-meow-index", limit: 100}
+                options: {indexName: "type-meow-index", limit: 100, descending: true}
             });
         });
 
@@ -151,7 +151,7 @@ describe("SearchParams", function () {
 
             expect(searchParams.Dynamoose).to.eql({
                 uid: {eq: `meow${util.compositeKeySeparator}woof`},
-                options: {limit: 100}
+                options: {limit: 100, descending: true}
             });
         });
 
@@ -166,7 +166,7 @@ describe("SearchParams", function () {
             expect(searchParams.Dynamoose).to.eql({
                 hash: {source: {eq: "woof"}},
                 range: {meow: {gt: "grr"}},
-                options: {indexName: "source-meow-index", limit: 100}
+                options: {indexName: "source-meow-index", limit: 100, descending: true}
             });
         });
 
@@ -184,7 +184,8 @@ describe("SearchParams", function () {
                 range: {meow: {gt: "grr"}},
                 options: {
                     indexName: "source-meow-index",
-                    limit: 20
+                    limit: 20,
+                    descending: true
                 }
             });
         });