[ux] warn on default requestBody name, tests

Author: lispyclouds

api.yaml

@@ -37,7 +37,7 @@ paths:
         - ap
 
       requestBody:
-        description: The numebers map
+        description: The numbers map
         required: true
         x-cli-name: nmap
         content:

lib.go

@@ -149,6 +149,12 @@ func addRequestBody(cmd *cobra.Command, op *v3.Operation, handlerData *HandlerDa
 		paramName := "climate-data"
 		if altName := bExts.name; altName != "" {
 			paramName = altName
+		} else {
+			slog.Warn(
+				fmt.Sprintf("No name set of requestBody, defaulting to %s", paramName),
+				"id",
+				op.OperationId,
+			)
 		}
 
 		// TODO: Handle all the different MIME types and schemas from body.Content
@@ -244,7 +250,7 @@ func BootstrapV3(rootCmd *cobra.Command, model libopenapi.DocumentModel[v3.Docum
 
 			handler, ok := handlers[op.OperationId]
 			if !ok {
-				slog.Warn("Ho handler defined, skipping", "id", op.OperationId)
+				slog.Warn("No handler defined, skipping", "id", op.OperationId)
 				continue
 			}
 

lib_test.go

@@ -10,9 +10,7 @@ import (
 
 func TestLoadFileV3(t *testing.T) {
 	_, err := LoadFileV3("api.yaml")
-	if err != nil {
-		t.Fatalf("LoadFileV3 failed with: %e", err)
-	}
+	assert.NoError(t, err)
 }
 
 func TestInterpolatePath(t *testing.T) {
@@ -43,9 +41,7 @@ func assertCmdTree(t *testing.T, cmd *cobra.Command, assertConf map[string]map[s
 	fmt.Println("Checking cmd level " + prefix)
 
 	expected, ok := assertConf[prefix]
-	if !ok {
-		t.Fatalf("Invalid prefix found: %s", prefix)
-	}
+	assert.Truef(t, ok, "Invalid prefix found: %s", prefix)
 
 	assert.Equal(t, expected["Use"], cmd.Use)
 	assert.Equal(t, expected["Short"], cmd.Short)
@@ -68,7 +64,7 @@ func assertCmdTree(t *testing.T, cmd *cobra.Command, assertConf map[string]map[s
 				_, err = cmd.Flags().GetBool(name)
 			}
 
-			assert.NoError(t, err, fmt.Sprintf("Flag: %s Type: %s", name, typ))
+			assert.NoErrorf(t, err, "Flag: %s Type: %s", name, typ)
 		}
 	}
 
@@ -79,9 +75,7 @@ func assertCmdTree(t *testing.T, cmd *cobra.Command, assertConf map[string]map[s
 
 func TestBootstrapV3(t *testing.T) {
 	model, err := LoadFileV3("api.yaml")
-	if err != nil {
-		t.Fatalf("LoadFileV3 failed with: %e", err)
-	}
+	assert.NoError(t, err)
 
 	handler := func(opts *cobra.Command, args []string, data HandlerData) {
 		assert.Equal(t, data.PathParams, []ParamMeta{{Name: "p1", Type: Integer}})
@@ -103,9 +97,7 @@ func TestBootstrapV3(t *testing.T) {
 	}
 
 	err = BootstrapV3(rootCmd, *model, handlers)
-	if err != nil {
-		t.Fatalf("BootstrapV3 failed with: %e", err)
-	}
+	assert.NoError(t, err)
 
 	var noAlias []string
 	assertConf := map[string]map[string]any{