Skip to content

Commit d587582

Browse files
author
andrewelliott
committed
Remove Resolved* typealiases
1 parent fba8f32 commit d587582

17 files changed

+42
-54
lines changed

Sources/_OpenAPIGeneratorCore/Translator/Operations/OperationDescription.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ struct OperationDescription {
2525
var endpoint: OpenAPI.PathItem.Endpoint
2626

2727
/// The path parameters at the operation level.
28-
var pathParameters: ResolvedParameter.Array
28+
var pathParameters: OpenAPI.Parameter.Array
2929

3030
/// The OpenAPI components, used to resolve JSON references.
3131
var components: OpenAPI.Components
@@ -142,7 +142,7 @@ extension OperationDescription {
142142
/// Returns all parameters by resolving any parameter references first.
143143
///
144144
/// - Throws: When an invalid JSON reference is found.
145-
var allResolvedParameters: [ResolvedParameter] {
145+
var allResolvedParameters: [OpenAPI.Parameter] {
146146
get throws {
147147
try allParameters.map { try $0.resolve(in: components) }
148148
}

Sources/_OpenAPIGeneratorCore/Translator/Parameters/TypedParameter.swift

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ import OpenAPIKit30
1717
struct TypedParameter {
1818

1919
/// The OpenAPI parameter.
20-
var parameter: ResolvedParameter
20+
var parameter: OpenAPI.Parameter
2121

2222
/// The underlying schema.
2323
var schema: UnresolvedSchema
@@ -54,7 +54,7 @@ extension TypedParameter {
5454
}
5555

5656
/// The location of the parameter in the HTTP request.
57-
var location: ResolvedParameter.Context.Location {
57+
var location: OpenAPI.Parameter.Context.Location {
5858
parameter.location
5959
}
6060

@@ -117,7 +117,7 @@ extension FileTranslator {
117117
) throws -> TypedParameter? {
118118

119119
// Collect the parameter
120-
let parameter: ResolvedParameter
120+
let parameter: OpenAPI.Parameter
121121
switch unresolvedParameter {
122122
case let .a(ref):
123123
parameter = try components.lookup(ref)
@@ -228,10 +228,7 @@ extension FileTranslator {
228228
/// Can be either a reference or an inline parameter.
229229
typealias UnresolvedParameter = Either<JSONReference<OpenAPI.Parameter>, OpenAPI.Parameter>
230230

231-
/// A resolved OpenAPI parameter.
232-
typealias ResolvedParameter = OpenAPI.Parameter
233-
234-
extension ResolvedParameter.Context.Location {
231+
extension OpenAPI.Parameter.Context.Location {
235232

236233
/// A name of the location usable as a Swift type name.
237234
var shortTypeName: String {

Sources/_OpenAPIGeneratorCore/Translator/Parameters/translateParameter.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ extension TypesFileTranslator {
6060
componentKey: OpenAPI.ComponentKey,
6161
parameter: TypedParameter
6262
) throws -> [Declaration] {
63-
let typeName = TypeAssigner.typeName(for: componentKey, of: ResolvedParameter.self)
63+
let typeName = TypeAssigner.typeName(for: componentKey, of: OpenAPI.Parameter.self)
6464
return try translateParameterInTypes(
6565
typeName: typeName,
6666
parameter: parameter

Sources/_OpenAPIGeneratorCore/Translator/RequestBody/TypedRequestBody.swift

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ import OpenAPIKit30
1717
struct TypedRequestBody {
1818

1919
/// The OpenAPI request body.
20-
var request: ResolvedRequest
20+
var request: OpenAPI.Request
2121

2222
/// The computed type usage.
2323
var typeUsage: TypeUsage
@@ -84,7 +84,7 @@ extension FileTranslator {
8484
from unresolvedRequest: UnresolvedRequest
8585
) throws -> TypedRequestBody? {
8686

87-
let request: ResolvedRequest
87+
let request: OpenAPI.Request
8888
let isInlined: Bool
8989
switch unresolvedRequest {
9090
case .a(let reference):
@@ -118,6 +118,3 @@ extension FileTranslator {
118118
///
119119
/// Can be either a reference or an inline request.
120120
typealias UnresolvedRequest = Either<JSONReference<OpenAPI.Request>, OpenAPI.Request>
121-
122-
/// A resolved OpenAPI request.
123-
typealias ResolvedRequest = OpenAPI.Request

Sources/_OpenAPIGeneratorCore/Translator/Responses/ResponseKind.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ enum ResponseKind {
4545
case _5XX
4646

4747
/// Creates a new range that matches the specified OpenAPI range.
48-
init(_ range: ResolvedResponse.StatusCode.Range) {
48+
init(_ range: OpenAPI.Response.StatusCode.Range) {
4949
switch range {
5050
case .information:
5151
self = ._1XX
@@ -191,7 +191,7 @@ extension ResponseKind: CustomStringConvertible {
191191
}
192192
}
193193

194-
extension ResolvedResponse.StatusCode.Code {
194+
extension OpenAPI.Response.StatusCode.Code {
195195

196196
/// Returns the matching OpenAPI response kind.
197197
var asKind: ResponseKind {

Sources/_OpenAPIGeneratorCore/Translator/Responses/TypedResponse.swift

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ import OpenAPIKit30
1717
struct TypedResponse {
1818

1919
/// The OpenAPI response.
20-
var response: ResolvedResponse
20+
var response: OpenAPI.Response
2121

2222
/// The computed type usage.
2323
var typeUsage: TypeUsage
@@ -39,7 +39,7 @@ extension FileTranslator {
3939
) throws -> TypedResponse {
4040
let unresolvedResponse = outcome.response
4141
let typeName: TypeName
42-
let response: ResolvedResponse
42+
let response: OpenAPI.Response
4343
let isInlined: Bool
4444
switch unresolvedResponse {
4545
case .a(let reference):
@@ -64,6 +64,3 @@ extension FileTranslator {
6464
///
6565
/// Can be either a reference or an inline response.
6666
typealias UnresolvedResponse = Either<JSONReference<OpenAPI.Response>, OpenAPI.Response>
67-
68-
/// A resolved OpenAPI response.
69-
typealias ResolvedResponse = OpenAPI.Response

Sources/_OpenAPIGeneratorCore/Translator/Responses/TypedResponseHeader.swift

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ import OpenAPIKit30
1818
struct TypedResponseHeader {
1919

2020
/// The OpenAPI response header.
21-
var header: ResolvedHeader
21+
var header: OpenAPI.Header
2222

2323
/// The name of the header.
2424
var name: String
@@ -65,7 +65,7 @@ extension FileTranslator {
6565
/// headers are specified in the OpenAPI document, or if all headers are
6666
/// unsupported.
6767
func typedResponseHeaders(
68-
from response: ResolvedResponse,
68+
from response: OpenAPI.Response,
6969
inParent parent: TypeName
7070
) throws -> [TypedResponseHeader] {
7171
guard let headers = response.headers else {
@@ -93,7 +93,7 @@ extension FileTranslator {
9393
) throws -> TypedResponseHeader? {
9494

9595
// Collect the header
96-
let header: ResolvedHeader
96+
let header: OpenAPI.Header
9797
switch unresolvedHeader {
9898
case let .a(ref):
9999
header = try components.lookup(ref)
@@ -169,6 +169,3 @@ extension FileTranslator {
169169
///
170170
/// Can be either a reference or an inline response header.
171171
typealias UnresolvedHeader = Either<JSONReference<OpenAPI.Header>, OpenAPI.Header>
172-
173-
/// A resolved OpenAPI response header.
174-
typealias ResolvedHeader = OpenAPI.Header

Sources/_OpenAPIGeneratorCore/Translator/Responses/translateResponse.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ extension TypesFileTranslator {
128128
) throws -> Declaration {
129129
let typeName = TypeAssigner.typeName(
130130
for: componentKey,
131-
of: ResolvedResponse.self
131+
of: OpenAPI.Response.self
132132
)
133133
return try translateResponseInTypes(
134134
typeName: typeName,

Sources/_OpenAPIGeneratorCore/Translator/Responses/translateResponseHeader.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ extension TypesFileTranslator {
4747
componentKey: OpenAPI.ComponentKey,
4848
header: TypedResponseHeader
4949
) throws -> [Declaration] {
50-
let typeName = TypeAssigner.typeName(for: componentKey, of: ResolvedHeader.self)
50+
let typeName = TypeAssigner.typeName(for: componentKey, of: OpenAPI.Header.self)
5151
return try translateResponseHeaderInTypes(
5252
typeName: typeName,
5353
header: header

Sources/_OpenAPIGeneratorCore/Translator/ServerTranslator/translateServerMethod.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ extension ServerFileTranslator {
2424
let inputTypeName = operation.inputTypeName
2525

2626
func locationSpecificInputDecl(
27-
locatedIn location: ResolvedParameter.Context.Location,
27+
locatedIn location: OpenAPI.Parameter.Context.Location,
2828
fromParameters parameters: [UnresolvedParameter]
2929
) throws -> Declaration {
3030
let variableName = location.shortVariableName
@@ -75,7 +75,7 @@ extension ServerFileTranslator {
7575
inputMemberDecls.append(bodyDecl)
7676

7777
func functionArgumentForLocation(
78-
_ location: ResolvedParameter.Context.Location
78+
_ location: OpenAPI.Parameter.Context.Location
7979
) -> FunctionArgumentDescription {
8080
.init(
8181
label: location.shortVariableName,

Sources/_OpenAPIGeneratorCore/Translator/TypeAssignment/TypeAssigner.swift

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -381,13 +381,13 @@ struct TypeAssigner {
381381
///
382382
/// # Mapping
383383
/// - `#/components/schemas` -> `JSONSchema`
384-
/// - `#/components/responses` -> `ResolvedResponse`
384+
/// - `#/components/responses` -> `OpenAPI.Response`
385385
/// - `#/components/callbacks` -> `OpenAPI.Callbacks`
386-
/// - `#/components/parameters` -> `ResolvedParameter`
386+
/// - `#/components/parameters` -> `OpenAPI.Parameter`
387387
/// (includes request headers)
388388
/// - `#/components/examples` -> `OpenAPI.Example`
389-
/// - `#/components/requestBodies` -> `ResolvedRequest`
390-
/// - `#/components/headers` -> `ResolvedHeader` (response headers)
389+
/// - `#/components/requestBodies` -> `OpenAPI.Request`
390+
/// - `#/components/headers` -> `OpenAPI.Header` (response headers)
391391
/// - `#/components/securitySchemes` -> `OpenAPI.SecurityScheme`
392392
/// - `#/components/links` -> `OpenAPI.Link`
393393
///
@@ -460,7 +460,7 @@ fileprivate extension TypeLocation {
460460
case .schemas:
461461
return TypeAssigner.typeName(for: JSONSchema.self)
462462
case .parameters:
463-
return TypeAssigner.typeName(for: ResolvedParameter.self)
463+
return TypeAssigner.typeName(for: OpenAPI.Parameter.self)
464464
}
465465
}
466466
}

Sources/_OpenAPIGeneratorCore/Translator/TypesTranslator/translateComponentHeaders.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,15 @@ extension TypesFileTranslator {
2020
/// - Parameter headers: The reusable response headers.
2121
/// - Returns: An enum declaration representing the headers namespace.
2222
func translateComponentHeaders(
23-
_ headers: OpenAPI.ComponentDictionary<ResolvedHeader>
23+
_ headers: OpenAPI.ComponentDictionary<OpenAPI.Header>
2424
) throws -> Declaration {
2525

2626
let typedHeaders: [(OpenAPI.ComponentKey, TypedResponseHeader)] =
2727
try headers
2828
.compactMap { key, header in
2929
let parent = TypeAssigner.typeName(
3030
for: key,
31-
of: ResolvedHeader.self
31+
of: OpenAPI.Header.self
3232
)
3333
guard
3434
let value = try typedResponseHeader(
@@ -51,7 +51,7 @@ extension TypesFileTranslator {
5151
}
5252

5353
let componentsParametersEnum = Declaration.commentable(
54-
ResolvedHeader.sectionComment(),
54+
OpenAPI.Header.sectionComment(),
5555
.enum(
5656
accessModifier: config.access,
5757
name: Constants.Components.Headers.namespace,

Sources/_OpenAPIGeneratorCore/Translator/TypesTranslator/translateComponentParameters.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,15 @@ extension TypesFileTranslator {
2020
/// - Parameter parameters: The reusable request parameters.
2121
/// - Returns: An enum declaration representing the parameters namespace.
2222
func translateComponentParameters(
23-
_ parameters: OpenAPI.ComponentDictionary<ResolvedParameter>
23+
_ parameters: OpenAPI.ComponentDictionary<OpenAPI.Parameter>
2424
) throws -> Declaration {
2525

2626
let typedParameters: [(OpenAPI.ComponentKey, TypedParameter)] =
2727
try parameters
2828
.compactMap { key, parameter in
2929
let parent = TypeAssigner.typeName(
3030
for: key,
31-
of: ResolvedParameter.self
31+
of: OpenAPI.Parameter.self
3232
)
3333
guard
3434
let value = try parseAsTypedParameter(
@@ -50,7 +50,7 @@ extension TypesFileTranslator {
5050
}
5151

5252
let componentsParametersEnum = Declaration.commentable(
53-
ResolvedParameter.sectionComment(),
53+
OpenAPI.Parameter.sectionComment(),
5454
.enum(
5555
accessModifier: config.access,
5656
name: Constants.Components.Parameters.namespace,

Sources/_OpenAPIGeneratorCore/Translator/TypesTranslator/translateComponentRequestBodies.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,15 @@ extension TypesFileTranslator {
2020
/// - Parameter requestBodies: The reusable request bodies.
2121
/// - Returns: An enum declaration representing the requestBodies namespace.
2222
func translateComponentRequestBodies(
23-
_ items: OpenAPI.ComponentDictionary<ResolvedRequest>
23+
_ items: OpenAPI.ComponentDictionary<OpenAPI.Request>
2424
) throws -> Declaration {
2525

2626
let typedItems: [TypedRequestBody] =
2727
try items
2828
.compactMap { key, item in
2929
let typeName = TypeAssigner.typeName(
3030
for: key,
31-
of: ResolvedRequest.self
31+
of: OpenAPI.Request.self
3232
)
3333
return try typedRequestBody(
3434
typeName: typeName,
@@ -44,7 +44,7 @@ extension TypesFileTranslator {
4444
}
4545

4646
let componentsEnum = Declaration.commentable(
47-
ResolvedRequest.sectionComment(),
47+
OpenAPI.Request.sectionComment(),
4848
.enum(
4949
accessModifier: config.access,
5050
name: Constants.Components.RequestBodies.namespace,

Sources/_OpenAPIGeneratorCore/Translator/TypesTranslator/translateComponentResponses.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,15 @@ extension TypesFileTranslator {
2020
/// - Parameter responses: The reusable responses.
2121
/// - Returns: An enum declaration representing the responses namespace.
2222
func translateComponentResponses(
23-
_ responses: OpenAPI.ComponentDictionary<ResolvedResponse>
23+
_ responses: OpenAPI.ComponentDictionary<OpenAPI.Response>
2424
) throws -> Declaration {
2525

2626
let typedResponses: [TypedResponse] =
2727
responses
2828
.map { key, response in
2929
let typeName = TypeAssigner.typeName(
3030
for: key,
31-
of: ResolvedResponse.self
31+
of: OpenAPI.Response.self
3232
)
3333
let value = TypedResponse(
3434
response: response,
@@ -47,7 +47,7 @@ extension TypesFileTranslator {
4747
}
4848

4949
let componentsResponsesEnum = Declaration.commentable(
50-
ResolvedResponse.sectionComment(),
50+
OpenAPI.Response.sectionComment(),
5151
.enum(
5252
accessModifier: config.access,
5353
name: Constants.Components.Responses.namespace,

Sources/_OpenAPIGeneratorCore/Translator/TypesTranslator/translateOperations.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ extension TypesFileTranslator {
2525
let inputTypeName = description.inputTypeName
2626

2727
func propertyBlueprintForNamespacedStruct(
28-
locatedIn location: ResolvedParameter.Context.Location,
28+
locatedIn location: OpenAPI.Parameter.Context.Location,
2929
withPropertiesFrom parameters: [UnresolvedParameter]
3030
) throws -> PropertyBlueprint {
3131
let inputTypeName = description.inputTypeName

Tests/OpenAPIGeneratorCoreTests/Translator/TypeAssignment/Test_TypeAssigner.swift

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,30 +26,30 @@ class Test_TypeAssigner: Test_Core {
2626
)
2727
)
2828
try XCTAssertEqual(
29-
TypeAssigner.typeName(for: JSONReference<ResolvedParameter>.component(named: "mumble")),
29+
TypeAssigner.typeName(for: JSONReference<OpenAPI.Parameter>.component(named: "mumble")),
3030
newTypeName(
3131
swiftFQName: "Components.Parameters.mumble",
3232
jsonFQName: "#/components/parameters/mumble"
3333
)
3434
)
3535
try XCTAssertEqual(
36-
TypeAssigner.typeName(for: JSONReference<ResolvedHeader>.component(named: "mumble")),
36+
TypeAssigner.typeName(for: JSONReference<OpenAPI.Header>.component(named: "mumble")),
3737
newTypeName(
3838
swiftFQName: "Components.Headers.mumble",
3939
jsonFQName: "#/components/headers/mumble"
4040
)
4141

4242
)
4343
try XCTAssertEqual(
44-
TypeAssigner.typeName(for: JSONReference<ResolvedRequest>.component(named: "mumble")),
44+
TypeAssigner.typeName(for: JSONReference<OpenAPI.Request>.component(named: "mumble")),
4545
newTypeName(
4646
swiftFQName: "Components.RequestBodies.mumble",
4747
jsonFQName: "#/components/requestBodies/mumble"
4848
)
4949

5050
)
5151
try XCTAssertEqual(
52-
TypeAssigner.typeName(for: JSONReference<ResolvedResponse>.component(named: "mumble")),
52+
TypeAssigner.typeName(for: JSONReference<OpenAPI.Response>.component(named: "mumble")),
5353
newTypeName(
5454
swiftFQName: "Components.Responses.mumble",
5555
jsonFQName: "#/components/responses/mumble"

0 commit comments

Comments
 (0)