Projet

Général

Profil

Development #10284

utiliser tox pour lancer les tests sous django 1.8

Ajouté par Serghei Mihai il y a environ 8 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
13 mars 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

0001-jenkins.sh-launch-tests-with-tox-in-django-1.7-and-1.patch (2,73 ko) 0001-jenkins.sh-launch-tests-with-tox-in-django-1.7-and-1.patch Serghei Mihai, 14 mars 2016 18:56
0001-jenkins.sh-launche-tests-with-tox-in-django-1.7-and-.patch (2,81 ko) 0001-jenkins.sh-launche-tests-with-tox-in-django-1.7-and-.patch Serghei Mihai, 08 avril 2016 11:51
0001-misc-use-tox-to-run-tests-in-jenkins-10284.patch (3,17 ko) 0001-misc-use-tox-to-run-tests-in-jenkins-10284.patch Frédéric Péters, 24 avril 2017 15:46
0001-misc-use-tox-to-run-tests-in-jenkins-10284.patch (4,08 ko) 0001-misc-use-tox-to-run-tests-in-jenkins-10284.patch Frédéric Péters, 28 décembre 2017 14:54
0001-misc-use-tox-to-run-tests-in-jenkins-10284.patch (4,12 ko) 0001-misc-use-tox-to-run-tests-in-jenkins-10284.patch Frédéric Péters, 28 décembre 2017 15:09

Révisions associées

Révision b0bea13b (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

misc: use tox to run tests in jenkins (#10284)

Historique

#1

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-jenkins.sh-launch-tests-with-tox-in-django-1.7-and-1.patch supprimé
#2

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-jenkins.sh-launch-tests-with-tox-in-django-1.7-and-1.patch ajouté
  • Statut changé de Nouveau à En cours
  • Assigné à mis à Serghei Mihai
#3

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-jenkins.sh-launch-tests-with-tox-in-django-1.7-and-1.patch supprimé
#5

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

Il ne s'applique pas chez moi.

#6

Mis à jour par Serghei Mihai il y a environ 8 ans

Hm... Ton master est à jour? Tu n'as pas de modifications locales?

#7

Mis à jour par Frédéric Péters il y a environ 8 ans

Il ne s'applique pas parce qu'il retire une ligne

-pip install --upgrade pytest pytest-django pytest-cov WebTest

alors que la ligne est désormais pip install --upgrade pytest pytest-django pytest-cov WebTest django-webtest.

Et peut-être d'autres choses.

#8

Mis à jour par Frédéric Péters il y a environ 8 ans

(autre modif, dans la référence à pylint, la dernière version est désormais tirée, le tox.ini exige 1.4.0)

Aussi, le setup.py permet maintenant de fonctionner avec un vieux django, il faudrait garder le >=1.7.

#10

Mis à jour par Thomas Noël il y a environ 7 ans

Simplification possible suite à #15851 : on ne supporte plus Django 1.7

#11

Mis à jour par Frédéric Péters il y a environ 7 ans

À nouveau trop de temps perdu ici, work in progress, ça plantait sur cmsplugin_blurp, j'ai uploadé une nouvelle version sur pypi, ça plante désormais sur cmsplugin_blurp, différemment (et bien sûr hors de tox pas de problème).

Building wheels for collected packages: django-cmsplugin-blurp
  Running setup.py bdist_wheel for django-cmsplugin-blurp: started
  Running setup.py bdist_wheel for django-cmsplugin-blurp: finished with status 'error'
  Complete output from command /tmp/tox-fred/combo/coverage-django18-pylint/bin/python2 -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-7nKjr0/django-cmsplugin-blurp/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" bdist_wheel -d /tmp/tmpITBg7Ipip-wheel- --python-tag cp27:
  running bdist_wheel
  running build
  running compile_translations

  !!! Please install Django >= 1.7 to build translations
  running build_py
  error: package directory 'src/cmsplugin_blurp' does not exist

  ----------------------------------------
  Failed building wheel for django-cmsplugin-blurp

Aussi tenté, taper directement vers le dépôt git, avec le même résultat.

#12

Mis à jour par Frédéric Péters il y a plus de 6 ans

Les blurps dégagés (#20914), quelques soucis avec w.c.s. à régler et là c'est enfin bon. (côté w.c.s. j'ai en local eu besoin de #20916, et on pourrait trouver idéal d'installer wcs via tox également mais comme combo et wcs installent leur manage.py sous ce nom, ça n'est pas possible et je laisse ça pour un autre moment).

#13

Mis à jour par Frédéric Péters il y a plus de 6 ans

Et avec #20917, on peut même avoir Django 1.11 dans le tox.ini.

#14

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Sujet changé de utiliser tox pour lancer les tests sous django 1.8 et django 1.11 à utiliser tox pour lancer les tests sous django 1.8

Et avec #20917, on peut même avoir Django 1.11 dans le tox.ini.

Ah ah, sauf qu'il me semble qu'un jeu de dépendance fait que l'environnement "django 1.11" construit par Tox, en fait, contient la version 1.8. Restons-en au patch précédent.

#15

Mis à jour par Serghei Mihai il y a plus de 6 ans

Ack pour le patch avec la version 1.8

#16

Mis à jour par Frédéric Péters il y a plus de 6 ans

Ack pour le patch avec la version 1.8

C'est suite à une lecture ou un test ?

#17

Mis à jour par Serghei Mihai il y a plus de 6 ans

Testé d'abord ton dernier patch et constaté la même chose: dans l'environnement 1.11 c'est django 1.8 qui est installé.

#18

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)

Les commits "django 1.11" étant passés, j'ai poussé un tox.ini qui teste des deux côtés.

commit b0bea13bc92b96b2f7daf5fd243b960ae9de8cca
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Apr 24 15:25:45 2017 +0200

    misc: use tox to run tests in jenkins (#10284)
#19

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF