more error handling

nucleus/gnmi_transport.go

@@ -113,6 +113,11 @@ func (g *Gnmi) Set(ctx context.Context, args ...interface{}) error {
 				v: o,
 				t: reflect.TypeOf("placeholder"),
 			}
+		} else if attrs == nil || len(attrs) == 0 {
+			return &ErrInvalidParameters{
+				f: "gnmi.Set()",
+				r: "no parameters provided",
+			}
 		}
 		opts = append(opts, &gnmi.Operation{
 			// Hardcoded TransportUpdate until multiple operations are supported

test/integration/cmdIntegration_test.go

@@ -85,7 +85,7 @@ func testSetupIntegration() {
 	}
 }
 
-func TestCliIntegration(t *testing.T) {
+func TestCmdIntegration(t *testing.T) {
 	if testing.Short() {
 		t.Skip("skipping integration test")
 	}

test/integration/nucleusIntegration_test.go

@@ -41,7 +41,7 @@ func TestGnmi_SetIntegration(t *testing.T) {
 			},
 			args: args{
 				ctx:    context.Background(),
-				params: []string{},
+				params: []string{"/system/config/hostname", "ceos3000"},
 			},
 			wantErr: true,
 		},
@@ -54,6 +54,15 @@ func TestGnmi_SetIntegration(t *testing.T) {
 			},
 			wantErr: false,
 		},
+		{
+			name:   "invalid update",
+			fields: fields{opt: opt},
+			args: args{
+				ctx:    context.Background(),
+				params: nil,
+			},
+			wantErr: true,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {