test: servicesnames field in getpeerinfo and getnetworkinfo#16850
Merged
laanwj merged 2 commits intobitcoin:masterfrom Sep 12, 2019
Merged
test: servicesnames field in getpeerinfo and getnetworkinfo#16850laanwj merged 2 commits intobitcoin:masterfrom
servicesnames field in getpeerinfo and getnetworkinfo#16850laanwj merged 2 commits intobitcoin:masterfrom
Conversation
servicesnames field in getpeerinfo and getnetworkinfoservicesnames field in getpeerinfo and getnetworkinfo
Member
Author
|
Could someone restart the stopped job (583339836) please ? |
servicesnames field in getpeerinfo and getnetworkinfoservicesnames field in getpeerinfo and getnetworkinfo
jamesob
reviewed
Sep 11, 2019
Contributor
jamesob
left a comment
There was a problem hiding this comment.
Concept ACK, thanks for writing tests.
In getpeerinfo and getnetworkinfo
Since it's the name of the RPC call
2791ed0 to
1d524c6
Compare
Member
Author
|
Rebased to regroup into a function as suggested by @MarcoFalke. |
Member
|
ACK 1d524c6 |
laanwj
added a commit
that referenced
this pull request
Sep 12, 2019
…tworkinfo` 1d524c6 tests: rename 'test_getnetworkinginfo' in 'test_getnetworkinfo' (darosior) 07a8f65 tests: add a test for the 'servicesnames' RPC field (darosior) Pull request description: As per #16787 (comment), fixes #16844. This adds a test for both commands in the first commit and renames the test for `getnetworkinfo` in the second commit. ACKs for top commit: laanwj: ACK 1d524c6 Tree-SHA512: 8267dce4d54356debab75014e6f9ba885b892da605ed32f26a5446c232992fcae761861bb678adbdb942815d4706f3768c70deee6afec68f219b23605475be01
luke-jr
pushed a commit
to bitcoinknots/bitcoin
that referenced
this pull request
Sep 21, 2019
In getpeerinfo and getnetworkinfo Github-Pull: bitcoin#16850 Rebased-From: 07a8f65
jasonbcox
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Oct 16, 2020
Summary: This adds a test for both commands in the first commit and renames the test for getnetworkinfo. Depends on D7948 This is a backport of Core [[bitcoin/bitcoin#16850 | PR16850]] Test Plan: `ninja && test/functional/test_runner.py rpc_net.py` Reviewers: O1 Bitcoin ABC, #bitcoin_abc, Fabien Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien Differential Revision: https://reviews.bitcoinabc.org/D7949
jasonbcox
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Oct 23, 2020
Summary: The tests added in [[bitcoin/bitcoin#16850 | PR16850]] assumed that the field `localservices` returned by `getnetworkinfo` was a string representation of an integer, when it is really a string representation of a hex. This fixes the test to decode the field properly and do the correct bitwise comparisons. This is a backport of Core [[bitcoin/bitcoin#16991 | PR16991]] Test Plan: `ninja && ./test/functional/test_runner.py rpc_net.py` ``` $ bitcoin-cli getnetworkinfo ... "localservices": "0000000000000425", "localservicesnames": [ "NETWORK", "BLOOM", "BITCOIN_CASH", "NETWORK_LIMITED" ], ... $ python >>> int("425", 16) 1061 >>> NODE_NETWORK = (1 << 0) >>> NODE_BLOOM = (1 << 2) >>> NODE_BITCOIN_CASH = (1 << 5) >>> NODE_NETWORK_LIMITED = (1 << 10) >>> NODE_NETWORK | NODE_BLOOM | NODE_BITCOIN_CASH | NODE_NETWORK_LIMITED 1061 >>> ``` Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix, Fabien Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, deadalnix, Fabien Differential Revision: https://reviews.bitcoinabc.org/D8083
kwvg
added a commit
to kwvg/dash
that referenced
this pull request
Dec 12, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
As per #16787 (comment), fixes #16844.
This adds a test for both commands in the first commit and renames the test for
getnetworkinfoin the second commit.