Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deprecated dependencies upgraded #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ See the [examples](examples/) directory for uh...examples.
package main

import (
"github.com/golang/protobuf/proto"
"github.com/golang/protobuf/protoc-gen-go/plugin"
"google.golang.org/protobuf/proto"
plugin_go "google.golang.org/protobuf/types/pluginpb"
"github.com/pseudomuto/protokit"
_ "google.golang.org/genproto/googleapis/api/annotations" // Support (google.api.http) option (from google/api/annotations.proto).

Expand Down
4 changes: 2 additions & 2 deletions comments.go
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
package protokit

import (
"github.com/golang/protobuf/protoc-gen-go/descriptor"

"bytes"
"strconv"
"strings"

descriptor "google.golang.org/protobuf/types/descriptorpb"
)

// A Comment describes the leading, trailing, and detached comments for a proto object. See `SourceCodeInfo_Location` in
Expand Down
9 changes: 5 additions & 4 deletions example_plugin_test.go
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
package protokit_test

import (
"github.com/golang/protobuf/proto"
"github.com/golang/protobuf/protoc-gen-go/plugin"
"github.com/pseudomuto/protokit"

"log"

"google.golang.org/protobuf/proto"
plugin_go "google.golang.org/protobuf/types/pluginpb"

"github.com/pseudomuto/protokit"
)

type plugin struct{}
Expand Down
11 changes: 6 additions & 5 deletions examples/jsonator/main.go
Original file line number Diff line number Diff line change
@@ -1,15 +1,16 @@
package main

import (
"github.com/golang/protobuf/proto"
"github.com/golang/protobuf/protoc-gen-go/plugin"
"github.com/pseudomuto/protokit"
"google.golang.org/genproto/googleapis/api/annotations"

"bytes"
"encoding/json"
"fmt"
"log"

"google.golang.org/genproto/googleapis/api/annotations"
"google.golang.org/protobuf/proto"
plugin_go "google.golang.org/protobuf/types/pluginpb"

"github.com/pseudomuto/protokit"
)

func main() {
Expand Down
Binary file modified fixtures/fileset.pb
Binary file not shown.
12 changes: 6 additions & 6 deletions go.mod
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
module github.com/pseudomuto/protokit

go 1.17
go 1.22.5

require (
github.com/golang/protobuf v1.0.0
github.com/stretchr/testify v1.2.1
google.golang.org/genproto v0.0.0-20180427144745-86e600f69ee4
github.com/stretchr/testify v1.9.0
google.golang.org/genproto/googleapis/api v0.0.0-20240924160255-9d4c2d233b61
google.golang.org/protobuf v1.34.2
)

require (
github.com/davecgh/go-spew v1.1.0 // indirect
github.com/davecgh/go-spew v1.1.1 // indirect
github.com/pmezard/go-difflib v1.0.0 // indirect
golang.org/x/sync v0.0.0-20210220032951-036812b2e83c // indirect
gopkg.in/yaml.v3 v3.0.1 // indirect
)
26 changes: 16 additions & 10 deletions go.sum
Original file line number Diff line number Diff line change
@@ -1,12 +1,18 @@
github.com/davecgh/go-spew v1.1.0 h1:ZDRjVQ15GmhC3fiQ8ni8+OwkZQO4DARzQgrnXU1Liz8=
github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
github.com/golang/protobuf v1.0.0 h1:lsek0oXi8iFE9L+EXARyHIjU5rlWIhhTkjDz3vHhWWQ=
github.com/golang/protobuf v1.0.0/go.mod h1:6lQm79b+lXiMfvg/cZm0SGofjICqVBUtrP5yJMmIC1U=
github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
github.com/google/go-cmp v0.5.5 h1:Khx7svrCpmxxtHBq5j2mp/xVjsi8hQMfNLvJFAlrGgU=
github.com/google/go-cmp v0.5.5/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE=
github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
github.com/stretchr/testify v1.2.1 h1:52QO5WkIUcHGIR7EnGagH88x1bUzqGXTC5/1bDTUQ7U=
github.com/stretchr/testify v1.2.1/go.mod h1:a8OnRcib4nhh0OaRAV+Yts87kKdq0PP7pXfy6kDkUVs=
golang.org/x/sync v0.0.0-20210220032951-036812b2e83c h1:5KslGYwFpkhGh+Q16bwMP3cOontH8FOep7tGV86Y7SQ=
golang.org/x/sync v0.0.0-20210220032951-036812b2e83c/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
google.golang.org/genproto v0.0.0-20180427144745-86e600f69ee4 h1:0rk3/gV3HbvCeUzVMhdxV3TEVKMVPDnayjN7sYRmcxY=
google.golang.org/genproto v0.0.0-20180427144745-86e600f69ee4/go.mod h1:JiN7NxoALGmiZfu7CAH4rXhgtRTLTxftemlI0sWmxmc=
github.com/stretchr/testify v1.9.0 h1:HtqpIVDClZ4nwg75+f6Lvsy/wHu+3BoSGCbBAcpTsTg=
github.com/stretchr/testify v1.9.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
golang.org/x/xerrors v0.0.0-20191204190536-9bdfabe68543 h1:E7g+9GITq07hpfrRu66IVDexMakfv52eLZ2CXBWiKr4=
golang.org/x/xerrors v0.0.0-20191204190536-9bdfabe68543/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=
google.golang.org/genproto/googleapis/api v0.0.0-20240924160255-9d4c2d233b61 h1:pAjq8XSSzXoP9ya73v/w+9QEAAJNluLrpmMq5qFJQNY=
google.golang.org/genproto/googleapis/api v0.0.0-20240924160255-9d4c2d233b61/go.mod h1:O6rP0uBq4k0mdi/b4ZEMAZjkhYWhS815kCvaMha4VN8=
google.golang.org/protobuf v1.34.2 h1:6xV6lTsCfpGD21XK49h7MhtcApnLqkfYgPcdHftf6hg=
google.golang.org/protobuf v1.34.2/go.mod h1:qYOHts0dSfpeUzUFpOMr/WGzszTmLH+DiWniOlNbLDw=
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
14 changes: 7 additions & 7 deletions parser.go
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
package protokit

import (
"github.com/golang/protobuf/protoc-gen-go/descriptor"
"github.com/golang/protobuf/protoc-gen-go/plugin"

"context"
"fmt"
"strings"

descriptor "google.golang.org/protobuf/types/descriptorpb"
plugin_go "google.golang.org/protobuf/types/pluginpb"
)

const (
Expand Down Expand Up @@ -36,7 +36,7 @@ const (
//
// For example, given the following invocation, only booking.proto will be returned even if it imports other protos:
//
// protoc --plugin=protoc-gen-test=./test -I. protos/booking.proto
// protoc --plugin=protoc-gen-test=./test -I. protos/booking.proto
func ParseCodeGenRequest(req *plugin_go.CodeGeneratorRequest) []*FileDescriptor {
allFiles := make(map[string]*FileDescriptor)
genFiles := make([]*FileDescriptor, len(req.GetFileToGenerate()))
Expand Down Expand Up @@ -117,10 +117,10 @@ func parseEnumValues(ctx context.Context, protos []*descriptor.EnumValueDescript
longName := fmt.Sprintf("%s.%s", enum.GetLongName(), vd.GetName())

values[i] = &EnumValueDescriptor{
common: newCommon(file, "", longName),
common: newCommon(file, "", longName),
EnumValueDescriptorProto: vd,
Enum: enum,
Comments: file.comments.Get(fmt.Sprintf("%s.%d.%d", enum.path, enumValueCommentPath, i)),
Enum: enum,
Comments: file.comments.Get(fmt.Sprintf("%s.%d.%d", enum.path, enumValueCommentPath, i)),
}
if vd.Options != nil {
values[i].setOptions(vd.Options)
Expand Down
93 changes: 47 additions & 46 deletions parser_test.go
Original file line number Diff line number Diff line change
@@ -1,12 +1,13 @@
package protokit_test

import (
"github.com/golang/protobuf/proto"
"github.com/golang/protobuf/protoc-gen-go/descriptor"
"github.com/stretchr/testify/suite"

"testing"

"github.com/stretchr/testify/suite"
"google.golang.org/protobuf/reflect/protoregistry"
"google.golang.org/protobuf/runtime/protoimpl"
descriptor "google.golang.org/protobuf/types/descriptorpb"

"github.com/pseudomuto/protokit"
"github.com/pseudomuto/protokit/utils"
)
Expand Down Expand Up @@ -37,7 +38,7 @@ func (assert *ParserTest) SetupSuite() {
}

func registerTestExtensions() {
var E_ExtendFile = &proto.ExtensionDesc{
var E_ExtendFile = &protoimpl.ExtensionInfo{
ExtendedType: (*descriptor.FileOptions)(nil),
ExtensionType: (*bool)(nil),
Field: 20000,
Expand All @@ -46,7 +47,7 @@ func registerTestExtensions() {
Filename: "extend.proto",
}

var E_ExtendService = &proto.ExtensionDesc{
var E_ExtendService = &protoimpl.ExtensionInfo{
ExtendedType: (*descriptor.ServiceOptions)(nil),
ExtensionType: (*bool)(nil),
Field: 20000,
Expand All @@ -55,7 +56,7 @@ func registerTestExtensions() {
Filename: "extend.proto",
}

var E_ExtendMethod = &proto.ExtensionDesc{
var E_ExtendMethod = &protoimpl.ExtensionInfo{
ExtendedType: (*descriptor.MethodOptions)(nil),
ExtensionType: (*bool)(nil),
Field: 20000,
Expand All @@ -64,7 +65,7 @@ func registerTestExtensions() {
Filename: "extend.proto",
}

var E_ExtendEnum = &proto.ExtensionDesc{
var E_ExtendEnum = &protoimpl.ExtensionInfo{
ExtendedType: (*descriptor.EnumOptions)(nil),
ExtensionType: (*bool)(nil),
Field: 20000,
Expand All @@ -73,7 +74,7 @@ func registerTestExtensions() {
Filename: "extend.proto",
}

var E_ExtendEnumValue = &proto.ExtensionDesc{
var E_ExtendEnumValue = &protoimpl.ExtensionInfo{
ExtendedType: (*descriptor.EnumValueOptions)(nil),
ExtensionType: (*bool)(nil),
Field: 20000,
Expand All @@ -82,7 +83,7 @@ func registerTestExtensions() {
Filename: "extend.proto",
}

var E_ExtendMessage = &proto.ExtensionDesc{
var E_ExtendMessage = &protoimpl.ExtensionInfo{
ExtendedType: (*descriptor.MessageOptions)(nil),
ExtensionType: (*bool)(nil),
Field: 20000,
Expand All @@ -91,7 +92,7 @@ func registerTestExtensions() {
Filename: "extend.proto",
}

var E_ExtendField = &proto.ExtensionDesc{
var E_ExtendField = &protoimpl.ExtensionInfo{
ExtendedType: (*descriptor.FieldOptions)(nil),
ExtensionType: (*bool)(nil),
Field: 20000,
Expand All @@ -100,13 +101,13 @@ func registerTestExtensions() {
Filename: "extend.proto",
}

proto.RegisterExtension(E_ExtendFile)
proto.RegisterExtension(E_ExtendService)
proto.RegisterExtension(E_ExtendMethod)
proto.RegisterExtension(E_ExtendEnum)
proto.RegisterExtension(E_ExtendEnumValue)
proto.RegisterExtension(E_ExtendMessage)
proto.RegisterExtension(E_ExtendField)
protoregistry.GlobalTypes.RegisterExtension(E_ExtendFile)
protoregistry.GlobalTypes.RegisterExtension(E_ExtendService)
protoregistry.GlobalTypes.RegisterExtension(E_ExtendMethod)
protoregistry.GlobalTypes.RegisterExtension(E_ExtendEnum)
protoregistry.GlobalTypes.RegisterExtension(E_ExtendEnumValue)
protoregistry.GlobalTypes.RegisterExtension(E_ExtendMessage)
protoregistry.GlobalTypes.RegisterExtension(E_ExtendField)
}

func (assert *ParserTest) TestFileParsing() {
Expand Down Expand Up @@ -288,97 +289,97 @@ func (assert *ParserTest) TestNestedMessages() {
func (assert *ParserTest) TestExtendedOptions() {
assert.Contains(proto2.OptionExtensions, "com.pseudomuto.protokit.v1.extend_file")

extendedValue, ok := proto2.OptionExtensions["com.pseudomuto.protokit.v1.extend_file"].(*bool)
extendedValue, ok := proto2.OptionExtensions["com.pseudomuto.protokit.v1.extend_file"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

service := proto2.GetService("BookingService")
assert.Contains(service.OptionExtensions, "com.pseudomuto.protokit.v1.extend_service")

extendedValue, ok = service.OptionExtensions["com.pseudomuto.protokit.v1.extend_service"].(*bool)
extendedValue, ok = service.OptionExtensions["com.pseudomuto.protokit.v1.extend_service"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

method := service.GetNamedMethod("BookVehicle")
assert.Contains(method.OptionExtensions, "com.pseudomuto.protokit.v1.extend_method")

extendedValue, ok = method.OptionExtensions["com.pseudomuto.protokit.v1.extend_method"].(*bool)
extendedValue, ok = method.OptionExtensions["com.pseudomuto.protokit.v1.extend_method"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

message := proto2.GetMessage("Booking")
assert.Contains(message.OptionExtensions, "com.pseudomuto.protokit.v1.extend_message")

extendedValue, ok = message.OptionExtensions["com.pseudomuto.protokit.v1.extend_message"].(*bool)
extendedValue, ok = message.OptionExtensions["com.pseudomuto.protokit.v1.extend_message"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

field := message.GetMessageField("payment_received")
assert.Contains(field.OptionExtensions, "com.pseudomuto.protokit.v1.extend_field")

extendedValue, ok = field.OptionExtensions["com.pseudomuto.protokit.v1.extend_field"].(*bool)
extendedValue, ok = field.OptionExtensions["com.pseudomuto.protokit.v1.extend_field"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

enum := proto2.GetEnum("BookingType")
assert.Contains(enum.OptionExtensions, "com.pseudomuto.protokit.v1.extend_enum")

extendedValue, ok = enum.OptionExtensions["com.pseudomuto.protokit.v1.extend_enum"].(*bool)
extendedValue, ok = enum.OptionExtensions["com.pseudomuto.protokit.v1.extend_enum"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

enumValue := enum.GetNamedValue("FUTURE")
assert.Contains(enumValue.OptionExtensions, "com.pseudomuto.protokit.v1.extend_enum_value")

extendedValue, ok = enumValue.OptionExtensions["com.pseudomuto.protokit.v1.extend_enum_value"].(*bool)
extendedValue, ok = enumValue.OptionExtensions["com.pseudomuto.protokit.v1.extend_enum_value"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

assert.Contains(proto3.OptionExtensions, "com.pseudomuto.protokit.v1.extend_file")

extendedValue, ok = proto3.OptionExtensions["com.pseudomuto.protokit.v1.extend_file"].(*bool)
extendedValue, ok = proto3.OptionExtensions["com.pseudomuto.protokit.v1.extend_file"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

service = proto3.GetService("Todo")
assert.Contains(service.OptionExtensions, "com.pseudomuto.protokit.v1.extend_service")

extendedValue, ok = service.OptionExtensions["com.pseudomuto.protokit.v1.extend_service"].(*bool)
extendedValue, ok = service.OptionExtensions["com.pseudomuto.protokit.v1.extend_service"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

method = service.GetNamedMethod("CreateList")
assert.Contains(method.OptionExtensions, "com.pseudomuto.protokit.v1.extend_method")

extendedValue, ok = method.OptionExtensions["com.pseudomuto.protokit.v1.extend_method"].(*bool)
extendedValue, ok = method.OptionExtensions["com.pseudomuto.protokit.v1.extend_method"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

message = proto3.GetMessage("List")
assert.Contains(message.OptionExtensions, "com.pseudomuto.protokit.v1.extend_message")

extendedValue, ok = message.OptionExtensions["com.pseudomuto.protokit.v1.extend_message"].(*bool)
extendedValue, ok = message.OptionExtensions["com.pseudomuto.protokit.v1.extend_message"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

field = message.GetMessageField("name")
assert.Contains(field.OptionExtensions, "com.pseudomuto.protokit.v1.extend_field")

extendedValue, ok = field.OptionExtensions["com.pseudomuto.protokit.v1.extend_field"].(*bool)
extendedValue, ok = field.OptionExtensions["com.pseudomuto.protokit.v1.extend_field"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

enum = proto3.GetEnum("ListType")
assert.Contains(enum.OptionExtensions, "com.pseudomuto.protokit.v1.extend_enum")

extendedValue, ok = enum.OptionExtensions["com.pseudomuto.protokit.v1.extend_enum"].(*bool)
extendedValue, ok = enum.OptionExtensions["com.pseudomuto.protokit.v1.extend_enum"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)

enumValue = enum.GetNamedValue("CHECKLIST")
assert.Contains(enumValue.OptionExtensions, "com.pseudomuto.protokit.v1.extend_enum_value")

extendedValue, ok = enumValue.OptionExtensions["com.pseudomuto.protokit.v1.extend_enum_value"].(*bool)
extendedValue, ok = enumValue.OptionExtensions["com.pseudomuto.protokit.v1.extend_enum_value"].(bool)
assert.True(ok)
assert.True(*extendedValue)
assert.True(extendedValue)
}
Loading