Tests: only match error strings we create

I think matching error strings in tests is an anti-pattern. It's often fragile. We should only do it when we create the error string ourselves, and then still it's not preferable.

After this is merged we can deprecate the "error contains" argument of testhelper.AssertGrpcError.