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

Make it possible to test syncing wallets in the integration tests #285

Open
piotr-iohk opened this issue Jan 30, 2019 · 3 comments
Open
Labels
EPIC Epic Story: keeps track of smaller ones

Comments

@piotr-iohk
Copy link
Contributor

Context

From the user standpoint wallet can be in two states -> syncing and synced.
Regardless of the state of the wallet there are certain operations that can be performed on it. These are:

  • updating name and assurance level
  • updating password
  • deleting wallet
  • getting wallet details
  • (possibly others)

We should be able to test that, ideally in the integration tests.

Unfortunately at this point it is not possible to achieve that in the integration tests as We do keep the last 2160 blocks in memory, so restoring on a blockchain that is shorter than that will be instantaneous
cf. -> cardano-foundation/cardano-wallet#274 (comment)

This request is about providing such possibility for the local cluster so we are able to add such tests for the wallet. The idea from the comment above -> start the cluster on a "pre-constructed" DB dump

Decision

Make it possible to test wallet in the syncing state

Acceptance Criterias


Development Plan

PR

Number Base
#? develop

QA

Criteria Coverage
? -

Retrospective

@KtorZ
Copy link
Contributor

KtorZ commented Jan 31, 2019

The ticket is definitely a good idea and, as we discussed during the weekly, it may be -- in its current form -- too big of a task. Therefore, we should find a way to split that across multiple smaller tickets.

@piotr-iohk
Copy link
Contributor Author

Maybe it makes sense then to make this one an epic?

@KtorZ
Copy link
Contributor

KtorZ commented Feb 4, 2019

As a matter of fact, it does. 👍

@piotr-iohk piotr-iohk added EPIC Epic Story: keeps track of smaller ones and removed PROPOSED ADR just proposed labels Feb 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
EPIC Epic Story: keeps track of smaller ones
Projects
None yet
Development

No branches or pull requests

2 participants