input-output-hk / cardano-wallet-legacy

Official Wallet Backend & API for Cardano-SL
https://input-output-hk.github.io/cardano-wallet/
MIT License
21 stars 12 forks source link

Make EO endpoints to manipulate EO wallets and FO to manipulate FO wallets only #336

Open piotr-iohk opened 5 years ago

piotr-iohk commented 5 years ago

Context

While testing #331 we have observed that one can manipulate EOS wallets with FO endpoints and FO wallets with EOS endpoints. In particular:

There were integration tests added for that under #331, they are currently failing:

  EOSWALLETS_DELETE_02 - cannot delete FO wallet with EOS endpoint FAILED [1]
  EOSWALLETS_DELETE_02 - cannot delete EOS wallet with FO endpoint FAILED [2]
  EOSWALLETS_DETAILS_01 - cannot get EOS wallet with FO endpoint FAILED [3]
  EOSWALLETS_LIST_01 - cannot get EOS wallets with FO endpoint FAILED [4]
  EOSWALLETS_LIST_01 - cannot get FO wallets with EOS endpoint FAILED [5]
  EOSWALLETS_UPDATE_01 - cannot update FO wallets with EOS endpoint FAILED [6]
  EOSWALLETS_UPDATE_01 - cannot update EOS wallets with FO endpoint FAILED [7]

Decision

Manipulation with EO wallets must be restricted only to EO endpoints. Manipulation with FO wallets must be restricted only to FO endpoints.

Acceptance Criterias

See ^. Integration tests mentioned here should pass.


Development Plan

PR

Number Base
#335 develop
#337 develop

QA

Criteria Coverage
? -

Retrospective

denisshevchenko commented 5 years ago

All tests are passed now, except this one: EOSWALLETS_LIST_01 - cannot get FO wallets with EOS endpoint FAILED [5]

piotr-iohk commented 5 years ago

This test fails due to the fact Eos wallets list endpoint fails with error WalletNotFound, which still needs to be fixed.