Merge #12996: tests: Remove redundant bytes(…) calls
ClosedPublic
Actions

Authored by nakihito on Aug 12 2019, 18:47.

Details

Reviewers

deadalnix
Fabien
jasonbcox

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rSTAGINGf6969d52800c: Merge #12996: tests: Remove redundant bytes(…) calls
rABCf6969d52800c: Merge #12996: tests: Remove redundant bytes(…) calls

Summary

9db48c5634 tests: Remove redundant bytes² (practicalswift)

Pull request description:

This is a follow-up to #12993. As @jnewbery noted `bytes()` is idempotent.

Tree-SHA512: 0eb25e0c2c46f9abaac30f964c5eb422bece1414c840a717d86794424294cb19d995a6db7c8df2a2f4ec84776b05274a637f2c111738f397051f510e57184752

Backport of Core PR12996
https://github.com/bitcoin/bitcoin/pull/12996/

Test Plan

make check
test_runner.py

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

nakihito created this revision.Aug 12 2019, 18:47

Owners added a reviewer: Restricted Owners Package.Aug 12 2019, 18:47

Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 12 2019, 18:47

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B7119: Diff 10735.Aug 12 2019, 18:58

jasonbcox accepted this revision.Aug 13 2019, 01:16

This revision is now accepted and ready to land.Aug 13 2019, 01:16

Closed by commit rABCf6969d52800c: Merge #12996: tests: Remove redundant bytes(…) calls (authored by MarcoFalke <falke.marco@gmail.com>, committed by nakihito). · Explain WhyAug 13 2019, 18:30

This revision was automatically updated to reflect the committed changes.

Nico Guiton <nico@bitframe.org> added a commit: rABCf6969d52800c: Merge #12996: tests: Remove redundant bytes(…) calls.

Nico Guiton <nico@bitframe.org> added a commit: rSTAGINGf6969d52800c: Merge #12996: tests: Remove redundant bytes(…) calls.Sep 19 2019, 01:48

Revision Contents
Changeset List

Path

Size

test/

functional/

test_framework/

script.py

2 lines

Diff 10767

View Options

test/functional/test_framework/script.py

Merge #12996: tests: Remove redundant bytes(…) callsClosedPublicActions