Project

General

Profile

Bug #30485

tipi: un blanc s'est glissé dans les refdet générées

Added by Benjamin Dauvergne 5 months ago. Updated 5 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
08 Feb 2019
Due date:
% Done:

100%

Patch proposed:
Yes
Planning:
No

Description

Car '%10s' pad les nombres avec des blancs sur la gauche.

0002-eopayment-transaction_id-must-always-be-equals-to-re.patch View (1.61 KB) Benjamin Dauvergne, 26 Feb 2019 01:19 PM

0004-tipi-prevent-whitespaces-in-refdet-fixes-30485.patch View (1.52 KB) Benjamin Dauvergne, 26 Feb 2019 01:19 PM

0001-setup.py-handle-case-of-a-tagged-dirty-workdir.patch View (1.07 KB) Benjamin Dauvergne, 26 Feb 2019 01:19 PM

0003-tests-add-non-regression-test-30485.patch View (2.33 KB) Benjamin Dauvergne, 26 Feb 2019 01:19 PM

Associated revisions

Revision 144f227d (diff)
Added by Benjamin Dauvergne 5 months ago

eopayment: transaction_id must always be equals to refdet (#30485)

As only refdet is returned by TIPI we can't generate transaction_id that
differs from it.

Revision bd10b0b0 (diff)
Added by Benjamin Dauvergne 5 months ago

tests: add non regression test (#30485)

Revision 23c7bec9 (diff)
Added by Benjamin Dauvergne 5 months ago

tipi: prevent whitespaces in refdet (fixes #30485)

History

#2 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Nouveau to Solution proposée
  • Tracker changed from Support to Bug

#4 Updated by Emmanuel Cazenave 5 months ago

  • Status changed from Solution proposée to Solution validée

#5 Updated by Frédéric Péters 5 months ago

Le 0001 n'a pas de rapport; je ne pige pas bien le rapport entre la correction d'ajouter 0 et la mise en xfail de deux tests.

#6 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Solution validée to Résolu (à déployer)
  • % Done changed from 0 to 100

#7 Updated by Benjamin Dauvergne 5 months ago

C'est du TDD.

#8 Updated by Frédéric Péters 5 months ago

Désolé ça ne m'aide pas à comprendre.

#9 Updated by Benjamin Dauvergne 5 months ago

Dans une autre vie, j'écrivais d'abord des tests qui foirent, qu'on marquait explicitement comme devant échouer et ensuite une fois corrigé on les marquait comme passant, l'idée étant qu'à chaque commit tous les tests devaient passer, comme cela on était certain d'avoir vraiment corrigé quelque chose; mais là effectivement j'aurai pu rebaser.

#10 Updated by Frédéric Péters 5 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF