From d667bbee0cb0f08aa407cbeaededfedbdc629c6d Mon Sep 17 00:00:00 2001 From: Evstigneev Denis Date: Wed, 28 Feb 2024 15:06:29 +0300 Subject: [PATCH 1/3] add option metadata --- options/options.go | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/options/options.go b/options/options.go index cda572a6..df38591f 100644 --- a/options/options.go +++ b/options/options.go @@ -157,6 +157,22 @@ func Metadata(md metadata.Metadata) Option { } } +func MetadataAny(md any) Option { + result := metadata.Metadata{} + switch vt := md.(type) { + case metadata.Metadata: + result = metadata.Copy(vt) + case []string: + for index := 0; index < len(vt); index += 2 { + result[vt[index]] = result[vt[index]] + } + } + + return func(src interface{}) error { + return Set(src, result, ".Metadata") + } +} + // Namespace to use func Namespace(ns string) Option { return func(src interface{}) error { -- 2.45.2 From 47940a5ca2c31e27e611aa0e05a87f6b2ae900d0 Mon Sep 17 00:00:00 2001 From: Evstigneev Denis Date: Wed, 28 Feb 2024 15:06:29 +0300 Subject: [PATCH 2/3] add option metadata --- options/options.go | 16 ++++++++++ options/options_test.go | 69 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 85 insertions(+) diff --git a/options/options.go b/options/options.go index cda572a6..5749c15e 100644 --- a/options/options.go +++ b/options/options.go @@ -157,6 +157,22 @@ func Metadata(md metadata.Metadata) Option { } } +func MetadataAny(md any) Option { + result := metadata.Metadata{} + switch vt := md.(type) { + case metadata.Metadata: + result = metadata.Copy(vt) + case map[string]string: + result = metadata.Copy(vt) + case []string: + result.Set(vt...) + } + + return func(src interface{}) error { + return Set(src, result, ".Metadata") + } +} + // Namespace to use func Namespace(ns string) Option { return func(src interface{}) error { diff --git a/options/options_test.go b/options/options_test.go index 7d3b6a62..c4b3677c 100644 --- a/options/options_test.go +++ b/options/options_test.go @@ -1,10 +1,13 @@ package options_test import ( + "fmt" "testing" "go.unistack.org/micro/v4/codec" + "go.unistack.org/micro/v4/metadata" "go.unistack.org/micro/v4/options" + "go.unistack.org/micro/v4/util/reflect" ) func TestAddress(t *testing.T) { @@ -84,3 +87,69 @@ func TestLabels(t *testing.T) { t.Fatal("failed to set labels") } } + +func TestMetadataAny(t *testing.T) { + type s struct { + Metadata metadata.Metadata + } + + testCases := []struct { + Name string + Data any + Expected metadata.Metadata + }{ + { + "strings_even", + []string{"key1", "val1", "key2", "val2"}, + metadata.Metadata{ + "Key1": "val1", + "Key2": "val2", + }, + }, + { + "strings_odd", + []string{"key1", "val1", "key2"}, + metadata.Metadata{ + "Key1": "val1", + }, + }, + { + "map", + map[string]string{ + "key1": "val1", + "key2": "val2", + }, + metadata.Metadata{ + "Key1": "val1", + "Key2": "val2", + }, + }, + { + "metadata.Metadata", + metadata.Metadata{ + "key1": "val1", + "key2": "val2", + }, + metadata.Metadata{ + "Key1": "val1", + "Key2": "val2", + }, + }, + } + + for _, tt := range testCases { + t.Run(tt.Name, func(t *testing.T) { + src := &s{} + var opts []options.Option + opts = append(opts, options.MetadataAny(tt.Data)) + for _, o := range opts { + if err := o(src); err != nil { + t.Fatal(err) + } + if !reflect.Equal(tt.Expected, src.Metadata) { + t.Fatal(fmt.Sprintf("expected: %v, actual: %v", tt.Expected, src.Metadata)) + } + } + }) + } +} -- 2.45.2 From e6feca2fb1014c42d6b1b200919fc12e2e2c9e6c Mon Sep 17 00:00:00 2001 From: Vasiliy Tolstov Date: Wed, 28 Feb 2024 23:50:38 +0300 Subject: [PATCH 3/3] fixup Signed-off-by: Vasiliy Tolstov --- options/options.go | 8 +------- options/options_test.go | 2 +- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/options/options.go b/options/options.go index 5749c15e..226fd5e4 100644 --- a/options/options.go +++ b/options/options.go @@ -151,13 +151,7 @@ func ContentType(ct string) Option { } // Metadata pass additional metadata -func Metadata(md metadata.Metadata) Option { - return func(src interface{}) error { - return Set(src, metadata.Copy(md), ".Metadata") - } -} - -func MetadataAny(md any) Option { +func Metadata(md any) Option { result := metadata.Metadata{} switch vt := md.(type) { case metadata.Metadata: diff --git a/options/options_test.go b/options/options_test.go index c4b3677c..1bb0ce23 100644 --- a/options/options_test.go +++ b/options/options_test.go @@ -141,7 +141,7 @@ func TestMetadataAny(t *testing.T) { t.Run(tt.Name, func(t *testing.T) { src := &s{} var opts []options.Option - opts = append(opts, options.MetadataAny(tt.Data)) + opts = append(opts, options.Metadata(tt.Data)) for _, o := range opts { if err := o(src); err != nil { t.Fatal(err) -- 2.45.2