Skip to content
Snippets Groups Projects

Distance Oracle

Merged Pascal Engélibert requested to merge distance into master

close #10 (closed)

Edited by Pascal Engélibert

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Cédric Moreau
  • 1ère passe.

    • Resolved by Pascal Engélibert

      Comment est généré le fichier resources/metadata.scale ? Ce serait bien de l'écrire dans le fichier resources/README.md.

      Car quand j'essaye de lancer cargo cucumber -i 'distance*' j'obtiens :

         ✘  When alice requests distance evaluation for dave
            Step failed: ./cucumber-features/distance.feature:13:9
            Captured output: 
            Step panicked. Captured output: Metadata: Pallet Distance Call Distance has incompatible metadata

      Je suppose que c'est à cause de ce fichier.

  • Merci pour le commit wip, je regarde ce que je peux faire. Notamment j'aimerais voir si je peux rapatrier les tests présents sur ma branche et les migrer dans distance-oracle, ça permettrait d'être confiant dans la non-régression.

    • Resolved by Pascal Engélibert

      @tuxmain : la variante DistanceStatus::Pending n'est pas exploitée actuellement, pourtant il y a bien des appels à IdentitiesDistanceStatus::<T, I>::remove, notamment un suivi de <T as Config<I>>::Currency::slash_reserved laissant penser que cette variante y était attendue.

      Est-ce le cas ? J'aurais bien vu l'instruction IdentitiesDistanceStatus::<T, I>::insert(idty_index, DistanceStatus::Pending); positionnée juste avant le retour Ok(()) dans do_evaluate_distance.

      Edited by Cédric Moreau
  • added 7 commits

    Compare with previous version

  • added 1 commit

    • af6ef24b - fixes & working end2end tests

    Compare with previous version

  • added 1 commit

    • 4cca3fd7 - integration tests, expiration

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Pascal Engélibert resolved all threads

    resolved all threads

  • added 9 commits

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading