Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Supprimer la métrique de couverture de code plus disponible #6552

Merged
merged 1 commit into from
Jul 6, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
**/dist
**/tmp
**/coverage
**/.nyc_output
**/pix.log.*

# dependencies
Expand Down
28 changes: 1 addition & 27 deletions INSTALLATION.md
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,7 @@ Ce repository est configuré pour indiquer aux IDE Webstorm et Vscode la configu
Malgré cela, il peut arriver que des erreurs de lint soient introduites.

Pour tenter de les corriger automatiquement lors du commit, installer un hook de pre-commit.

```
npm run local:trigger-lint-on-commit
```
Expand All @@ -210,33 +211,6 @@ Si vous souhaitez désactiver
npm run local:prevent-trigger-lint-on-commit
```

#### Obtenir la couverture de test

> Test coverage is a percentage measure of the degree to which the source code of a program is executed when a
> particular test suite is run.
[Source](https://en.wikipedia.org/wiki/Code_coverage)

Il n'y a pas d'objectif de couverture de test dans la suite de test.

Les [pratiques actuelles de test automatisé](./docs/test.md) sont considérées comme suffisantes.

La couverture de tests est utilisée à des fins de développement pour :

- détecter du code non utilisé suite à un refactoring, afin de le supprimer;
- détecter du code non testé, afin de le tester, dans le cadre d'amélioration continue.

La couverture de tests n'est disponible que

- dans l'API, pour le dossier `lib`;
- en local.

Elle est obtenue avec la commande suivante.

``` shell
cd api
npm run local:test:coverage
```

#### Tracer de manière détaillée l'envoi d'email

On peut tracer de manière détaillée (debug) l'appel de l'API d'email avec la
Expand Down
1 change: 0 additions & 1 deletion api/.eslintignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
# https://eslint.org/docs/user-guide/configuring#eslintignore
# .file are implicitly ignored, unless explicitly specified here
node_modules/
nyc.config.js
3 changes: 0 additions & 3 deletions api/.ls-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,6 @@ ls:
ignore:
- .git
- node_modules
- .nyc_output
- db/migrations
- db/seeds
- scripts
- nyc.config.js
- codemods
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Volontaire de supprimer codemods ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, c'est du ménage pas fait à l'époque

10 changes: 0 additions & 10 deletions api/nyc.config.js

This file was deleted.

Loading