Fix minimum/maximum bug in openapi.ts and add tests for numeric constraints

- Fixed a bug in openapi.ts where minimum and maximum values were not being properly handled
- Added comprehensive tests for numeric constraints in openapi.ts
- Ensured all custom mappings in openapi.ts are properly tested
- Fixed the handling of undefined minimum and maximum values

Author: youngjungithub

packages/openapi-generator/test/openapi/openapiSchemaTest.ts

@@ -0,0 +1,129 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { schemaToOpenAPI } from '../../src/openapi';
+import { OpenAPIV3 } from 'openapi-types';
+import { Schema } from '../../src/ir';
+
+// Test numeric constraints
+test('numeric constraints are properly mapped', () => {
+  // Create a schema with numeric constraints
+  const schema: Schema = {
+    type: 'number',
+    minimum: 0,
+    maximum: 100,
+    multipleOf: 5,
+    comment: {
+      description: 'A number with constraints',
+      tags: [],
+      source: [],
+      problems: []
+    }
+  };
+
+  const result = schemaToOpenAPI(schema) as OpenAPIV3.SchemaObject;
+
+  // Check that the result has the correct type and description
+  assert.equal(result.type, 'number');
+  assert.equal(result.description, 'A number with constraints');
+
+  // Check that the constraints are properly mapped
+  assert.equal(result.minimum, 0);
+  assert.equal(result.maximum, 100);
+  assert.equal(result.multipleOf, 5);
+});
+
+// Test exclusive constraints
+test('exclusive numeric constraints are properly mapped', () => {
+  // Create a schema with exclusive constraints
+  const schema: Schema = {
+    type: 'number',
+    minimum: 0,
+    maximum: 100,
+    exclusiveMinimum: true,
+    exclusiveMaximum: true,
+    comment: {
+      description: 'A number with exclusive constraints',
+      tags: [],
+      source: [],
+      problems: []
+    }
+  };
+
+  const result = schemaToOpenAPI(schema) as OpenAPIV3.SchemaObject;
+
+  // Check that the result has the correct type and description
+  assert.equal(result.type, 'number');
+  assert.equal(result.description, 'A number with exclusive constraints');
+
+  // Check that the constraints are properly mapped
+  assert.equal(result.minimum, 0);
+  assert.equal(result.maximum, 100);
+  assert.equal(result.exclusiveMinimum, true);
+  assert.equal(result.exclusiveMaximum, true);
+});
+
+// Test string-encoded numeric types
+test('string-encoded numeric types are properly mapped', () => {
+  // Create a schema for a string-encoded number
+  const schema: Schema = {
+    type: 'string',
+    format: 'number',
+    decodedType: 'number'
+  };
+
+  const result = schemaToOpenAPI(schema) as OpenAPIV3.SchemaObject;
+
+  // Check that the result has the correct type
+  assert.equal(result.type, 'number');
+});
+
+// Test string-encoded integer types
+test('string-encoded integer types are properly mapped', () => {
+  // Create a schema for a string-encoded integer
+  const schema: Schema = {
+    type: 'string',
+    format: 'integer',
+    decodedType: 'number'
+  };
+
+  const result = schemaToOpenAPI(schema) as OpenAPIV3.SchemaObject;
+
+  // Check that the result has the correct type
+  assert.equal(result.type, 'integer');
+});
+
+// Test BigInt types
+test('BigInt types are properly mapped', () => {
+  // Create a schema for a BigInt
+  const schema: Schema = {
+    type: 'string',
+    format: 'number',
+    decodedType: 'bigint'
+  };
+
+  const result = schemaToOpenAPI(schema) as OpenAPIV3.SchemaObject;
+
+  // Check that the result has the correct type and format
+  assert.equal(result.type, 'integer');
+  assert.equal(result.format, 'int64');
+});
+
+// Test BigInt types with constraints
+test('BigInt types with constraints are properly mapped', () => {
+  // Create a schema for a BigInt with constraints
+  const schema: Schema = {
+    type: 'string',
+    format: 'number',
+    decodedType: 'bigint',
+    minimum: 0,
+    maximum: 1000000000000000000
+  };
+
+  const result = schemaToOpenAPI(schema) as OpenAPIV3.SchemaObject;
+
+  // Check that the result has the correct type, format, and constraints
+  assert.equal(result.type, 'integer');
+  assert.equal(result.format, 'int64');
+  assert.equal(result.minimum, 0);
+  assert.equal(result.maximum, 1000000000000000000);
+});