hagopj13 / hagopj13/node-express-boilerplate

Tests issue

Open
#294 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
7.7k
Forks
2.3k
PR merge metrics
No merged PRs in 30d

Description

First of all, thanks for this template or boilerplate, there is a lot of modules that I learnt about.

When I run `npm run test` a lot of tests are failing like below, although I am running `npm run dev` in another instance.

```
FAIL tests/integration/auth.test.js (11.239 s)
Auth routes
POST /v1/auth/register
✕ should return 201 and successfully register user if request data is ok (683 ms)
✕ should return 400 error if email is invalid (105 ms)
```
Correct me if I am wrong. I can see that this project is not actively maintained? The reason for this is that I am seeing some not supported modules like `xss-clean`

```
npx create-nodejs-express-app project-name
Need to install the following packages:
create-nodejs-express-app@1.7.0
Ok to proceed? (y)

npm warn deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated xss-clean@0.1.4: Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
npm warn deprecated glob@7.1.6: Glob versions prior to v9 are no longer supported
npm warn deprecated uuid@3.4.0: Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.
Downloading files from repo https://github.com/hagopj13/node-express-boilerplate.git

```

Contributor guide

Open the contributing guide

Research direction

Reproduce the report with `npm run dev` running in one instance and `npm run test` in another, starting with `tests/integration/auth.test.js`. Read the failing output and inspect the reported dependency warnings, including `xss-clean`; done means the relevant integration tests pass and the unsupported dependency warnings are addressed or clearly explained.

Written by the indexing model from the issue text.

Assessment

Tech stack
express, javascript, node.js
Domain
backend, testing-qa
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.