Skip to content

rpcserver: Use solved mock orphan block in tests.#3104

Merged
davecgh merged 1 commit intodecred:masterfrom
davecgh:rpcserver_solve_mock_orphan_block
Apr 26, 2023
Merged

rpcserver: Use solved mock orphan block in tests.#3104
davecgh merged 1 commit intodecred:masterfrom
davecgh:rpcserver_solve_mock_orphan_block

Conversation

@davecgh
Copy link
Copy Markdown
Member

@davecgh davecgh commented Apr 24, 2023

Currently, the RPC server getwork test for orphan block submission is rather brittle because it relies on the specific current implementation of the handler.

This makes the test more robust by modifying it to solve the mock orphan block submission so that it reliably tests what it intends to even if the handler code changes.

@davecgh davecgh added the test coverage Discussion and pull requests for improving test coverage. label Apr 24, 2023
@davecgh davecgh added this to the 1.8.0 milestone Apr 24, 2023
Currently, the RPC server getwork test for orphan block submission is
rather brittle because it relies on the specific current implementation
of the handler.

This makes the test more robust by modifying it to solve the mock orphan
block submission so that it reliably tests what it intends to even if
the handler code changes.
@davecgh davecgh force-pushed the rpcserver_solve_mock_orphan_block branch from dc9c60e to fdf5ca9 Compare April 24, 2023 18:59
@davecgh davecgh merged commit fdf5ca9 into decred:master Apr 26, 2023
@davecgh davecgh deleted the rpcserver_solve_mock_orphan_block branch April 26, 2023 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test coverage Discussion and pull requests for improving test coverage.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants