Add a make command for checking expected failures

Auto merge of #1411 - bitcartel:master_bitcoin_7106, r=daira

Upstream patch: Fix and improve relay from whitelisted peers

https://github.com/bitcoin/bitcoin/pull/7106
a9f3d3db5c0c8d1697998ed9b3e192ddbf9a31f4

An extra commit modifies the log message string, otherwise there are are a number of commits that need be to backported to add methods e.g. GetDebugMessage.  These commits modify the interface in consensus/validation.h so there are conflicts to be resolved. e.g.
9003c7c
a9ac95c
5f12263
fbf44e6

Auto merge of #1407 - bitcartel:master_bitcoin_7079, r=daira

Upstream patch: Prevent peer flooding inv request queue

https://github.com/bitcoin/bitcoin/pull/7079
5029698186445bf3cd69d0e720f019c472661bff
ebb25f4c23adbcb55796c402bafd6064a136f16f

Benchmark 50 iterations of solveequihash

Since the parameters changed in z8, the benchmark on speed.z.cash is showing misleading results due to variability. (The quartile and extrema bars will still show the variability with 50 runs, they just won't jump around as much between benchmark data points.)

Replace %i format specifier with more commonly used %d.

Link to beta guide

Modify message string so we don't need to backport commits which implement FormatStateMessage and GetDebugMessage and involve changes to consensus/validation.h

Fix and improve relay from whitelisted peers

This makes sure that retransmits by a whitelisted peer also actually
result in a retransmit.

Further, this changes the logic to never relay in case we would assign
a DoS score, as we expect to get DoS banned ourselves as a result.

Limit setAskFor and retire requested entries only when a getdata returns.

The setAskFor duplicate elimination was too eager and removed entries
 when we still had no getdata response, allowing the peer to keep
 INVing and not responding.

prevent peer flooding request queue for an inv

mapAlreadyAskedFor does not keep track of which peer has a request queued for a
particular tx. As a result, a peer can blind a node to a tx indefinitely by
sending many invs for the same tx, and then never replying to getdatas for it.
Each inv received will be placed 2 minutes farther back in mapAlreadyAskedFor,
so a short message containing 10 invs would render that tx unavailable for 20
minutes.

This is fixed by disallowing a peer from having more than one entry for a
particular inv in mapAlreadyAskedFor at a time.

Updating wording to match Beta Guide

Typo in params README

Closes #1371 by updating signed message

Add release notes for 1.0.0-beta1.

Update pchMessageStart and add testnet DNS boostrapper.

Change version to 1.0.0. This is just a beta.

Remove .z# suffix from version

Replace GetTxid() with GetHash()

Improve check that user supplied memo field is too long.

Remove line of commented out code we don't need

Update comment with ticket issue number

Add assert

Fix formatting

Use zcash constants

Improve error reporting when attempting to spend coinbase utxos.

Add assert for two mutually exclusive member variables.

Add ticket number to issues raised in comment.

Remove unused varible.

Add lock for member variables.
Clean up and clarify that id_ and creation_time_ are never to be mutated anywhere.
Fix incomplete copy/assignment constructors.

Formatting and updated test per review.

Fix incorrect default value for argument of GetFilteredNotes.

Update log statement to include fee.

Fix formatting

Add test for calling RPC z_getbalance, z_gettotalbalance, z_listreceivedbyaddress with invalid parameters.

Add extra RPC parameter checks for minconf<0 and zaddr not belonging to wallet.

Replace unique_lock with lock_guard, where appropriate, for consistency

Add lock guard to getNumberOfWorkers()

Fix formatting

Move lock guard to start of addOperation to protect isClosed() and isFinishing()

Update formatting and documentation.

Add 'DEPRECATED' to help message of zcraw* commands

Add z_listreceivedbyaddress RPC call

Rename GetUnspentNotes to GetFilteredNotes

Added parameter to ignore spent notes, which is true by default.

Disable z_sendmany in safe mode

Add more logging.

Refactor async sendmany and getbalance calls to use GetUnspentNotes().

Add test for GetUnspentNotes() in wallet.

Add GetUnspentNotes to wallet.

Add tests to try and improve coverage of perform_joinsplit.

Replace zcashParams_ with global.

Revert "Reduce use of global pzcashParams with private member variable"

This reverts commit 4d71ba58d37eb4f93d68b5e437086ce14fb24825.

Reduce use of global pzcashParams with private member variable

Disable proof generation when testmode is enabled in async SendMany operation.

Fix typo in error message

Add z_getbalance and z_gettotalbalance RPC calls to close #1201.

Add friend class for testing private members of AsyncRPCOperation_sendmany.

Add extra checking of memo data in hexadecimal string format.

Add logging under the category "asyncrpc".

Remove redundant call.

Add tests for async queue and rpc commands: z_getoperationstatus, z_getoperationresult, z_listoperationids, z_sendmany

Remove redundant check when getting spending key for a payment address.

Update RPCServer to use AsyncRPCQueue's shared queue.

Add shared queue to AsynRPCQueue.

Add test for AsyncRPCQueue and AsyncRPCOperation.

Clear the operation queue when closing it.

Refactor to use wallet note tracking from commit a72379

Fix bug where wallet was not persisting witnesses to disk.
Author: str4d

Added option to close a queue and wait for queued up operations to finish,
rather than just closing a queue and immediately cancelling all operations.

Fix bug where call to sign and send a transaction was in wrong scope.

Coinbase utxos can only be spent when sending to a single zaddr.
Change from the transaction will be sent to the same zaddr.

Disable option to allow multiple async rpc workers.

Add chaining of JoinSplits within a transaction.

Closes #1293 by adding z_getoperationresult and making z_getoperationstatus idempotent.

Refactoring and small improvements to async rpc operations.

Added AsyncRPCQueue::closeAndWait() so rpcserver can block on worker threads when shutting down.
AsyncRPCOperation is no longer copyable - copy constructor and assignment operators now private.
Refactoring: renamed methods, renamed member variables
Tidy up: comments, const, size_t, braces

Implement RPC call z_listoperationids and update z_getoperationstatus to take a list parameter.

Add public method to AsycnRPCQueue to retrieve all the known operation ids.

Add public method to get state as a human readable string from an AsyncRPCOperation.

Successful result of z_sendmany returns txid so it doesn't need to return raw hex.

z_sendmany from a taddr now routes change to a new address instead of back to the sender's taddr,

Update find_unspent_notes() as mapNoteAddrs_t has been replaced by mapNoteData_t.

Implement z_sendmany RPC call.

Simple implementation does not try to optimize coin or note selection.
Caller can send from a taddr or zaddr to multiple recipients.
Currently only one of the recipients can be a zaddr.

Add public field 'memo' to JSOutput to enable creation of notes with custom memos.

Add config option 'rpcasyncthreads' to specify number of async rpc workers. Default is 1.

Add prefix to async operation id so it is easier to manage on cli.

Add async RPC queue and operation classes.
Add z_getoperationstatus RPC command.
Add z_sendmany RPC command (dummy implementation, does not send actual
coins).

Add tests for refactored wallet code

Refactor wallet note code for testing

Auto merge of #1338 - str4d:147-tweak-difficulty-adjustment-algorithm, r=ebfull

Tweaks to difficulty adjustment algorithm

This PR changes the difficulty algorithm to adjust from the average difficulty over the
block window instead of from the last difficulty. It also removes the special rules for the
testnet, which are incompatible with difficulty averaging.

Closes #147 again.

Auto merge of #1355 - bitcartel:zc.v0.11.2.latest_paymentapi, r=ebfull

Update payment API documentation for beta 1

Remove 'not in alpha 9' labels as RPC calls are now implemented.

Adjust test to account for integer division precision loss

Remove unnecessary method

Update payment API documentation for beta 1

Adjust test to avoid spurious failures

Previously, if the two random numbers happened to be equal, the block time
would not be updated, and subsequent checks would fail.

Replace GetTxid() with GetHash() after rebase on latest.

Remove GetTxid() from CTransaction and update test_txid

Refactor: replace calls to GetTxid() with GetHash()

Update txid gtest to verify #1144 has been removed: GetTxid() and
GetHash() return the same result.

Remove #1144 from input data of script_tests.

Revert script_invalid.json to commit df1609f.
Revert script_valid.json to commit 1c54757.

Remove #1144 from bloom_tests by reverting to commit 5012190.

Remove #1144 from transaction.cpp by reverting back to commit 942bc46.