fix: reformat test

internal/proto/struct_test.go

@@ -4,50 +4,112 @@ import (
 	"testing"
 
 	pb_struct "github.com/golang/protobuf/ptypes/struct"
-
-	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
 )
 
-func TestToPBStruct(t *testing.T) {
+func Test_ToPBStruct(t *testing.T) {
-	obj := struct {
+	type obj struct {
-		ID   string
+		ID  string
-		Name string
+		Seq uint64
-		Seq  uint64
+	}
-	}{
+	type args struct {
-		ID:   "asdf",
+		obj obj
-		Name: "ueli",
+	}
-		Seq:  208582075,
+	tests := []struct {
+		name    string
+		args    args
+		wantErr bool
+		length  int
+		result  obj
+	}{
+		{
+			name: "to pb stuct",
+			args: args{
+				obj: obj{ID: "ID", Seq: 12345},
+			},
+			wantErr: false,
+			length:  2,
+			result:  obj{ID: "ID", Seq: 12345},
+		},
+		{
+			name: "empty struct",
+			args: args{
+				obj: obj{},
+			},
+			wantErr: false,
+			length:  2,
+			result:  obj{ID: "", Seq: 0},
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			fields, err := ToPBStruct(tt.args.obj)
+			if tt.wantErr && err == nil {
+				t.Errorf("got wrong result, should get err: actual: %v ", err)
+			}
+			if !tt.wantErr && len(fields.Fields) != tt.length {
+				t.Errorf("got wrong result length, expecting: %v, actual: %v ", tt.length, len(fields.Fields))
+			}
+			if !tt.wantErr && tt.result.ID != fields.Fields["ID"].GetStringValue() {
+				t.Errorf("got wrong result, ID should be same: expecting: %v, actual: %v ", tt.result.ID, fields.Fields["ID"].GetStringValue())
+			}
+			if !tt.wantErr && int(tt.result.Seq) != int(fields.Fields["Seq"].GetNumberValue()) {
+				t.Errorf("got wrong result, Seq should be same: expecting: %v, actual: %v ", tt.result.Seq, fields.Fields["Seq"].GetStringValue())
+			}
+		})
 	}
-	fields, err := ToPBStruct(&obj)
-	require.NoError(t, err)
-	require.Len(t, fields.Fields, 3)
-
-	assert.Equal(t, obj.ID, fields.Fields["ID"].GetStringValue())
-	assert.Equal(t, int(obj.Seq), int(fields.Fields["Seq"].GetNumberValue()))
-	assert.Equal(t, obj.Name, fields.Fields["Name"].GetStringValue())
 }
 
-func TestFromPBStruct(t *testing.T) {
+func Test_FromPBStruct(t *testing.T) {
-	name := "ueli"
+	type obj struct {
-	id := "asdf"
+		ID  string
-	seq := float64(208582075)
+		Seq uint64
-	s := &pb_struct.Struct{Fields: map[string]*pb_struct.Value{
+	}
-		"ID":   &pb_struct.Value{Kind: &pb_struct.Value_StringValue{StringValue: id}},
+	type args struct {
-		"Name": &pb_struct.Value{Kind: &pb_struct.Value_StringValue{StringValue: name}},
+		obj    *obj
-		"Seq":  &pb_struct.Value{Kind: &pb_struct.Value_NumberValue{NumberValue: seq}},
+		fields *pb_struct.Struct
-	}}
+	}
-
+	tests := []struct {
-	obj := struct {
+		name    string
-		ID   string
+		args    args
-		Name string
+		wantErr bool
-		Seq  uint64
+		result  obj
-	}{}
+	}{
-
+		{
-	err := FromPBStruct(&obj, s)
+			name: "from pb stuct",
-	require.NoError(t, err)
+			args: args{
-
+				obj: &obj{},
-	assert.Equal(t, id, obj.ID)
+				fields: &pb_struct.Struct{Fields: map[string]*pb_struct.Value{
-	assert.Equal(t, name, obj.Name)
+					"ID":  &pb_struct.Value{Kind: &pb_struct.Value_StringValue{StringValue: "ID"}},
-	assert.Equal(t, int(seq), int(obj.Seq))
+					"Seq": &pb_struct.Value{Kind: &pb_struct.Value_NumberValue{NumberValue: 12345}},
+				},
+				},
+			},
+			wantErr: false,
+			result:  obj{ID: "ID", Seq: 12345},
+		},
+		{
+			name: "no fields",
+			args: args{
+				obj: &obj{},
+				fields: &pb_struct.Struct{Fields: map[string]*pb_struct.Value{},
+				},
+			},
+			wantErr: false,
+			result:  obj{ID: "", Seq: 0},
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			err := FromPBStruct(tt.args.obj, tt.args.fields)
+			if tt.wantErr && err == nil {
+				t.Errorf("got wrong result, should get err: actual: %v ", err)
+			}
+			if !tt.wantErr && tt.result.ID != tt.args.obj.ID {
+				t.Errorf("got wrong result, ID should be same: expecting: %v, actual: %v ", tt.result.ID, tt.args.obj.ID)
+			}
+			if !tt.wantErr && int(tt.result.Seq) != int(tt.args.obj.Seq) {
+				t.Errorf("got wrong result, Seq should be same: expecting: %v, actual: %v ", tt.result.Seq, tt.args.obj.Seq)
+			}
+		})
+	}
 }