fix: Error when CreateVMReq does not have Spec (#470)

* fix: Error when CreateVMReq does not have Spec

In the rare case that a user either:
- Sends the wrong request type to the server (ie uses BloomRPC to send a
  ListReq to a Create call)
- Does not include a Spec at all on a CreateMicroVMRequest

we check for its presence and error if not found.

We are doing something similar for other server requests, so this
completes it.

infrastructure/grpc/server.go

@@ -39,11 +39,11 @@ func (s *server) CreateMicroVM(
 	logger := log.GetLogger(ctx)
 	logger.Trace("converting request to model")
 
-	if req == nil {
-		logger.Error("invalid create microvm request")
+	if req == nil || req.Microvm == nil {
+		logger.Error("invalid create microvm request: MicroVMSpec required")
 
 		//nolint:wrapcheck // don't wrap grpc errors when using the status package
-		return nil, status.Error(codes.InvalidArgument, "invalid request")
+		return nil, status.Error(codes.InvalidArgument, "invalid create microvm request: MicroVMSpec required")
 	}
 
 	modelSpec, err := convertMicroVMToModel(req.Microvm)

infrastructure/grpc/server_test.go

@@ -28,6 +28,12 @@ func TestServer_CreateMicroVM(t *testing.T) {
 			expectError: true,
 			expect:      func(cm *mock.MockMicroVMCommandUseCasesMockRecorder, qm *mock.MockMicroVMQueryUseCasesMockRecorder) {},
 		},
+		{
+			name:        "nil spec should fail with error",
+			createReq:   &mvm1.CreateMicroVMRequest{},
+			expectError: true,
+			expect:      func(cm *mock.MockMicroVMCommandUseCasesMockRecorder, qm *mock.MockMicroVMQueryUseCasesMockRecorder) {},
+		},
 		{
 			name:        "missing id should fail with error",
 			createReq:   createTestCreateRequest("", ""),