feat(kitsu): handle nested (relationship) routes using kitsu-core's splitModel

Author: wopian

packages/kitsu/src/delete.spec.js

@@ -46,6 +46,24 @@ describe('kitsu', () => {
       done()
     })
 
+    it('handles nested routes', async done => {
+      expect.assertions(1)
+      const api = new Kitsu({ headers: { Authorization: true } })
+      mock.onDelete('/posts/1/comments/1').reply(config => {
+        expect(JSON.parse(config.data)).toEqual({
+          data: {
+            id: '1',
+            type: 'comments'
+          }
+        })
+        return [ 200 ]
+      })
+      api.delete('posts/1/comments', 1).catch(err => {
+        done.fail(err)
+      })
+      done()
+    })
+
     it('throws an error if ID is missing', async () => {
       expect.assertions(1)
       const api = new Kitsu()

packages/kitsu/src/index.js

@@ -1,6 +1,6 @@
 import axios from 'axios'
 import pluralise from 'pluralize'
-import { camel, deserialise, error, kebab, query, serialise, snake } from 'kitsu-core'
+import { camel, deserialise, error, kebab, query, serialise, snake, splitModel } from 'kitsu-core'
 
 /**
  * Creates a new `kitsu` instance
@@ -218,13 +218,16 @@ export default class Kitsu {
    */
   async patch (model, body, headers = {}) {
     try {
-      const serialData = serialise(model, body, 'PATCH', {
+      const [ resourceModel, url ] = splitModel(model, {
+        resourceCase: this.resCase,
+        pluralModel: this.plural
+      })
+      const serialData = serialise(resourceModel, body, 'PATCH', {
         camelCaseTypes: this.camel,
         pluralTypes: this.plural
       })
-      const url = this.plural(this.resCase(model)) + '/' + body.id
       const { data } = await this.axios.patch(
-        url,
+        `${url}/${body.id}`,
         serialData,
         { headers: Object.assign(this.headers, headers) }
       )
@@ -258,10 +261,13 @@ export default class Kitsu {
    */
   async post (model, body, headers = {}) {
     try {
-      const url = this.plural(this.resCase(model))
+      const [ resourceModel, url ] = splitModel(model, {
+        resourceCase: this.resCase,
+        pluralModel: this.plural
+      })
       const { data } = await this.axios.post(
         url,
-        serialise(model, body, 'POST', {
+        serialise(resourceModel, body, 'POST', {
           camelCaseTypes: this.camel,
           pluralTypes: this.plural
         }),
@@ -287,9 +293,12 @@ export default class Kitsu {
    */
   async delete (model, id, headers = {}) {
     try {
-      const url = this.plural(this.resCase(model)) + '/' + id
-      const { data } = await this.axios.delete(url, {
-        data: serialise(model, { id }, 'DELETE', {
+      const [ resourceModel, url ] = splitModel(model, {
+        resourceCase: this.resCase,
+        pluralModel: this.plural
+      })
+      const { data } = await this.axios.delete(`${url}/${id}`, {
+        data: serialise(resourceModel, { id }, 'DELETE', {
           camelCaseTypes: this.camel,
           pluralTypes: this.plural
         }),

packages/kitsu/src/patch.spec.js

@@ -80,5 +80,13 @@ describe('kitsu', () => {
       const request = await api.patch('posts', patchSingle.kitsu)
       expect(request).toEqual({ data: patchSingle.kitsu })
     })
+
+    it('handes nested routes', async () => {
+      expect.assertions(1)
+      const api = new Kitsu({ headers: { Authorization: true } })
+      mock.onPatch(`something/1/relationships/posts/${patchSingle.jsonapi.data.id}`).reply(200, patchSingle.jsonapi)
+      const request = await api.patch('something/1/relationships/posts', patchSingle.kitsu)
+      expect(request).toEqual({ data: patchSingle.kitsu })
+    })
   })
 })

packages/kitsu/src/post.spec.js

@@ -48,6 +48,26 @@ describe('kitsu', () => {
       done()
     })
 
+    it('handles nested routes', async done => {
+      expect.assertions(1)
+      const api = new Kitsu({ headers: { Authorization: true } })
+      mock.onPost('/something/1/relationships/anime').reply(config => {
+        expect(JSON.parse(config.data)).toEqual({
+          data: {
+            type: 'anime',
+            attributes: {
+              name: 'Name'
+            }
+          }
+        })
+        return [ 200 ]
+      })
+      api.post('something/1/relationships/anime', { type: 'anime', name: 'Name' }).catch(err => {
+        done.fail(err)
+      })
+      done()
+    })
+
     it('sends data in request with client-generated ID', async done => {
       expect.assertions(1)
       const api = new Kitsu()