Fix distance end2end tests

• Rendre explicite que le slash de distance prend dans le montant réservé et non libre : il existe plusieurs Balanced et plusieurs slash dans Substrate. Le code était bon mais semblait buggé à première vue. Le bug aurait aussi pu être introduit en changeant malencontreusement une contrainte de traits sur Config::Currency.
• Corriger le test Cucumber identity_creation, qui était en erreur mais ignoré à cause du fail_on_skip désactivé. Il s'agit surtout des changements entre session BABE et période ad-hoc qui n'étaient pas complets.
• Réactiver fail_on_skip : on ne peut plus écrire de commentaire implicite dans les cucumber features. Une ligne qui ne match pas une regex provoquera une erreur, ce qui permet de détecter des bugs rapidement.
• Ajouter un test Cucumber pour le défaut de distance sur une TdC pathologique : permet de tester le slash et la non-obtention du statut de membre.
• Gérer l'imbalance causée lors du slash de distance (fix #251 (closed)) et ajouter un test cucumber pour ça.

changed milestone to %runtime-802

added RN-runtime label

requested review from @HugoTrentesaux

added 4 commits

• 6dec13d1 - 1 commit from branch master
• 500a01f9 - Make it explicit that distance slashes from reserve
• 989a4d2e - Fix distance end2end tests
• 93297c6b - end2endtests: fail on skipped

Compare with previous version

added 2 commits

• b1ed3e18 - Fix distance end2end tests
• 0d851829 - end2endtests: fail on skipped

Compare with previous version

added 2 commits

• b99fd84f - distance: handle slash unbalance
• b02218c7 - cucumber: distance_fail treasury

Compare with previous version

changed the description

added 2 commits

• 1bd7f26e - distance: handle slash unbalance
• 19a74f26 - cucumber: distance_fail treasury

Compare with previous version

added 1 commit

• 46dfbb16 - Update metadata

Compare with previous version

approved this merge request

added 9 commits

• 46dfbb16...e26f94af - 4 commits from branch master
• dc3c9b03 - Make it explicit that distance slashes from reserve
• 5c2f2736 - Fix distance end2end tests
• 32179c78 - end2endtests: fail on skipped
• 78c6c744 - distance: handle slash unbalance
• d2ceed94 - cucumber: distance_fail treasury

Compare with previous version

Rebased on master.

Why 46dfbb16 "Update metadata"? They shouldn't have changed for gdev.

Je les générais avec la commande qui est dans la doc (end2endtests/readme), j'ai dû chercher dans le script du pipeline pour comprendre qu'il utilisait une autre commande et que ça faisait échouer le pipeline parce qu'il ne trouvait pas le même fichier metadata.

Je ne sais plus si j'avais prévu de mettre à jour la doc dans une MR future (et que j'avais un peu marre de cette MR) ou si j'ai juste oublié.

Ok, je vois deux possibilités :

• le --version 14 n'est plus d'actualité (perso je génère sans et on doit être en v15 des metadata par défaut j'imagine
• la compilation en mode --features constant-fees change quelque chose aux métadonnées

Pareil, je suis un peu perdu avec les scripts de pipelines qui sont long et peu lisibles.

J'ai un peu délaissé cette partie de la doc, j'imagine la reprendre au moment où de nouveaux contributeurs s'intéresseront à ces aspects, mais j'ai peu d'espoir que ça arrive prochainement, et certainement pas avant la migration.

À mon avis si la doc n'est pas systématiquement mise à jour en même temps que le code alors autant la supprimer.

Je l'utilise comme une référence (ce que ne peut pas être le blog/wiki, sauf pour les releases). Sans la doc, je devrais noter les commandes chez moi, et le jour où quelque chose est changé je découvrirai que ça ne marche plus, et passerai une semaine à chercher si c'est la faute à mes commandes, à un bug, et quelles sont les nouvelles commandes.

Donc pas forcément une doc très verbeuse, mais au moins une cheatsheet à jour.

merged

mentioned in commit 7fe42a23