Test middleend Update with non-AIP complaint names.

Signed-off-by: Artsiom Koltun <[email protected]>

pkg/middleend/encryption_test.go

@@ -613,6 +613,16 @@ func TestMiddleEnd_UpdateEncryptedVolume(t *testing.T) {
 			false,
 			false,
 		},
+		"malformed name": {
+			nil,
+			&pb.EncryptedVolume{Name: "-ABC-DEF"},
+			nil,
+			[]string{""},
+			codes.Unknown,
+			fmt.Sprintf("segment '%s': not a valid DNS name", "-ABC-DEF"),
+			false,
+			false,
+		},
 	}
 
 	// run tests

pkg/middleend/qos.go

@@ -109,11 +109,7 @@ func (s *Server) UpdateQosVolume(_ context.Context, in *pb.UpdateQosVolumeReques
 		log.Printf("error: %v", err)
 		return nil, err
 	}
-	// Validate that a resource name conforms to the restrictions outlined in AIP-122.
-	// if err := resourcename.Validate(in.QosVolume.Name); err != nil {
-	// 	log.Printf("error: %v", err)
-	// 	return nil, err
-	// }
+
 	// fetch object from the database
 	if err := s.verifyQosVolume(in.QosVolume); err != nil {
 		log.Println("error:", err)
@@ -253,6 +249,10 @@ func (s *Server) verifyQosVolume(volume *pb.QosVolume) error {
 	if volume.Name == "" {
 		return fmt.Errorf("QoS volume name cannot be empty")
 	}
+	// Validate that a resource name conforms to the restrictions outlined in AIP-122.
+	if err := resourcename.Validate(volume.Name); err != nil {
+		return err
+	}
 	if volume.VolumeId == nil || volume.VolumeId.Value == "" {
 		return fmt.Errorf("volume_id cannot be empty")
 	}

pkg/middleend/qos_test.go

@@ -735,6 +735,17 @@ func TestMiddleEnd_UpdateQosVolume(t *testing.T) {
 			existBefore: true,
 			missing:     false,
 		},
+		"malformed name": {
+			mask:        nil,
+			in:          &pb.QosVolume{Name: "-ABC-DEF"},
+			out:         nil,
+			spdk:        []string{""},
+			errCode:     codes.InvalidArgument,
+			errMsg:      fmt.Sprintf("segment '%s': not a valid DNS name", "-ABC-DEF"),
+			start:       false,
+			existBefore: false,
+			missing:     false,
+		},
 	}
 
 	for name, tt := range tests {