Development #10564
mise à jour des tests wcs
0%
Description
Périodiquement, en local, les tests combo échouent:
tests/test_wcs.py ....F....
------------------------------------------------------------------ generated xml file: /home/serghei/dev/combo/test_results.xml ------------------------------------------------------------------
------------------------------------------------------------------------ coverage: platform linux2, python 2.7.10-final-0 ------------------------------------------------------------------------
Coverage XML written to file coverage.xml
============================================================================================ FAILURES ============================================================================================
_________________________________________________________________________________ test_current_forms_cell_setup __________________________________________________________________________________
@wcsctl_present
def test_current_forms_cell_setup():
cell = WcsCurrentFormsCell()
form_class = cell.get_default_form_class()
form = form_class()
> assert form.fields['wcs_site'].widget.choices == [
('', 'All'), (u'default', u'test'), (u'other', u'test2')]
E assert [('', <django...ult', 'test')] == [('', 'All'), ...er', 'test2')]
E At index 1 diff: ('other', 'test2') != (u'default', u'test')
E Use -v to get the full diff
Fichiers
Révisions associées
Revert "update wcs tests (#10564)"
This reverts commit d4033ffcd83cc4a517e974520707c1eac47000e1.
wcs: order wcs_sites by title (#10564)
Historique
Mis à jour par Frédéric Péters il y a environ 8 ans
- Assigné à mis à Serghei Mihai (congés, retour 15/05)
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Ça ressemble à un non-déterminisme quand on lit un dico (comme dans w.c.s.), joue sur --hashseed quand tu lances tox. Mouaip c'est ça:
<prep>
combo_wcs_sites.extend([(x, y.get('title')) for x, y in get_wcs_services().items()])
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
- Fichier 0001-update-wcs-tests-10564.patch 0001-update-wcs-tests-10564.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
C'est ça
Mis à jour par Frédéric Péters il y a environ 8 ans
Mauvais test. Il faut vérifier, 1) que All est en premier, 2) que toutes les entrées sont présentes.
Mis à jour par Frédéric Péters il y a environ 8 ans
En fait non, ce qu'il faut c'est assurer le déterminisme, en modifiant la cellule pour trier, par ordre alphabétique de titre, les différents wcs.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Vous êtes sûr que ce n'est pas déjà traité dans #10477 ?
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
Frédéric Péters a écrit :
lambda x.
Pas sûr d'avoir compris.
Mis à jour par Frédéric Péters il y a environ 8 ans
Traditionnellement la variable utilisée sur un lambda s'appelle "x", pas "s". Si tu l'appelles "s" pylint le signalera.
good-names=_,i,j,k,e,qs,pk,setUp,tearDown
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
- Fichier 0001-wcs-order-wcs_sites-by-title-10564.patch 0001-wcs-order-wcs_sites-by-title-10564.patch ajouté
Ah ok
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit e03cb0846cbe8ba00ad69d4050997e3e9c3a83f6
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Sun Mar 13 18:31:40 2016 +0100
wcs: order wcs_sites by title (#10564)
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de Résolu (à déployer) à Fermé
update wcs tests (#10564)