Fix identities live tests

See here for discussion: https://forum.duniter.org/t/sanity-tests/10040/6

This MR is paused until there is a need to have this on gdev and that membership management is updated.

assigned to @HugoTrentesaux

added 1 commit

• 2041ff4f - wip add coherence test

Compare with previous version

added 1 commit

• cd65d4bd - wip add counter for owner key

Compare with previous version

added 1 commit

• 5731e0fb - wip add position of duplicate

Compare with previous version

added RN-silent T-end2endtests labels

changed the description

changed title from Draft: Fix identities live tests to Draft: (paused) Fix identities live tests

changed the description

added 73 commits

• 5731e0fb...b356db9a - 68 commits from branch master
• f909a7d0 - wip
• a7e19b54 - wip add coherence test
• b6c95d64 - wip add counter for owner key
• f129feab - wip add position of duplicate
• 87e60376 - update live tests

Compare with previous version

changed title from Draft: (paused) Fix identities live tests to Draft: Fix identities live tests

J'ai rebase sur master, et les live tests donnent actuellement une erreur :

st main ... FAILED

failures:

---- main stdout ----
Run sanity tests against ĞDev at last best block
accounts.len(): 34342.
identities.len(): 8434.
identity_index_of.len(): 8434.
Account 5CfdJjEgh3jDkg3bzmZ1ED1xVhXAARtNmZJWbcXh53rU8z5a has no random_id nor consumer.
Error: Storage corrupted: 1 errors.

@c-geek il y a donc un bug avec la migration d'identité au genesis. On peut corriger ce bug dans duniter ou se contenter de donner qq ĞD à ce compte :)

J'ai envoyé un DU_ĞD à ce compte au bloc 102158 et les tests passent à nouveau.

Peux-tu préciser quel est le bug si tu l'as identifié ?

Dans le genesis on pourrait supprimer le compte qui correspond à une old_owner_key puisque l'identité et les Ğ1 ont été migrées sur le nouveau compte. J'ai découvert ça en bossant sur duniter-squid, et c'est pour ça que je suis revenu mettre à jour cette MR.

OK je reformule :

1. Tu as lancé les live_tests, et une partie des tests vérifie qu'il n'existe pas de compte sans random_id ni consumer.
2. Cette partie des tests a échoué, voir sortie ci-dessus.
3. Le compte incriminé correspond à mon compte Ğ1v1 (cgeek) : c'est logique, puisque le compte a été migré au Genesis via le champ migration_address, le compte n'est plus utilisé pour rien.

J'ai consigné dans le ticket #146 (closed), pourrais-tu le faire la prochaine fois ? Sinon on risque de louper des sujets.

marked this merge request as ready

added 6 commits

• 335ab2cc - 1 commit from branch master
• 3011d206 - wip
• 8e16af47 - wip add coherence test
• 7a2df52a - wip add counter for owner key
• 9c346c55 - wip add position of duplicate
• babe1055 - update live tests

Compare with previous version

resolved all threads

merged

mentioned in commit cca4eb30

mentioned in issue #146 (closed)