test: added test for JSONRPC-objects #18394

pull brakmic wants to merge 1 commits into bitcoin:master from brakmic:expanding-rpc-test-coverage changing 1 files +61 −1
  1. brakmic commented at 6:53 PM on March 20, 2020: contributor

    This PR adds tests for JSONRPC objects: Request, Reply, Error and Batch.

    The idea for this and my other PR's is based on this issue: #16450

    As always, I'm using this website to search for missing coverages: https://marcofalke.github.io/btc_cov/test_bitcoin.coverage/src/rpc/request.cpp.gcov.html

  2. DrahtBot added the label Tests on Mar 20, 2020
  3. test: added test for JSONRPC-objects d0f5b012ac
  4. MarcoFalke commented at 12:23 AM on March 21, 2020: member

    The rpc code is tested by the functional tests. They used to be called "rpc-tests".

    See the combination of unit test and functional test coverage: https://marcofalke.github.io/btc_cov/total.coverage/src/rpc/request.cpp.gcov.html

    I think we should focus on the missing coverage that is missing from both unit and functional tests.

  5. brakmic commented at 12:25 AM on March 21, 2020: contributor

    The rpc code is tested by the functional tests. They used to be called "rpc-tests".

    See the combination of unit test and functional test coverage: https://marcofalke.github.io/btc_cov/total.coverage/src/rpc/request.cpp.gcov.html

    I think we should focus on the missing coverage that is missing from both unit and functional tests.

    Ok, many thanks. Will then close this PR.

  6. brakmic closed this on Mar 21, 2020
  7. DrahtBot locked this on Feb 15, 2022
Contributors
brakmicMarcoFalke
Labels
Tests
Linked (view graph)
#16450 Code with no test coverage sorted by priority?
github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 15:14 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me