Merge pull request #68 from github/feelepxyz/simplify-fake-server-setup

Refactor fake server setup

__tests__/server/db.json

@@ -48,8 +48,5 @@
       }
     }
   ],
-  "credentials": [{"id": 1}],
-  "dependencies": [],
-  "update_job_errors": [],
   "pull_requests": []
 }

__tests__/server/server.js

@@ -24,21 +24,21 @@ const SERVER_PORT = process.argv.slice(2)[0] || 9000
 // - POST /update_jobs {data: {...attrs}} would persist a new update job with id
 //   2
 
-// NOTE: Serialise the response like dependabot-api
-router.render = (_, res) => {
-  const id = res.locals.data.id
-  const data = {
-    attributes: res.locals.data
-  }
-  if (id) {
-    data.id = id
+server.use(middlewares)
+
+server.get('/update_jobs/:id/details', (req, res) => {
+  const id = req.params.id
+  const updateJob = db.update_jobs.find(job => `${job.id}` === id)
+  if (!updateJob) {
+    return res.status(404).end()
   }
+
   res.jsonp({
-    data
+    data: {
+      attributes: updateJob
+    }
   })
-}
-
-server.use(middlewares)
+})
 
 // Inject a legit GITHUB_TOKEN to increase rate limits fetching manifests from github
 server.get('/update_jobs/:id/credentials', (_, res) => {
@@ -64,33 +64,28 @@ server.post(
     db.pull_requests.push(data)
     router.db.write()
 
-    res.jsonp({})
+    res.status(204).send()
   }
 )
 
-// TEMP HACK: Always return 204 on post so the updater doesn't buil out
-server.use(jsonServer.bodyParser, (req, res, next) => {
-  if (req.method === 'POST' && req.body.data) {
-    req.body = req.body.data
-    res.sendStatus(204)
-    return
-  }
-  next()
+server.post('/update_jobs/:id/record_update_job_error', (_, res) => {
+  res.status(204).send()
 })
 
-// NOTE: These map to resources in db.json
-server.use(
-  jsonServer.rewriter({
-    '/update_jobs/:id/details': '/update_jobs/:id',
-    '/update_jobs/:id/credentials': '/credentials/:id',
-    '/update_jobs/:id/record_update_job_error': '/update_job_errors/:id',
-    '/update_jobs/:id/mark_as_processed': '/update_jobs/:id',
-    '/update_jobs/:id/update_dependency_list': '/dependencies/:id',
-    '/update_jobs/:id/record_package_manager_version': '/update_jobs/:id'
-  })
-)
+server.patch('/update_jobs/:id/mark_as_processed', (_, res) => {
+  res.status(204).send()
+})
+
+server.post('/update_jobs/:id/update_dependency_list', (_, res) => {
+  res.status(204).send()
+})
+
+server.post('/update_jobs/:id/record_package_manager_version', (_, res) => {
+  res.status(204).send()
+})
 
 server.use(router)
+
 server.listen(SERVER_PORT, () => {
   console.log(`json-server is running on http://localhost:${SERVER_PORT}`)
 })

__tests__/updater-integration.test.ts

@@ -74,7 +74,7 @@ describe('Updater', () => {
     const res = await client.get('/pull_requests/1')
 
     expect(res.status).toEqual(200)
-    expect(res.data.data.attributes['pr-title']).toEqual(
+    expect(res.data['pr-title']).toEqual(
       'Bump fetch-factory from 0.0.1 to 0.2.1'
     )
   })