46ce223d1 Add tests for CMerkleBlock usage with txids specified (James O'Beirne)
Pull request description:
What started as a simple task to add test coverage ended up giving way to a light refactoring. This consolidates the mostly-identical `CMerkleBlock` constructors into one (using C++11 constructor delegation) and adds coverage for the by-txids construction case. ### Before ![selection_006](https://user-images.githubusercontent.com/73197/30242104-0f381fe4-9545-11e7-9617-83b87fce0456.png) ### After ![selection_008](https://user-images.githubusercontent.com/73197/30242107-1425dfaa-9545-11e7-9e6b-2c3432517dd1.png)
Changes made in D3371 have decided the issue of code ownership. In addition, the changes to the constructor make the code both more difficult to read and harder to maintain without providing any real benefit. Because of these factors, I skipped the changes to the constructor and only include the changes/additions to tests.
Partial backport of Core PR11293 (only including test changes)
Also includes left over changes to PR12920