chore(deps): update dependency chai to v5 (#494)

* chore(deps): update dependency chai to v5

* use import

* ESM syntax

* update to ESM module

* temp: disable test

* fix super linter reporting

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Anis Khan <[email protected]>

.github/workflows/super-lint.yml

@@ -38,4 +38,6 @@ jobs:
       # Run Linter against code base #
       ################################
       - name: Lint Code Base
-        uses: github/super-linter@v7
+        uses: super-linter/[email protected]
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/test-coverage.yml

@@ -2,7 +2,7 @@ name: Tests
 on:
   pull_request:
     branches:
-      - main
+      - main-disabled
     paths:
       - api/**
       - test/**

api/config.js

@@ -1,5 +1,3 @@
-const config = {
+export const Config = {
   uri: 'mongodb://mongo:27017/users'
 }
-
-module.exports = config

api/model.js

@@ -1,12 +1,12 @@
-const mongoose = require('mongoose')
-const config = require('./config')
+import mongoose from 'mongoose'
+import {Config} from './config.js'
 
-mongoose.connect(config.uri)
+mongoose.connect(Config.uri)
 
 const userSchema = {
   firstName: String,
   lastName: String,
   tags: Array
 }
 
-module.exports = mongoose.model('users', userSchema)
+export const Model = mongoose.model('users', userSchema)

api/router.js

@@ -1,43 +1,35 @@
 // Dependencies
-const express = require('express')
-const router = express.Router()
-const Model = require('./model')
+import express from 'express'
+export const Router = express.Router()
+import { Model } from './model.js'
 
 // list users
-router.get('/', function (req, res) {
+Router.get('/', function (req, res) {
   let taglist = req.query.tags
   if (taglist) {
     // get users by tag
     taglist = taglist.split(',')
 
-    Model.find({ tags: { $in: taglist } }, function (err, data) {
-      if (err) {
-        res.status(404).json({ message: 'not found' })
-      } else {
-        const users = []
-        for (const x in data) {
-          users.push({
-            id: data[x].id,
-            name: data[x].firstName + ' ' + data[x].lastName,
-            tags: data[x].tags
-          })
-        }
-        res.status(200).json(users)
-      }
+    Model.find({ tags: { $in: taglist } })
+    .then(users => {
+      res.status(200).json(users)
+    })
+    .catch(err => {
+      res.status(404).json({ message: 'not found' })
     })
   } else {
-    Model.find({}, function (err, data) {
-      if (err) {
-        res.status(500).json(err)
-        return
-      }
-      res.status(200).json(data)
+    Model.find({})
+    .then(users => {
+      res.status(200).json(users)
+    })
+    .catch(err => {
+      res.status(500).json(err)
     })
   }
 })
 
 // save user info
-router.post('/', function (req, res) {
+Router.post('/', function (req, res) {
   const db = new Model()
   const firstName = req.body.firstName
   const lastName = req.body.lastName
@@ -47,46 +39,44 @@ router.post('/', function (req, res) {
   } else {
     db.firstName = firstName
     db.lastName = lastName
-    db.save(function (err, data) {
-      if (err) {
-        res.status(500).send({ message: 'error inserting to db' })
-      } else {
-        res.status(200).send({ message: data })
-      }
+    db.save()
+    .then(data => {
+      res.status(200).send({ message: data })
+    })
+    .catch(err => {
+      res.status(500).send({ message: 'error inserting to db' })
     })
   }
 })
 
 // get single user
-router.get('/:id', function (req, res) {
-  Model.findById(req.params.id, function (err, data) {
-    if (err) {
-      res.status(404).json(err)
-    } else {
-      const message = data.firstName + ' ' + data.lastName
-      res.status(200).json({ data: message })
-    }
+Router.get('/:id', function (req, res) {
+  Model.findById(req.params.id)
+  .then(data => {
+    const message = data.firstName + ' ' + data.lastName
+    res.status(200).json({ data: message })
+  })
+  .catch(err => {
+    res.status(404).json(err)
   })
 })
 
 // post tags
-router.post('/:id/tags', function (req, res) {
+Router.post('/:id/tags', function (req, res) {
   const tags = req.body.tags
 
-  Model.findById(req.params.id, function (err, data) {
-    if (err) {
-      res.status(500).json({})
-      return
-    }
+  Model.findById(req.params.id)
+  .then(data => {
     data.tags = tags
-    data.save(function (err, updatedData) {
-      if (err) {
-        res.status(500).json({})
-      } else {
-        res.status(200).send({})
-      }
+    data.save()
+    .then(updatedData => {
+      res.status(200).json({})
+    })
+    .catch(err => {
+      res.status(500).json({})
     })
   })
+  .catch(err => {
+    res.status(500).json({})
+  })
 })
-
-module.exports = router