Change Details

This adds explicit tests for the returned HTTP status codes to interface_rpc.py (for error cases) and the HTTP JSON-RPC client in general for success. [[https://github.com/bitcoin/bitcoin/pull/15381 | PR15381]] brought up discussion about the HTTP status codes in general, and the general opinion was that the current choice may not be ideal but should not be changed to preserve compatibility with existing JSON-RPC clients. Thus it makes sense to actually test the current status to ensure this desired compatibility is not broken accidentally. This is a backport of Core [[https://github.com/bitcoin/bitcoin/pull/15495 | PR15495]]

PR description: > This adds explicit tests for the returned HTTP status codes to > interface_rpc.py (for error cases) and the HTTP JSON-RPC client in > general for success. > > [[https://github.com/bitcoin/bitcoin/pull/15381 | PR15381]] brought up discussion about the HTTP status codes in general, > and the general opinion was that the current choice may not be ideal > but should not be changed to preserve compatibility with existing > JSON-RPC clients. Thus it makes sense to actually test the current > status to ensure this desired compatibility is not broken accidentally. This is a backport of Core [[https://github.com/bitcoin/bitcoin/pull/15495 | PR15495]]

PR description: > This adds explicit tests for the returned HTTP status codes to > interface_rpc.py (for error cases) and the HTTP JSON-RPC client in general for success. [[https://github.com/bitcoin/bitcoin/pull/15381 | PR15381]] brought up discussion about the HTTP status codes in general,> general for success. and the general opinion was that the current choice may not be ideal> > [[https://github.com/bitcoin/bitcoin/pull/15381 | PR15381]] brought up discussion about the HTTP status codes in general, but should not be changed to preserve compatibility with existing> and the general opinion was that the current choice may not be ideal JSON-RPC clients. Thus it makes sense to actually test the current> but should not be changed to preserve compatibility with existing > JSON-RPC clients. Thus it makes sense to actually test the current > status to ensure this desired compatibility is not broken accidentally. This is a backport of Core [[https://github.com/bitcoin/bitcoin/pull/15495 | PR15495]]