Page MenuHomePhabricator
Differential D4520

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

Authored by nakihito on Sat, Nov 23, 02:15.

Details

Reviewers
deadalnix
Fabien
jasonbcox
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABCd5a5598bee32: [tests] Remove 'account' API from wallet_basic
Summary

Removes usage of account API from the following functional tests:

  • wallet_basic.py

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

Test Plan
test_runner.py wallet_basic

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nakihito created this revision.Sat, Nov 23, 02:15
Owners added a reviewer: Restricted Owners Package.Sat, Nov 23, 02:15
Herald added a reviewer: Restricted Project. · View Herald TranscriptSat, Nov 23, 02:15
deadalnix requested changes to this revision.Sat, Nov 23, 15:06
deadalnix added inline comments.
test/functional/wallet_basic.py
402 ↗(On Diff #14345)

Fix formatting

This revision now requires changes to proceed.Sat, Nov 23, 15:06
nakihito updated this revision to Diff 14389.Mon, Nov 25, 18:18

Fixed comment formatting.

deadalnix accepted this revision.Mon, Nov 25, 22:22
This revision is now accepted and ready to land.Mon, Nov 25, 22:22
Closed by commit rABCd5a5598bee32: [tests] Remove 'account' API from wallet_basic (authored by nakihito). · Explain WhyMon, Nov 25, 22:37
This revision was automatically updated to reflect the committed changes.
nakihito added a commit: rABCd5a5598bee32: [tests] Remove 'account' API from wallet_basic.

Revision Contents
Changeset List

PathPackages
Mtest/functional/wallet_basic.py (37 lines)

Diff 14411

View Options

test/functional/wallet_basic.py

Loading...