Skip to content

Commit c724e5a

Browse files
committed
fix: make private
1 parent 3875d0a commit c724e5a

3 files changed

Lines changed: 22 additions & 13 deletions

File tree

packages/openapi-code-generator/src/core/input.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,7 @@ export class Input {
6565
private loader: OpenapiLoader,
6666
readonly config: InputConfig,
6767
private readonly syntheticNameGenerator: SyntheticNameGenerator = defaultSyntheticNameGenerator,
68-
// todo: make private
69-
readonly schemaNormalizer = new SchemaNormalizer(config),
68+
private readonly schemaNormalizer = new SchemaNormalizer(config),
7069
private readonly parameterNormalizer = new ParameterNormalizer(
7170
loader,
7271
schemaNormalizer,

packages/openapi-code-generator/src/test/input.test-utils.ts

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import path from "node:path"
22
import {jest} from "@jest/globals"
33
import yaml from "js-yaml"
44
import {NodeFsAdaptor} from "../core/file-system/node-fs-adaptor"
5-
import {Input} from "../core/input"
5+
import {Input, type InputConfig, SchemaNormalizer} from "../core/input"
66
import {GenericLoader} from "../core/loaders/generic.loader"
77
import {TypespecLoader} from "../core/loaders/typespec.loader"
88
import {logger} from "../core/logger"
@@ -51,11 +51,14 @@ export async function unitTestInput(
5151
await TypespecLoader.create(),
5252
)
5353

54+
const config = {
55+
extractInlineSchemas: true,
56+
enumExtensibility: "closed",
57+
} satisfies InputConfig
58+
5459
return {
55-
input: new Input(loader, {
56-
extractInlineSchemas: true,
57-
enumExtensibility: "closed",
58-
}),
60+
input: new Input(loader, config),
61+
schemaNormalizer: new SchemaNormalizer(config),
5962
file,
6063
}
6164
}

packages/openapi-code-generator/src/typescript/common/schema-builders/schema-builder.test-utils.ts

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import ts from "typescript"
2-
import type {Input} from "../../../core/input"
2+
import type {Input, SchemaNormalizer} from "../../../core/input"
33
import type {
44
Reference,
55
Schema,
@@ -27,20 +27,27 @@ export function schemaBuilderTestHarness(
2727
schema: Schema,
2828
config: SchemaBuilderConfig = {allowAny: false},
2929
) {
30-
const {input} = await unitTestInput(version)
31-
return getResult(input, schema, true, config)
30+
const {input, schemaNormalizer} = await unitTestInput(version)
31+
return getResult(input, schemaNormalizer, schema, true, config)
3232
}
3333

3434
async function getActual(
3535
path: string,
3636
config: SchemaBuilderConfig = {allowAny: false},
3737
) {
38-
const {input, file} = await unitTestInput(version)
39-
return getResult(input, {$ref: `${file}#${path}`}, true, config)
38+
const {input, schemaNormalizer, file} = await unitTestInput(version)
39+
return getResult(
40+
input,
41+
schemaNormalizer,
42+
{$ref: `${file}#${path}`},
43+
true,
44+
config,
45+
)
4046
}
4147

4248
async function getResult(
4349
input: Input,
50+
schemaNormalizer: SchemaNormalizer,
4451
maybeSchema: Schema | Reference,
4552
required: boolean,
4653
config: SchemaBuilderConfig,
@@ -70,7 +77,7 @@ export function schemaBuilderTestHarness(
7077
.fromModel(
7178
isRef(maybeSchema)
7279
? maybeSchema
73-
: input.schemaNormalizer.normalize(maybeSchema),
80+
: schemaNormalizer.normalize(maybeSchema),
7481
required,
7582
)
7683

0 commit comments

Comments
 (0)