[electrum] rename tests in test_bip21
ClosedPublic
Actions

Authored by PiRK on Jan 22 2024, 16:35.

Details

Reviewers

bytesofman

Group Reviewers

Restricted Project

Commits

rABC1253a4dd4881: [electrum] rename tests in test_bip21

Summary

No need to repeat parse_URI in every test name and helper function name, just put it in the class name.

Depends on D15234

Test Plan

python test_runner.py

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.Jan 22 2024, 16:35

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 22 2024, 16:35

PiRK requested review of this revision.Jan 22 2024, 16:35

Harbormaster completed remote builds in B26549: Diff 44443.Jan 22 2024, 16:36

PiRK added a child revision: D15236: [electrum] add tests for BIP21 parsing and creation.Jan 22 2024, 16:37

PiRK mentioned this in D15229: [electrum] clean and add tests for BIP21.Jan 22 2024, 16:42

bytesofman accepted this revision.Jan 22 2024, 18:02

This revision is now accepted and ready to land.Jan 22 2024, 18:02

Closed by commit rABC1253a4dd4881: [electrum] rename tests in test_bip21 (authored by PiRK). · Explain WhyJan 23 2024, 07:46

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC1253a4dd4881: [electrum] rename tests in test_bip21.

Revision Contents
Changeset List

Path

Size

electrum/

electrumabc/

tests/

test_bip21.py

62 lines

Diff 44471

View Options

electrum/electrumabc/tests/test_bip21.py

[electrum] rename tests in test_bip21ClosedPublicActions