(bug) putting back missing 'required: true' from path parameters

eropple · Sep 24, 2024 · e044848 · e044848
1 parent 9101303
commit e044848
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/src/plugin.ts b/src/plugin.ts
@@ -239,6 +239,7 @@ export const oas3Plugin = fastifyPlugin<OAS3PluginOptions>(
                       paramExtra.description ??
                       paramValue.description ??
                       "No path parameter description specified.",
+                    required: true,
                     example: paramExtra.example ?? paramValue.example,
                     schema: paramExtra.schemaOverride ?? paramValue,
                   });

diff --git a/test/plugin.spec.ts b/test/plugin.spec.ts
@@ -323,8 +323,18 @@ describe("plugin", () => {
     const parameters = operation?.parameters;
     // remember: `required` is implied (and forced true) for path params
     expect(parameters).toMatchObject([
-      { in: "path", name: "primary", schema: { type: "string" } },
-      { in: "path", name: "secondary", schema: { type: "number" } },
+      {
+        in: "path",
+        name: "primary",
+        schema: { type: "string" },
+        required: true,
+      },
+      {
+        in: "path",
+        name: "secondary",
+        schema: { type: "number" },
+        required: true,
+      },
     ]);
   });
 });