Update policyestimator_tests to use COutPoint's accessor.
ClosedPublic
Actions

Authored by deadalnix on May 9 2018, 11:49.

Details

Reviewers

matiu

Group Reviewers

Restricted Project

Commits

rSTAGINGbed9b80836ff: Update policyestimator_tests to use COutPoint's accessor.
rABCbed9b80836ff: Update policyestimator_tests to use COutPoint's accessor.

Summary

Use nSequence field instead of changing the outpoint. This was done to generate a new TxId and never to have a valid outpoint to begin with, so it will do.

Depends on D1388 and D1385

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

policyestimatortxid

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2561
Build 3239: Bitcoin ABC Buildbot (legacy)
Build 3238: arc lint + arc unit

Event Timeline

deadalnix created this revision.May 9 2018, 11:49

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 9 2018, 11:49

Build Bitcoin-ABC :: Staging :: Bitcoin-ABC Staging started.

deadalnix edited the summary of this revision. (Show Details)May 9 2018, 11:54

Harbormaster completed remote builds in B2561: Diff 3847.May 9 2018, 12:13

Build Bitcoin-ABC :: Staging :: Bitcoin-ABC Staging passed.

deadalnix added a child revision: D1403: Make COutPoint's fields private.May 9 2018, 12:26

matiu accepted this revision.May 9 2018, 12:34

This revision is now accepted and ready to land.May 9 2018, 12:34

Closed by commit rABCbed9b80836ff: Update policyestimator_tests to use COutPoint's accessor. (authored by deadalnix). · Explain WhyMay 11 2018, 22:11

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

test/

policyestimator_tests.cpp

6 lines

Diff 3847

View Options

src/test/policyestimator_tests.cpp

Update policyestimator_tests to use COutPoint's accessor.ClosedPublicActions