group: add GET,POST,DELETE routes

NicTool · Feb 24, 2024 · af74ff9 · af74ff9
1 parent bb33a81
commit af74ff9
Show file tree

Hide file tree

Showing 5 changed files with 242 additions and 11 deletions.
diff --git a/routes/group.js b/routes/group.js
@@ -0,0 +1,113 @@
+const validate = require('@nictool/validate')
+
+const Group = require('../lib/group')
+const Util = require('../lib/util')
+
+module.exports = (server) => {
+  server.route([
+    {
+      method: 'GET',
+      path: '/group/{id}',
+      options: {
+        response: {
+          schema: validate.group.GET,
+        },
+        tags: ['api'],
+      },
+      handler: async (request, h) => {
+        const groups = await Group.get({
+          deleted: request.query.deleted ?? 0,
+          id: parseInt(request.params.id, 10),
+        })
+        if (groups.length !== 1) {
+          return h
+            .response({
+              meta: {
+                api: Util.meta.api,
+                msg: `No unique group match`,
+              },
+            })
+            .code(204)
+        }
+
+        return h
+          .response({
+            group: groups[0],
+            meta: {
+              api: Util.meta.api,
+              msg: `here's your group`,
+            },
+          })
+          .code(200)
+      },
+    },
+    {
+      method: 'POST',
+      path: '/group',
+      options: {
+        validate: {
+          payload: validate.group.POST,
+        },
+        response: {
+          schema: validate.group.GET,
+        },
+        tags: ['api'],
+      },
+      handler: async (request, h) => {
+        // console.log(request.payload)
+        const gid = await Group.create(request.payload)
+        if (!gid) {
+          console.log(`POST /group oops`) // TODO
+        }
+
+        const groups = await Group.get({ id: gid })
+
+        return h
+          .response({
+            group: groups[0],
+            meta: {
+              api: Util.meta.api,
+              msg: `I created this group`,
+            },
+          })
+          .code(201)
+      },
+    },
+    {
+      method: 'DELETE',
+      path: '/group/{id}',
+      options: {
+        response: {
+          schema: validate.group.GET,
+        },
+        tags: ['api'],
+      },
+      handler: async (request, h) => {
+        const groups = await Group.get(request.params)
+        if (groups.length !== 1) {
+          return h
+            .response({
+              meta: {
+                api: Util.meta.api,
+                msg: `No unique group match`,
+              },
+            })
+            .code(204)
+        }
+
+        await Group.delete({ id: groups[0].id })
+        delete groups[0].gid
+
+        return h
+          .response({
+            group: groups[0],
+            meta: {
+              api: Util.meta.api,
+              msg: `I deleted that group`,
+            },
+          })
+          .code(200)
+      },
+    },
+  ])
+}
diff --git a/routes/group.test.js b/routes/group.test.js
@@ -0,0 +1,119 @@
+const assert = require('node:assert/strict')
+const { describe, it, before, after } = require('node:test')
+
+const { init } = require('./index')
+const Group = require('../lib/Group')
+const groupCase = require('../test/v3/group.json')
+
+before(async () => {
+  const userCase = require('../test/v3/user.json')
+  this.server = await init()
+  const res = await this.server.inject({
+    method: 'POST',
+    url: '/session',
+    payload: {
+      username: `${userCase.username}@example.com`,
+      password: 'Wh@tA-Decent#P6ssw0rd',
+    },
+  })
+  assert.ok(res.headers['set-cookie'][0])
+  this.sessionCookie = res.headers['set-cookie'][0].split(';')[0]
+})
+
+after(async () => {
+  const res = await this.server.inject({
+    method: 'DELETE',
+    url: '/session',
+    headers: {
+      Cookie: this.sessionCookie,
+    },
+  })
+  // console.log(res.result)
+  assert.equal(res.statusCode, 200)
+
+  await this.server.stop()
+})
+
+describe('group', () => {
+  it('GET /group/4096', async () => {
+    const res = await this.server.inject({
+      method: 'GET',
+      url: '/group/4096',
+      headers: {
+        Cookie: this.sessionCookie,
+      },
+    })
+    // console.log(res.result)
+    assert.equal(res.statusCode, 200)
+  })
+
+  it('POST /group', async () => {
+    const testCase = JSON.parse(JSON.stringify(groupCase))
+    testCase.id = 4095 // make it unique
+    testCase.name = `example2.com`
+    delete testCase.deleted
+
+    const res = await this.server.inject({
+      method: 'POST',
+      url: '/group',
+      headers: {
+        Cookie: this.sessionCookie,
+      },
+      payload: testCase,
+    })
+    // console.log(res.result)
+    assert.equal(res.statusCode, 201)
+  })
+
+  it('GET /group', async () => {
+    const res = await this.server.inject({
+      method: 'GET',
+      url: '/group/4095',
+      headers: {
+        Cookie: this.sessionCookie,
+      },
+    })
+    // console.log(res.result)
+    assert.equal(res.statusCode, 200)
+  })
+
+  it('DELETE /group', async () => {
+    const res = await this.server.inject({
+      method: 'DELETE',
+      url: '/group/4095',
+      headers: {
+        Cookie: this.sessionCookie,
+      },
+    })
+    // console.log(res.result)
+    assert.equal(res.statusCode, 200)
+  })
+
+  it('GET /group/4095', async () => {
+    const res = await this.server.inject({
+      method: 'GET',
+      url: '/group/4095',
+      headers: {
+        Cookie: this.sessionCookie,
+      },
+    })
+    // console.log(res.result)
+    assert.equal(res.statusCode, 204)
+  })
+
+  it('GET /group/4095 (deleted)', async () => {
+    const res = await this.server.inject({
+      method: 'GET',
+      url: '/group/4095?deleted=1',
+      headers: {
+        Cookie: this.sessionCookie,
+      },
+    })
+    // console.log(res.result)
+    assert.equal(res.statusCode, 200)
+  })
+
+  it('nukes /group/4095', async () => {
+    assert.ok(Group.destroy({ id: 4095 }))
+  })
+})
diff --git a/routes/index.js b/routes/index.js
@@ -17,9 +17,6 @@ const Config = require('../lib/config')
 const Session = require('../lib/session')
 const User = require('../lib/user')
 
