The tests are to ensure that vspd errors are correctly wrapped as types.ErrorResponse, and non-vspd errors contain adequate information for debugging. A small change in the client code itself was required to satisfy these conditions.
The constructor contained no important logic and removing it makes the upcoming test code slightly more clean/convenient.
* Add vspd client from dcrwallet. * client: Extract and export ValidateServerSignature * client: Add helper funcs. * client: Add trace logging. * client: Use existing error types. * Add comments to explain error handling logic.