Set links.next to null when last page is passed

lib/utils/createPagePath-test.js

@@ -22,6 +22,16 @@ test('should return path for next page and replace existing page qs', (t) => {
   t.is(ret, expected)
 })
 
+test('should return null when no next page', (t) => {
+  const paging = {next: null}
+  const request = {path: '/pages'}
+  const expected = null
+
+  const ret = createPagePath(paging, request)
+
+  t.is(ret, expected)
+})
+
 test('should return path with baseUri', (t) => {
   const paging = {next: {type: 'page', pageAfter: 'page19', pageSize: 20}}
   const request = {path: '/pages'}

lib/utils/createPagePath.js

@@ -1,6 +1,10 @@
 const querystring = require('querystring')
 
 function createPagePath (paging, {path}, baseUri = '') {
+  if (!paging.next) {
+    return null
+  }
+
   if (baseUri.endsWith('/')) {
     baseUri = baseUri.substr(0, baseUri.length - 1)
   }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "integreat-api-json",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Integreat json api",
   "author": "Kjell-Morten Bratsberg Thorsen <[email protected]> (http://kjellmorten.no)",
   "license": "ISC",
@@ -53,6 +53,6 @@
     "standard": "^11.0.1"
   },
   "dependencies": {
-    "jsonwebtoken": "^8.2.0"
+    "jsonwebtoken": "^8.2.1"
   }
 }

tests/helpers/adapters.js

@@ -46,6 +46,14 @@ const send = async ({params, action, data, relationship}) => {
             next: {type: 'page', pageSize: 2, pageAfter: 'page4'}
           }
         }
+      } else if (params.pageAfter === 'page4') {
+        return {
+          status: 'ok',
+          data: setDateAttrs([]),
+          paging: {
+            next: null
+          }
+        }
       }
     } else if (params.authCode === '12345') {
       return {

tests/paging-test.js

@@ -22,8 +22,8 @@ const defs = {
 
 const great = integreat(defs, {adapters})
 
-const firstPage = 'eyJ0eXBlIjoicGFnZSIsInBhZ2VTaXplIjoyLCJwYWdlQWZ0ZXIiOiJwYWdlMiJ9'
-const secondPage = 'eyJ0eXBlIjoicGFnZSIsInBhZ2VTaXplIjoyLCJwYWdlQWZ0ZXIiOiJwYWdlNCJ9'
+const secondPage = 'eyJ0eXBlIjoicGFnZSIsInBhZ2VTaXplIjoyLCJwYWdlQWZ0ZXIiOiJwYWdlMiJ9'
+const thirdPage = 'eyJ0eXBlIjoicGFnZSIsInBhZ2VTaXplIjoyLCJwYWdlQWZ0ZXIiOiJwYWdlNCJ9'
 
 // Tests
 
@@ -55,7 +55,7 @@ test('should GET first page', async (t) => {
       first: null,
       last: null,
       prev: null,
-      next: `https://api.somesite.com/pages?page=${firstPage}`
+      next: `https://api.somesite.com/pages?page=${secondPage}`
     }
   }
 
@@ -72,8 +72,8 @@ test('should GET first page', async (t) => {
 test('should GET second page', async (t) => {
   const request = {
     method: 'GET',
-    path: `/pages?page=${firstPage}`,
-    params: {page: firstPage}
+    path: `/pages?page=${secondPage}`,
+    params: {page: secondPage}
   }
   const expected = {
     data: [
@@ -94,7 +94,33 @@ test('should GET second page', async (t) => {
       first: null,
       last: null,
       prev: null,
-      next: `/pages?page=${secondPage}`
+      next: `/pages?page=${thirdPage}`
+    }
+  }
+
+  const routes = jsonapi(great)
+  const route = findRoute(routes, {path: '/pages', method: 'GET'})
+  const response = await route.handler(request)
+
+  t.truthy(response)
+  t.is(response.statusCode, 200, response.statusMessage)
+  t.truthy(response.body)
+  t.deepEqual(response.body, expected)
+})
+
+test('should GET empty result after last page', async (t) => {
+  const request = {
+    method: 'GET',
+    path: `/pages?page=${thirdPage}`,
+    params: {page: thirdPage}
+  }
+  const expected = {
+    data: [],
+    links: {
+      first: null,
+      last: null,
+      prev: null,
+      next: null
     }
   }