Salut,
bonne galère à débuguer form_hidden(), qui avait été pétée par
mégarde. Là où je suis choqué -- voire abasourdi 
-- c'est que
j'avais pris la peine de faire un test unitaire pour cette fonction.
Du coup j'ai relancé ces tests et là, stupeur : il y en a pas mal qui
sont pétés... à quoi ça sert d'essayer de travailler proprement ?
-- Fil
Salut,
bonne galère à débuguer form_hidden(), qui avait été pétée par
mégarde.
non, il avait été pété par moi, ou plutôt il était déjà pété avant,
et visiblement en corrigeant des choses j'en ai pété d'autres.
Là où je suis choqué -- voire abasourdi
-- c'est que
j'avais pris la peine de faire un test unitaire pour cette fonction.
Du coup j'ai relancé ces tests et là, stupeur : il y en a pas mal qui
sont pétés... à quoi ça sert d'essayer de travailler proprement ?
Le pb, c'est que dans mon répertoire tests quand je fais
svn up
on me dit:
svn: Cannot replace a directory from within
je fais
svn info
j'obtiens
Path: .
URL: svn://zone.spip.org/spip-zone/_dev_/tests
et en allant sur le site, je découvre que ce répertoire n'est plus là.
Donc ce serait bien de prévenir quand on déménage ça,
et dans les logs dire précisément quels sont les tests qui ne passaient plus,
et donner leur URL. Mais même à l'époque où je pouvais les faire fonctionner,
j'avais tellement de faux négatifs que j'avoue que je finissais par ne plus les faire fonctionner.
Il faudrait effectivement essayer de stabiliser tout ça.
Committo,Ergo:Sum
Le pb, c'est que dans mon répertoire tests quand je fais
svn up
on me dit:
svn: Cannot replace a directory from within
Oui c'est Ben qui a tout déplacé sur spip-zone (et il a prévenu !).
j'avais tellement de faux négatifs que j'avoue que je finissais par ne plus
les faire fonctionner.
bien sûr, mais quand on recode une fonction c'est un endroit à
regarder ; j'ai ajouté un lien dans le commentaire de la fonction, je
ne sais pas si c'est une bonne méthode
Il faudrait effectivement essayer de stabiliser tout ça.
oui
-- Fil
Concernant les tests je rappelle la (proto-) documentation :
http://www.spip-blog.net/Extr3mz-pr0gr4mm1n.html
http://www.spip-contrib.net/Les-tests-unitaires
Dans les tests actuels j'ai remis un peu de vert, mais une chose est
cassée : il s'agit de la note suivante :
[[<a> <a href="x">x</a>]]
qui devrait donner une note nommée "a", et qui dans le cas présent
plante lamentablement.
C'est vrai que c'est un cas tordu, mais il respecte la spec.
-- Fil
http://trac.rezo.net/trac/spip/changeset/14066
répare ça et même plus, car visiblement [[<b> <b>x</b>]] n'a jamais dû marcher.
Committo,Ergo:Sum