New test for another error case in DepositScriptAddress

On 14/09/14 22:08, Jimmy Song wrote:
> In waddrmgr/internal_test.go:
> 
>> @@ -186,3 +186,12 @@ func TestDecryptExtendedKeyCannotCreateResultKey(t *testing.T) {
>>  		}
>>  	}
>>  }
>> +
>> +// Replace Manager.cryptoKeyScript with the given one and calls the given function,
>> +// resetting Manager.cryptoKeyScript to its original value after that.
>> +func RunWithReplacedCryptoKeyScript(mgr *Manager, cryptoKey EncryptorDecryptor, f func()) {
> 
> I would call the function "callback" or something similar. f is just not
> a very descriptive name.

Indeed, f is a horrible name if taken out of context, but when you see
it together with its type (which is always the case), it should be
pretty obvious. I don't really care either way, though, so I'll rename
it to callback