Upload 4.12

part4/bloglist/controllers/blogs.js

@@ -13,11 +13,22 @@ blogsRouter.post('/', async (request, response) => {
     title: body.title,
     author: body.author,
     url: body.url,
-    likes: body.likes
+    likes: body.likes || 0
   })).save()
 
   response.status(201).json(blog)
 
 })
 
+blogsRouter.get('/:id', async (request, response) => {
+  const blog = await Blog.findById(request.params.id)
+  if (blog) response.json(blog)
+  else response.status(404).end()
+})
+
+blogsRouter.delete('/:id', async (request, response) => {
+  await Blog.findByIdAndRemove(request.params.id)
+  response.status(204).end()
+})
+
 module.exports = blogsRouter

part4/bloglist/requests/delete_blog.rest

@@ -0,0 +1 @@
+DELETE http://localhost:3001/api/blogs/64778b41a8f175e16455b293

part4/bloglist/requests/get_by_id.rest

@@ -0,0 +1 @@
+GET http://localhost:3001/api/blogs/647788bc1d350a17ba599541

part4/bloglist/requests/post_blog.rest

@@ -2,6 +2,6 @@ POST http://localhost:3001/api/blogs
 content-type: application/json
 
 {
-    "title": "Another blog", "author": "Andrew", "url": "www.andrew.eu", "likes": 1000
+    "title": "Last blog", "author": "Andrew", "url": "www.andrew.eu", "likes": 1000
 
 }

part4/bloglist/tests/blog_api.test.js

@@ -12,53 +12,84 @@ beforeEach(async () => {
   await Blog.insertMany(helper.initialBlogs)
 })
 
-test('blogs are returned as json', async () => {
-  await api
-    .get('/api/blogs')
-    .expect(200)
-    .expect('Content-Type', /application\/json/)
+describe('getting blogs', () => {
+  test('blogs are returned as json', async () => {
+    await api
+      .get('/api/blogs')
+      .expect(200)
+      .expect('Content-Type', /application\/json/)
+  })
+
+  test('all blogs are returned', async () => {
+    const response = await api.get('/api/blogs')
+    expect(response.body).toHaveLength(helper.initialBlogs.length)
+  })
+
+  test('a specific blog is within the returned blogs', async () => {
+    const response = await api.get('/api/blogs')
+    const contents = response.body.map((r) => r.title)
+    expect(contents).toContain('Another blog')
+  })
 })
 
-test('all blogs are returned', async () => {
-  const response = await api.get('/api/blogs')
-  expect(response.body).toHaveLength(helper.initialBlogs.length)
+describe('adding blog', () => {
+  test('a valid blog can be added ', async () => {
+    const newBlog = {
+      title: 'Test',
+      author: 'Tester',
+      url: 'http:/test.com',
+      likes: 999
+    }
+
+    await api
+      .post('/api/blogs')
+      .send(newBlog)
+      .expect(201)
+      .expect('Content-Type', /application\/json/)
+
+    const blogsAtEnd = await helper.blogsInDb()
+    expect(blogsAtEnd).toHaveLength(helper.initialBlogs.length + 1)
+    const contents = blogsAtEnd.map((n) => n.title)
+    expect(contents).toContain('Test')
+  })
+
+  test('blog without title or url is not added', async () => {
+    const newBlog = {
+      author: 'Tester',
+      likes: 999
+    }
+
+    await api.post('/api/blogs').send(newBlog).expect(400)
+
+    const blogsAtEnd = await helper.blogsInDb()
+    expect(blogsAtEnd).toHaveLength(helper.initialBlogs.length)
+  })
+
+  test('blog without likes is added with 0 likes', async () => {
+    const newBlog = {
+      title: 'Another test',
+      author: 'Tester',
+      url: 'http:/test.com'
+    }
+
+    await api
+      .post('/api/blogs')
+      .send(newBlog)
+      .expect(201)
+      .expect('Content-Type', /application\/json/)
+
+    const blogsAtEnd = await helper.blogsInDb()
+    const addedBlog = blogsAtEnd.find(blog => blog.title === 'Another test')
+    expect(addedBlog.likes).toBe(0)
+  })
 })
 
-test('a specific blog is within the returned blogs', async () => {
-  const response = await api.get('/api/blogs')
-  const contents = response.body.map((r) => r.title)
-  expect(contents).toContain('Another blog')
-})
 
-test('a valid blog can be added ', async () => {
-  const newBlog = {
-    title: 'Test',
-    author: 'Tester',
-    url: 'http:/test.com',
-    likes: 999
-  }
-
-  await api
-    .post('/api/blogs')
-    .send(newBlog)
-    .expect(201)
-    .expect('Content-Type', /application\/json/)
-
-  const blogsAtEnd = await helper.blogsInDb()
-  expect(blogsAtEnd).toHaveLength(helper.initialBlogs.length + 1)
-  const contents = blogsAtEnd.map((n) => n.title)
-  expect(contents).toContain('Test')
-})
-
-test('blog without content is not added', async () => {
-  const newBlog = {
-    likes: 999
-  }
-
-  await api.post('/api/blogs').send(newBlog).expect(400)
-
-  const blogsAtEnd = await helper.blogsInDb()
-  expect(blogsAtEnd).toHaveLength(helper.initialBlogs.length)
+describe('blog id check', () => {
+  test('blog id is defined', async () => {
+    const response = await api.get('/api/blogs')
+    expect(response.body[0].id).toBeDefined()
+  })
 })
 
 afterAll(async () => {