Page MenuHomePhabricator

use f-strings in functional tests (long lines)
ClosedPublic

Authored by PiRK on Mar 1 2023, 09:27.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC29fa685df764: use f-strings in functional tests (long lines)
Summary

These lines were ignored by flynt in previous scripted-diffs because of the multiline nature.

I manually edited these lines to put them on a single line, ran flynt again, then manually formatted the resulting string.

There are more of these lines in test_runner.py, in test_framework/*.py files, in src/test/fuzz/test_runner.py and in test/util/bitcoin-util-test.py. These will be addressed in a follow-up to avoid making this diff even larger.

Depends on D13202

Test Plan

ninja check-functional-extended

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

PiRK requested review of this revision.Mar 1 2023, 09:27
test/functional/interface_usdt_net.py
41 ↗(On Diff #38158)

curly braces need to be doubled in f-strings to be escaped

add a few lines I previously missed

This revision is now accepted and ready to land.Mar 1 2023, 11:02
This revision was automatically updated to reflect the committed changes.