search

socialement-competents / backathon

0 stars 1 forks source link

Added ccov #1

Closed 0xthc closed 6 years ago

codecov[bot] commented 6 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@679291e). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   64.28%           
=========================================
  Files             ?       10           
  Lines             ?      294           
  Branches          ?       38           
=========================================
  Hits              ?      189           
  Misses            ?      105           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 679291e...c89ed1c. Read the comment docs.

tsauvajon commented 6 years ago

Ce qu'il serait bien de faire aussi c'est passer les configurations de jest et jest-junit (pourquoi pas aussi prettier et d'autres trucs) dans le package.json ou dans leur propre config.json plutôt que des variables d'environnement.

Je pense par exemple à JEST_JUNIT_OUTPUT dans integration-tests.Dockerfile.

Le problème c'est qu'à l'heure actuelle c'est éparpillé un peu partout et c'est assez difficile de comprendre qu'est-ce qui est configuré, et où.

Exemple de config jest-junit dans package.json :

  "jest-junit": {
    "suiteName": "Backathon integration tests",
    "output": "./src/test/coverage/junit.xml"
  }
tsauvajon commented 6 years ago

Bon ça reste bloqué à npm run test:ci apparemment ... Je vois pas d'où ça peut venir.

EDIT: c'est parce que npm et yarn ne passent pas les paramètres de la même manière.

tsauvajon commented 6 years ago

On sent que je galère ou pas ?