[tests] Remove 'account' API from wallet_import_rescan
ClosedPublic
Actions

Authored by nakihito on Nov 23 2019, 02:31.

Details

Reviewers

deadalnix
Fabien
jasonbcox

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rSTAGINGb417afb630c6: [tests] Remove 'account' API from wallet_import_rescan
rABCb417afb630c6: [tests] Remove 'account' API from wallet_import_rescan

Summary

Removes usage of account API from the following functional tests:

wallet_import_rescan.py

Partial backport of Core PR13075
https://github.com/bitcoin/bitcoin/pull/13075/commits/5d536619abda745c298fd827a856df775f223241#diff-41ca09babe91abee6c9f1bf338709ecc

Test Plan

test_runner.py wallet_import_rescan

Diff Detail

Repository

rABC Bitcoin ABC

Branch

PR13075-2

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 8246
Build 14517: Default Diff Build & Tests
Build 14516: arc lint + arc unit

Event Timeline

nakihito created this revision.Nov 23 2019, 02:31

Owners added a reviewer: Restricted Owners Package.Nov 23 2019, 02:31

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 23 2019, 02:31

Build Bitcoin-ABC / Diffs / Diff Testing started.

nakihito edited the summary of this revision. (Show Details)Nov 23 2019, 02:32

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8246: Diff 14347.Nov 23 2019, 02:41

deadalnix accepted this revision.Nov 23 2019, 15:04

This revision is now accepted and ready to land.Nov 23 2019, 15:04

Closed by commit rABCb417afb630c6: [tests] Remove 'account' API from wallet_import_rescan (authored by nakihito). · Explain WhyNov 25 2019, 18:19

This revision was automatically updated to reflect the committed changes.

nakihito added a commit: rABCb417afb630c6: [tests] Remove 'account' API from wallet_import_rescan.

nakihito added a commit: rSTAGINGb417afb630c6: [tests] Remove 'account' API from wallet_import_rescan.Apr 7 2020, 21:49

Revision Contents
Changeset List

Path

Size

test/

functional/

wallet_import_rescan.py

48 lines

Diff 14347

View Options

test/functional/wallet_import_rescan.py

[tests] Remove 'account' API from wallet_import_rescanClosedPublicActions