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

Réintégrer les tests générés automatiquement via le simulateur des impôts #734

Closed
3 of 5 tasks
MattiSG opened this issue Apr 20, 2017 · 6 comments
Closed
3 of 5 tasks

Comments

@MattiSG
Copy link
Member

MattiSG commented Apr 20, 2017

  • Tenter de faire passer tous les tests supprimés et réintégrer ceux qui sont au vert.
  • Passer sur CircleCI si la performance est un frein.
  • Supprimer des tests par partionnement de l'espace des résultats si la performance est un frein.
  • Créer une branche contenant les tests ne passant pas pour faciliter le travail de correction associé.
  • Supprimer les scripts de génération de tests non maintenus.

Fait suite à #730.

@fpagnoux
Copy link
Member

This could be useful: https://github.com/dlanger/nose-parallel

@fpagnoux
Copy link
Member

Circle says it can do it on its own though: https://circleci.com/docs/1.0/setting-up-parallelism/

@benjello
Copy link
Member

Ping me if some tests are failing. May be a neat and quick fix can expand there life expectancy.

@fpagnoux
Copy link
Member

fpagnoux commented May 19, 2017

J'ai fait passé pas mal de tests en mettant par défaut une marge d'erreur relative de O.1%. Il y a par contre beaucoup de tests qui ne passent pas parce qu'on retourne l'opposé du résultat attendu. Je pense a priori juste mettre des - dans les tests YAML.

@benjello
Copy link
Member

Oui l'irpp est l'impôt payé par le foyer fiscal donc retiré aux ménages donc négatifs dans la convention openfisca.

@fpagnoux
Copy link
Member

fpagnoux commented May 22, 2017

Après nettoyage, on a 4674 tests qui passent en 40 minutes. On peut en théorie passer en 10 avec du parallélisme, mais ça demande de configurer Circle finement.

Les tests supprimés sont disponibles sur la branche removed-tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants