fix codec.Frame in case of flatten struct tag
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
This commit is contained in:
parent
6673e7dd69
commit
016b3aa878
@ -3,8 +3,40 @@ package proto
|
|||||||
import (
|
import (
|
||||||
"bytes"
|
"bytes"
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
|
"github.com/unistack-org/micro/v3/codec"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
func TestFrame(t *testing.T) {
|
||||||
|
s := &codec.Frame{Data: []byte("test")}
|
||||||
|
|
||||||
|
buf, err := NewCodec().Marshal(s)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if !bytes.Equal(buf, []byte(`test`)) {
|
||||||
|
t.Fatalf("bytes not equal %s != %s", buf, `test`)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestFrameFlatten(t *testing.T) {
|
||||||
|
s := &struct {
|
||||||
|
One string
|
||||||
|
Name *codec.Frame `json:"name" codec:"flatten"`
|
||||||
|
}{
|
||||||
|
One: "xx",
|
||||||
|
Name: &codec.Frame{Data: []byte("test")},
|
||||||
|
}
|
||||||
|
|
||||||
|
buf, err := NewCodec().Marshal(s)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
if !bytes.Equal(buf, []byte(`test`)) {
|
||||||
|
t.Fatalf("bytes not equal %s != %s", buf, `test`)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestReadBody(t *testing.T) {
|
func TestReadBody(t *testing.T) {
|
||||||
t.Skip("skip as no proto")
|
t.Skip("skip as no proto")
|
||||||
s := &struct {
|
s := &struct {
|
||||||
|
46
proto.go
46
proto.go
@ -13,55 +13,61 @@ type protoCodec struct {
|
|||||||
opts codec.Options
|
opts codec.Options
|
||||||
}
|
}
|
||||||
|
|
||||||
|
var _ codec.Codec = &protoCodec{}
|
||||||
|
|
||||||
const (
|
const (
|
||||||
flattenTag = "flatten"
|
flattenTag = "flatten"
|
||||||
)
|
)
|
||||||
|
|
||||||
func (c *protoCodec) Marshal(v interface{}, opts ...codec.Option) ([]byte, error) {
|
func (c *protoCodec) Marshal(v interface{}, opts ...codec.Option) ([]byte, error) {
|
||||||
switch m := v.(type) {
|
if v == nil {
|
||||||
case nil:
|
|
||||||
return nil, nil
|
return nil, nil
|
||||||
case *codec.Frame:
|
}
|
||||||
return m.Data, nil
|
|
||||||
case proto.Message:
|
|
||||||
options := c.opts
|
options := c.opts
|
||||||
for _, o := range opts {
|
for _, o := range opts {
|
||||||
o(&options)
|
o(&options)
|
||||||
}
|
}
|
||||||
|
|
||||||
if nv, nerr := rutil.StructFieldByTag(v, options.TagName, flattenTag); nerr == nil {
|
if nv, nerr := rutil.StructFieldByTag(v, options.TagName, flattenTag); nerr == nil {
|
||||||
if nm, ok := nv.(proto.Message); ok {
|
v = nv
|
||||||
m = nm
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if m, ok := v.(*codec.Frame); ok {
|
||||||
|
return m.Data, nil
|
||||||
}
|
}
|
||||||
return proto.Marshal(m)
|
|
||||||
}
|
if _, ok := v.(proto.Message); !ok {
|
||||||
return nil, codec.ErrInvalidMessage
|
return nil, codec.ErrInvalidMessage
|
||||||
|
}
|
||||||
|
|
||||||
|
return proto.Marshal(v.(proto.Message))
|
||||||
}
|
}
|
||||||
|
|
||||||
func (c *protoCodec) Unmarshal(d []byte, v interface{}, opts ...codec.Option) error {
|
func (c *protoCodec) Unmarshal(d []byte, v interface{}, opts ...codec.Option) error {
|
||||||
if len(d) == 0 {
|
if v == nil || len(d) == 0 {
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
switch m := v.(type) {
|
|
||||||
case nil:
|
|
||||||
return nil
|
|
||||||
case *codec.Frame:
|
|
||||||
m.Data = d
|
|
||||||
case proto.Message:
|
|
||||||
options := c.opts
|
options := c.opts
|
||||||
for _, o := range opts {
|
for _, o := range opts {
|
||||||
o(&options)
|
o(&options)
|
||||||
}
|
}
|
||||||
|
|
||||||
if nv, nerr := rutil.StructFieldByTag(v, options.TagName, flattenTag); nerr == nil {
|
if nv, nerr := rutil.StructFieldByTag(v, options.TagName, flattenTag); nerr == nil {
|
||||||
if nm, ok := nv.(proto.Message); ok {
|
v = nv
|
||||||
m = nm
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if m, ok := v.(*codec.Frame); ok {
|
||||||
|
m.Data = d
|
||||||
|
return nil
|
||||||
}
|
}
|
||||||
return proto.Unmarshal(d, m)
|
|
||||||
}
|
if _, ok := v.(proto.Message); !ok {
|
||||||
return codec.ErrInvalidMessage
|
return codec.ErrInvalidMessage
|
||||||
|
}
|
||||||
|
|
||||||
|
return proto.Unmarshal(d, v.(proto.Message))
|
||||||
}
|
}
|
||||||
|
|
||||||
func (c *protoCodec) ReadHeader(conn io.Reader, m *codec.Message, t codec.MessageType) error {
|
func (c *protoCodec) ReadHeader(conn io.Reader, m *codec.Message, t codec.MessageType) error {
|
||||||
|
Loading…
Reference in New Issue
Block a user