Skip to content
This repository has been archived by the owner on Mar 1, 2019. It is now read-only.

Extend unit tests inductive model to work with sequential wallets #265

Open
1 task
KtorZ opened this issue Jan 28, 2019 · 0 comments
Open
1 task

Extend unit tests inductive model to work with sequential wallets #265

KtorZ opened this issue Jan 28, 2019 · 0 comments
Labels
ACCEPTED ADR has been accepted

Comments

@KtorZ
Copy link
Contributor

KtorZ commented Jan 28, 2019

Context

We need to extend or modify our test suite to not only test against random wallets, but also support testing against sequential wallets. This may ripple quite far in the testing code and the overall scope for this is still unclear.

Decision

Part of this ticket is about figuring out what needs to be done and have a clear understanding of the testing "evolutions" regarding sequential wallets (reminder: estimates can be adjusted when writing a development plan).
Then, we'd like ideally to have sequential wallets tested as part of our unit tests, and be on-par for coverage with current random wallets (when it makes sense).

Acceptance Criterias

  • unit tests must cover wallets using sequential derivation

Development Plan

PR

Number Base
#? develop

QA

Criteria Coverage
? -

Retrospective

@KtorZ KtorZ added this to the Address Derivation à la BIP-44 milestone Jan 28, 2019
@KtorZ KtorZ changed the title Extend unit tests inductive model to work with EOS wallets Extend unit tests inductive model to work with sequential wallets Jan 28, 2019
@KtorZ KtorZ added the ACCEPTED ADR has been accepted label Jan 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ACCEPTED ADR has been accepted
Projects
None yet
Development

No branches or pull requests

1 participant