From 09d869c66736c99b77f5a2d7035dca84b02139e5 Mon Sep 17 00:00:00 2001 From: Kenneth Jenkins <51246568+kenjenkins@users.noreply.github.com> Date: Mon, 30 Oct 2023 15:33:50 -0700 Subject: [PATCH] add a test case for unknown sub-fields --- pkg/protoutil/fieldmask_test.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/pkg/protoutil/fieldmask_test.go b/pkg/protoutil/fieldmask_test.go index 281b32df6..c821167cf 100644 --- a/pkg/protoutil/fieldmask_test.go +++ b/pkg/protoutil/fieldmask_test.go @@ -55,6 +55,12 @@ func TestOverwriteMaskedErrors(t *testing.T) { err := protoutil.OverwriteMasked(&s1, &s2, &fieldmaskpb.FieldMask{Paths: []string{"foo"}}) assert.Equal(t, `cannot overwrite unknown field "foo" in message session.Session`, err.Error()) + s2.OauthToken = &session.OAuthToken{AccessToken: "access-token"} + err = protoutil.OverwriteMasked(&s1, &s2, &fieldmaskpb.FieldMask{ + Paths: []string{"oauth_token.foo"}}) + assert.Equal(t, `cannot overwrite unknown field "foo" in message session.OAuthToken`, + err.Error()) + err = protoutil.OverwriteMasked(&s1, &s2, &fieldmaskpb.FieldMask{Paths: []string{"device_credentials.type_id"}}) assert.Equal(t, `cannot overwrite sub-fields of field "device_credentials" in message `+