-const SessionRoutes = require('./session')
-const UserRoutes = require('./user')
-
 let server
 
 const setup = async () => {
@@ -78,8 +75,9 @@ const setup = async () => {
     },
   })
 
-  SessionRoutes(server)
-  UserRoutes(server)
+  require('./group')(server)
+  require('./user')(server)
+  require('./session')(server)
 
   server.route({
     method: '*',

diff --git a/routes/user.js b/routes/user.js
@@ -10,7 +10,7 @@ module.exports = (server) => {
       path: '/user',
       options: {
         response: {
-          schema: validate.user.userGET,
+          schema: validate.user.GET,
         },
         // tags: ['api'],
       },
@@ -36,7 +36,7 @@ module.exports = (server) => {
       path: '/user/{id}',
       options: {
         response: {
-          schema: validate.user.userGET,
+          schema: validate.user.GET,
         },
         tags: ['api'],
       },
@@ -79,7 +79,7 @@ module.exports = (server) => {
           payload: validate.user.userPOST,
         },
         response: {
-          schema: validate.user.userGET,
+          schema: validate.user.GET,
         },
         tags: ['api'],
       },
@@ -103,15 +103,15 @@ module.exports = (server) => {
               msg: `I created this user`,
             },
           })
-          .code(200)
+          .code(201)
       },
     },
     {
       method: 'DELETE',
       path: '/user/{id}',
       options: {
         response: {
-          schema: validate.user.userGET,
+          schema: validate.user.GET,
         },
         tags: ['api'],
       },

diff --git a/routes/user.test.js b/routes/user.test.js
@@ -76,7 +76,8 @@ describe('user', () => {
       },
       payload: testCase,
     })
-    console.log(res.result)
+    // console.log(res.result)
+    assert.equal(res.statusCode, 201)
   })
 
   it('GET /user', async () => {