Convert protobuf message to map after applying a field mask doesn't work about fieldmask-utils HOT 6 CLOSED

I'll take a look within the next few days

from fieldmask-utils.

A PR is welcomed!

from fieldmask-utils.

@mennanov any chances you can take a look at this?

from fieldmask-utils.

I wrote a similar test that passes:

func TestTitleBug(t *testing.T) {
	mask := &fieldmaskpb.FieldMask{Paths: []string{"title"}}
	mask.Normalize()
	task := &testproto.Task{
		Title: "test",
	}
	require.True(t, mask.IsValid(task))
	protoMask, err := fieldmask_utils.MaskFromProtoFieldMask(mask, func(s string) string { return generator.CamelCase(s) })
	require.Nil(t, err)
	m := make(map[string]any)
	err = fieldmask_utils.StructToMap(protoMask, task, m)
	require.Nil(t, err)
	assert.Equal(t, map[string]any{"Title": "test"}, m)
}

Try using generator.CamelCase(s) which converts a lower case field name (in the field mask) into a camel case (golang struct naming).

from fieldmask-utils.

Thanks for the information, will give it a try! We should probably add this test case as it would help future devs that find this issue.

I can take care of creating a PR for this test and adding some docs about the generator package (which I was completely unaware of).

from fieldmask-utils.

Resolved in PR #42

from fieldmask-utils.