Merge pull request #163 from trojs/feature/no-example-value-in-response

Feature/no example value in response

.github/workflows/test.yml

@@ -8,7 +8,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18.x, 20.x, 21.x, 22.x]
+        node-version: [18.x, 20.x, 22.x]
 
     steps:
       - uses: actions/checkout@v4

package.json

@@ -1,7 +1,7 @@
 {
     "name": "@trojs/openapi-server",
     "description": "OpenAPI Server",
-    "version": "1.4.0",
+    "version": "1.5.0",
     "author": {
         "name": "Pieter Wigboldus",
         "url": "https://trojs.org/"

src/params.js

@@ -11,16 +11,16 @@ export const parseParams = ({ query, spec }) =>
     spec
         .map((parameter) => {
             const { name, schema } = parameter;
-            const {
-                type,
-                default: defaultValue,
-                example: exampleValue,
-            } = schema;
+            const { type, default: defaultValue } = schema;
             const Type = types[type];
             const paramName = query?.[name];
 
+            if (!paramName && defaultValue) {
+                return { name, value: defaultValue };
+            }
+
             if (!paramName) {
-                return { name, value: defaultValue ?? exampleValue };
+                return undefined;
             }
 
             if (Type === Boolean) {
@@ -33,6 +33,7 @@ export const parseParams = ({ query, spec }) =>
             const value = new Type(paramName).valueOf();
             return { name, value };
         })
+        .filter(Boolean)
         .reduce((acc, { name, value }) => {
             acc[name] = value;
             return acc;

src/params.test.js

@@ -86,9 +86,7 @@ const TestCases = [
                 },
             },
         ],
-        expectedResult: {
-            page: 0,
-        },
+        expectedResult: {},
     },
     {
         description: 'It should not throw if no query params are given',
@@ -106,9 +104,7 @@ const TestCases = [
                 },
             },
         ],
-        expectedResult: {
-            size: 10,
-        },
+        expectedResult: {},
     },
     {
         description: 'Parse mixed params to the types defined in the spec',