astelm/grpcurl
1
0
Fork 0
mirror of https://github.com/fullstorydev/grpcurl.git synced 2026-05-22 19:51:44 +03:00
Code Issues Packages Projects Releases Wiki Activity

Update protoreflect v1.15.2 and grpc v1.57.0 (#406)

Browse Source 
The proposal already exists by Dependabot, but the version bump pulls
in `grpc.reflection.v1.ServerReflection` resulting in minor test change
being required.

Transient dependency `github.com/bufbuild/protocompile` is now added,
and requires at least Go 1.18. This PR includes the version removal.
This commit is contained in:
Valters Jansons
2023-08-31 19:14:03 +03:00
committed by GitHub
parent 2a29c1e64b
commit b7a5d3bba8
4 changed files with 23 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
grpcurl_test.go
View File

@@ -177,7 +177,7 @@ func doTestListServices(t *testing.T, source DescriptorSource, includeReflection
var expected []string
if includeReflection {
// when using server reflection, we see the TestService as well as the ServerReflection service
expected = []string{"grpc.reflection.v1alpha.ServerReflection", "testing.TestService"}
expected = []string{"grpc.reflection.v1.ServerReflection", "grpc.reflection.v1alpha.ServerReflection", "testing.TestService"}
} else {
// without reflection, we see all services defined in the same test.proto file, which is the
// TestService as well as UnimplementedService
@@ -243,7 +243,7 @@ func doTestListMethods(t *testing.T, source DescriptorSource, includeReflection
func TestGetAllFiles(t *testing.T) {
expectedFiles := []string{"test.proto"}
expectedFilesWithReflection := []string{
"grpc/reflection/v1alpha/reflection.proto", "test.proto",
"grpc/reflection/v1/reflection.proto", "grpc/reflection/v1alpha/reflection.proto", "test.proto",
}
for _, ds := range descSources {