Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1245)

Unified Diff: common/testing/assertions/grpc.go

Issue 2951393002: [errors] de-specialize Transient in favor of Tags. (Closed)
Patch Set: more refactor Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: common/testing/assertions/grpc.go
diff --git a/common/testing/assertions/grpc.go b/common/testing/assertions/grpc.go
index 4cdab07aa706a11776eb20426cfd316d91e75b7c..a3e30663ff81c8394331a6604e85681ccedc976b 100644
--- a/common/testing/assertions/grpc.go
+++ b/common/testing/assertions/grpc.go
@@ -7,6 +7,7 @@ package assertions
import (
"fmt"
+ "github.com/luci/luci-go/grpc/grpcutil"
"github.com/smartystreets/goconvey/convey"
"google.golang.org/grpc"
"google.golang.org/grpc/codes"
@@ -55,7 +56,7 @@ func ShouldHaveRPCCode(actual interface{}, expected ...interface{}) string {
return "Expected argument must have the form: [codes.Code[string]]"
}
- if acode := grpc.Code(aerr); acode != ecode {
+ if acode := grpcutil.Code(aerr); acode != ecode {
return fmt.Sprintf("expected gRPC code %q (%d), not %q (%d), type %T: %v",
ecode, ecode, acode, acode, actual, actual)
}

Powered by Google App Engine
This is Rietveld 408576698