Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add coverage for unknown network in -onlynet #25173

Merged
merged 1 commit into from
May 20, 2022

Conversation

brunoerg
Copy link
Contributor

This PR adds test coverage for the following init error by passing an unknown network in -onlynet

return InitError(strprintf(_("Unknown network specified in -onlynet: '%s'"), snet));

@jonatack
Copy link
Member

jonatack commented May 19, 2022

Concept ACK, this case looks to be untested.

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK e9d32dd

test/functional/feature_proxy.py Outdated Show resolved Hide resolved
@DrahtBot DrahtBot added the Tests label May 19, 2022
@maflcko
Copy link
Member

maflcko commented May 20, 2022

rACK 055d94d

@maflcko maflcko merged commit d433f59 into bitcoin:master May 20, 2022
@jonatack
Copy link
Member

Post-merge re-ACK

sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 28, 2022
055d94d test: add coverage for unknown network in -onlynet (brunoerg)

Pull request description:

  This PR adds test coverage for the following init error by passing an unknown network in -onlynet
  https://github.com/bitcoin/bitcoin/blob/0de36941eca1bff91420dd878eb097db2b1a596c/src/init.cpp#L1311

ACKs for top commit:
  MarcoFalke:
    rACK 055d94d

Tree-SHA512: 01bbb297afff371f6345889fa04117ff195b68f0bbf934878ba446049791fdbd7d2ce119ee4f9b3616cc0a81330d7055507dc81151acf68532c077f3575258e9
@bitcoin bitcoin locked and limited conversation to collaborators May 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants