Development #46988
tests: ajouter une fixture pour simuler de grosses bases d'utilisateur
Début:
24 septembre 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Avec une option --slow pour ne pas lancer ça dans chaque branche wip/.
Fichiers
Révisions associées
tests: add a large_userbase fixture (#46988)
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-tests-add-a-large_userbase-fixture-46988.patch 0001-tests-add-a-large_userbase-fixture-46988.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-tests-add-a-large_userbase-fixture-46988.patch ajouté
Vite fait avec ce patch, pour 200 requêtes donc,
- Sans index
152.55s call tests/test_large_userbase.py::test_large_userbase_find_duplicates 147.55s call tests/test_large_userbase.py::test_large_userbase_find_duplicates_with_birthdate
Soit 0.75s par requête.
- Avec index
30.44s call tests/test_large_userbase.py::test_large_userbase_find_duplicates_with_birthdate 24.06s call tests/test_large_userbase.py::test_large_userbase_find_duplicates
Soit 0.15s par requête.
Conclusion, ça marche comme il faut.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier
0001-tests-add-a-large_userbase-fixture-46988.patchsupprimé
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-test_large_userbase.patch 0001-test_large_userbase.patch ajouté
Valentin Deniaud a écrit :
Vite fait avec ce patch,
Celui-là.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-tests-add-a-large_userbase-fixture-46988.patch 0001-tests-add-a-large_userbase-fixture-46988.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Petites remarques mais comme tu veux.
-PGPORT=`python -c 'import struct; import socket; s=socket.socket(); s.setsockopt(socket.SOL_SOCKET, socket.SO_LINGER, struct.pack("ii", 1, 0)); s.bind(("", 0)); print(s.getsockname()[1]); s.close()'` pg_virtualenv -o fsync=off ${env.TMPDIR}/venv/bin/tox -r"""
+A2_PYTESTOPTIONS=--slow PGPORT=`python -c 'import struct; import socket; s=socket.socket(); s.setsockopt(socket.SOL_SOCKET, socket.SO_LINGER, struct.pack("ii", 1, 0)); s.bind(("", 0)); print(s.getsockname()[1]); s.close()'` pg_virtualenv -o fsync=off ${env.TMPDIR}/venv/bin/tox -r"""
C'est moche comme diff, moyen d'ajouter A2_PYTESTOPTIONS=--slow sur une nouvelle ligne ?
tests: add a large_userbase fixture (#46988)
Et pourtant modif des tests rbac de dedans, j'aurais bien vu deux commits, le premier qui introduit slow et l'utilise dans rbac, le second qui fasse ce que le titre du commit dit.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0002-tests-add-a-large_userbase-fixture-46988.patch 0002-tests-add-a-large_userbase-fixture-46988.patch ajouté
- Fichier 0001-ci-add-a-slow-options-to-also-run-slow-tests-46988.patch 0001-ci-add-a-slow-options-to-also-run-slow-tests-46988.patch ajouté
- Statut changé de Solution validée à Solution proposée
C'est mieux comme ça ? Mais je ne sais pas si ma ligne de shell avec un \ va marcher.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne a écrit :
Mais je ne sais pas si ma ligne de shell avec un \ va marcher.
Moi non plus, en fait je ne comprends pas pourquoi il y a le droit à un saut à la ligne après ${env.TMPDIR}/venv/bin/pip install tox et pas après PYTESTOPTIONS=--slow, donc fais au plus sûr.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 458712039c3f88d2576ca8241daf30600f76ba15
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Sep 24 21:39:45 2020 +0200
tests: add a large_userbase fixture (#46988)
commit 9d9e34ee6a3ef079ce3f8fcef7cf5986e8cc2145
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Oct 1 11:49:10 2020 +0200
ci: add a --slow options to also run slow tests (#46988)Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
ci: add a --slow options to also run slow tests (#46988)