nodeSolidServer / nodeSolidServer/node-solid-server

Tests are not running anymore in CI

Ouverte
#1,817 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
JavaScript
Étoiles
1.8k
Forks
308
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Tests are not running anymore in CI
There is no issue locally. Issue is the same with c8 or nyc with and without --experimental-require-module (not needed anymore the require module has been backported in node 20.19.0 and 22.14.0)

This is an example https://github.com/nodeSolidServer/node-solid-server/tree/refs/heads/fix/issue%231798
branch fix/issue#1798

Run npm run nyc

> solid-server@5.8.5 nyc
> cross-env NODE_TLS_REJECT_UNAUTHORIZED=0 nyc --reporter=text-summary mocha --recursive test/integration/ test/unit/

(node:2062) Warning: Setting the NODE_TLS_REJECT_UNAUTHORIZED environment variable to '0' makes TLS connections and HTTPS requests insecure by disabling certificate verification.
(Use `node --trace-warnings ...` to show where the warning was created)
node:internal/process/promises:394
    triggerUncaughtException(err, true /* fromPromise */);
    ^

[Error: EEXIST: file already exists, mkdir '/home/runner/work/node-solid-server/node-solid-server/test/resources/accounts/single-user'] {
  errno: -1[7](https://github.com/nodeSolidServer/node-solid-server/actions/runs/14363534639/job/40270776945#step:13:8),
  code: 'EEXIST',
  syscall: 'mkdir',
  path: '/home/runner/work/node-solid-server/node-solid-server/test/resources/accounts/single-user'
}

Node.js v22.14.0

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez avec la commande npm run nyc et la branche d’exemple fix/issue#1798. Reproduisez l’échec dans CI avec Node.js 22.14.0, puis examinez comment test/resources/accounts/single-user est créé pendant l’exécution des tests d’intégration et unitaires. C’est terminé lorsque la commande de test CI s’achève sans l’erreur EEXIST.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript, nodejs
Domaine
ci-cd, testing-qa
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
38/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.