.github/workflows/docker-github.yml

@@ -0,0 +1,23 @@
+name: Publish Docker image
+on:
+  push:
+    branches:
+      - 'master'
+jobs:
+  push_to_registry:
+    name: Push Docker image to GitHub Packages
+    runs-on: ubuntu-latest
+    permissions:
+      packages: write
+      contents: read
+    steps:
+      - name: Check out the repo
+        uses: actions/checkout@v2
+      - name: Push to GitHub Packages
+        uses: docker/build-push-action@v1
+        with:
+          username: ${{ github.actor }}
+          password: ${{ secrets.GITHUB_TOKEN }}
+          registry: docker.pkg.github.com
+          repository: edysegura/nodejs-restful/node-restful
+          tag_with_ref: true

.github/workflows/docker.yml

@@ -0,0 +1,33 @@
+name: DockerHub
+
+on:
+  push:
+    branches:
+      - 'master'
+
+jobs:
+  docker:
+    runs-on: ubuntu-latest
+    steps:
+      -
+        name: Set up QEMU
+        uses: docker/setup-qemu-action@v1
+      -
+        name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v1
+      -
+        name: Login to DockerHub
+        uses: docker/login-action@v1
+        with:
+          username: ${{ secrets.DOCKERHUB_USERNAME }}
+          password: ${{ secrets.DOCKERHUB_TOKEN }}
+      -
+        name: Build and push
+        id: docker_build
+        uses: docker/build-push-action@v2
+        with:
+          push: true
+          tags: edysegura/node-restful:latest
+      -
+        name: Image digest
+        run: echo ${{ steps.docker_build.outputs.digest }}

.github/workflows/nodejs.yml

@@ -0,0 +1,21 @@
+name: Node CI
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node-version: [12.x, 14.x]
+    steps:
+    - uses: actions/checkout@v1
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v1
+      with:
+        node-version: ${{ matrix.node-version }}
+    - name: npm install, build, and test
+      run: |
+        npm ci --only=production
+      env:
+        CI: true

Dockerfile

@@ -1,4 +1,4 @@
-FROM node:12
+FROM node:lts-alpine
 
 # Create app directory
 WORKDIR /usr/src/app

app/TaskService.js

@@ -0,0 +1,57 @@
+const db = {}
+let sequence = 0
+
+class TaskService {
+
+  static add(newTask) {
+    return new Promise((resolve) => {
+      const task = {
+        id: ++sequence,
+        done: newTask.done || false,
+        description: newTask.description
+      }
+      db[task.id] = task
+      resolve(task)
+    })
+  }
+
+  static getAll() {
+    const toArray = key => db[key]
+    return new Promise((resolve) => {
+      const tasks = Object.keys(db).map(toArray)
+      resolve(tasks)
+    })
+  }
+
+  static getById(id) {
+    return new Promise((resolve) => {
+      resolve(db[id])
+    })
+  }
+
+  static update(taskId, updatedTask) {
+    return new Promise(async (resolve) => {
+      const task = await TaskService.getById(taskId)
+      if (task) {
+        const hasValue = updatedTask.done != null
+        task.done = hasValue ? updatedTask.done : task.done
+        task.description = updatedTask.description || task.description
+        resolve(task)
+      }
+      resolve(null)
+    })
+  }
+
+  static delete(id) {
+    return new Promise((resolve) => {
+      const task = db[id]
+      if (task) {
+        delete db[id]
+        resolve(true)
+      }
+      resolve(false)
+    })
+  }
+}
+
+module.exports = TaskService

app/taskApi.js

@@ -1,62 +1,49 @@
 const express = require('express')
 const router = express.Router()
 
-let db = {}
-let sequence = 0
+const TaskService = require('./TaskService')
 
 const notFound = response => {
   response.status(404).send('Not found!')
 }
 
-
-router.get('/', (request, response) => {
-  const toArray = key => {
-    return db[key]
-  }
-  const tasks = Object.keys(db).map(toArray)
-  tasks.length
+router.get('/', async (request, response) => {
+  const tasks = await TaskService.getAll()
+  tasks && tasks.length
     ? response.json(tasks)
-    : response.status(204).send('')
+    : response.status(204).end('')
 })
 
-router.get('/:id', (request, response) => {
-  const task = db[request.params.id]
+router.get('/:id', async (request, response) => {
+  const taskId = request.params.id
+  const task = await TaskService.getById(taskId)
   task
     ? response.json(task)
     : notFound(response)
 })
 
-router.post('/', (request, response) => {
-  const newTask = {
-    'id': ++sequence,
-    'done': request.body.done || false,
-    'description': request.body.description
-  }
-
-  db[newTask.id] = newTask
-
-  response.status(201).json(newTask)
+router.post('/', async (request, response) => {
+  const task = await TaskService.add(request.body)
+  response
+    .status(201)
+    .json(task)
 })
 
-router.put('/:id', (request, response) => {
-  const task = db[request.params.id]
-  if (task) {
-    task.done = request.body.done != null ? request.body.done : false
-    task.description = request.body.description || task.description
-    response.json(task)
-  } else {
-    notFound(response)
-  }
+router.patch('/:id', async (request, response) => {
+  const updatedTask = await TaskService.update(
+    request.params.id,
+    request.body
+  );
+  updatedTask
+    ? response.json(updatedTask)
+    : notFound(response);
 })
 
-router.delete('/:id', (request, response) => {
-  var task = db[request.params.id]
-  if (task) {
-    delete db[request.params.id]
-    response.send('Task deleted')
-  } else {
-    notFound(response)
-  }
+router.delete('/:id', async (request, response) => {
+  const isDeleted = await TaskService.delete(request.params.id);
+  isDeleted
+    ? response.end('Task deleted!')
+    : notFound(response)
 })
 
 module.exports = router