Add XML coding strategy

Author: ugocottin

Sources/_OpenAPIGeneratorCore/Translator/CommonTypes/Constants.swift

@@ -371,6 +371,9 @@ enum Constants {
 
         /// The substring used in method names for the multipart coding strategy.
         static let multipart: String = "Multipart"
+
+        /// The substring used in method names for the XML coding strategy.
+        static let xml: String = "XML"
     }
 
     /// Constants related to types used in many components.

Sources/_OpenAPIGeneratorCore/Translator/Content/CodingStrategy.swift

@@ -30,6 +30,9 @@ enum CodingStrategy: String, Hashable, Sendable {
     /// A strategy using multipart/form-data.
     case multipart
 
+    /// A strategy using optional CustomCoder.
+    case xml
+
     /// The name of the coding strategy in the runtime library.
     var runtimeName: String {
         switch self {
@@ -38,6 +41,7 @@ enum CodingStrategy: String, Hashable, Sendable {
         case .binary: return Constants.CodingStrategy.binary
         case .urlEncodedForm: return Constants.CodingStrategy.urlEncodedForm
         case .multipart: return Constants.CodingStrategy.multipart
+        case .xml: return Constants.CodingStrategy.xml
         }
     }
 }

Sources/_OpenAPIGeneratorCore/Translator/Content/ContentInspector.swift

@@ -21,8 +21,9 @@ extension FileTranslator {
     ///
     /// Priority:
     /// 1. JSON
-    /// 2. text
-    /// 3. binary
+    /// 2. XML
+    /// 3. text
+    /// 4. binary
     ///
     /// - Parameters:
     ///   - map: The content map from the OpenAPI document.
@@ -109,8 +110,9 @@ extension FileTranslator {
     ///
     /// Priority:
     /// 1. JSON
-    /// 2. text
-    /// 3. binary
+    /// 2. XML
+    /// 3. text
+    /// 4. binary
     ///
     /// - Parameters:
     ///   - map: The content map from the OpenAPI document.
@@ -128,7 +130,7 @@ extension FileTranslator {
         let mapWithContentTypes = try map.map { key, content in try (type: key.asGeneratorContentType, value: content) }
 
         let chosenContent: (type: ContentType, schema: SchemaContent, content: OpenAPI.Content)?
-        if let (contentType, contentValue) = mapWithContentTypes.first(where: { $0.type.isJSON }) {
+        if let (contentType, contentValue) = mapWithContentTypes.first(where: { $0.type.isJSON || $0.type.isXml }) {
             chosenContent = (contentType, .init(contentType: contentType, schema: contentValue.schema), contentValue)
         } else if !excludeBinary,
             let (contentType, contentValue) = mapWithContentTypes.first(where: { $0.type.isBinary })
@@ -161,6 +163,7 @@ extension FileTranslator {
     ///
     /// Priority of checking for known MIME types:
     /// 1. JSON
+    /// 2. XML
     /// 2. text
     /// 3. binary
     ///
@@ -189,6 +192,7 @@ extension FileTranslator {
             )
         }
         if contentType.isJSON { return .init(contentType: contentType, schema: contentValue.schema) }
+        if contentType.isXml { return .init(contentType: contentType, schema: contentValue.schema) }
         if contentType.isUrlEncodedForm { return .init(contentType: contentType, schema: contentValue.schema) }
         if contentType.isMultipart {
             guard isRequired else {

Sources/_OpenAPIGeneratorCore/Translator/Content/ContentType.swift

@@ -56,9 +56,14 @@ struct ContentType: Hashable {
         /// The type is encoded as an async sequence of parts.
         case multipart
 
+        /// A content type for XML.
+        ///
+        /// The bytes are provided to a CustomCoder.
+        case xml
+
         /// Creates a category from the provided type and subtype.
         ///
-        /// First checks if the provided content type is a JSON, then text,
+        /// First checks if the provided content type is a JSON, then XML, then text,
         /// and uses binary if none of the two match.
         /// - Parameters:
         ///   - lowercasedType: The first component of the MIME type.
@@ -68,6 +73,10 @@ struct ContentType: Hashable {
             if (lowercasedType == "application" && lowercasedSubtype == "json") || lowercasedSubtype.hasSuffix("+json")
             {
                 self = .json
+            } else if (lowercasedType == "application" && lowercasedSubtype == "xml")
+                || lowercasedSubtype.hasSuffix("+xml")
+            {
+                self = .xml
             } else if lowercasedType == "application" && lowercasedSubtype == "x-www-form-urlencoded" {
                 self = .urlEncodedForm
             } else if lowercasedType == "multipart" && lowercasedSubtype == "form-data" {
@@ -84,6 +93,7 @@ struct ContentType: Hashable {
             case .binary: return .binary
             case .urlEncodedForm: return .urlEncodedForm
             case .multipart: return .multipart
+            case .xml: return .xml
             }
         }
     }
@@ -214,12 +224,17 @@ struct ContentType: Hashable {
     /// A Boolean value that indicates whether the content type
     /// is a multipart form.
     var isMultipart: Bool { category == .multipart }
+    /// A Boolean value that indicates whether the content type
+    /// is a type of XML.
+    var isXml: Bool { category == .xml }
 
     /// The content type `text/plain`.
     static var textPlain: Self { try! .init(string: "text/plain") }
 
     /// The content type `application/json`.
     static var applicationJSON: Self { try! .init(string: "application/json") }
+    /// The content type `application/xml`.
+    static var applicationXML: Self { try! .init(string: "application/xml") }
 
     /// The content type `application/octet-stream`.
     static var applicationOctetStream: Self { try! .init(string: "application/octet-stream") }

Sources/_OpenAPIGeneratorCore/Translator/Multipart/translateMultipart.swift

@@ -353,7 +353,7 @@ extension FileTranslator {
             ])
         let bodyExpr: Expression
         switch codingStrategy {
-        case .json, .uri, .urlEncodedForm:
+        case .json, .xml, .uri, .urlEncodedForm:
             // Buffering.
             bodyExpr = .try(.await(converterExpr))
         case .binary, .multipart:

Sources/_OpenAPIGeneratorCore/Translator/RequestBody/translateRequestBody.swift

@@ -275,7 +275,7 @@ extension ServerFileTranslator {
                 )
             let bodyExpr: Expression
             switch codingStrategy {
-            case .json, .uri, .urlEncodedForm:
+            case .json, .xml, .uri, .urlEncodedForm:
                 // Buffering.
                 bodyExpr = .try(.await(converterExpr))
             case .binary, .multipart:

Sources/_OpenAPIGeneratorCore/Translator/Responses/translateResponseOutcome.swift

@@ -267,7 +267,7 @@ extension ClientFileTranslator {
                     )
                 let bodyExpr: Expression
                 switch codingStrategy {
-                case .json, .uri, .urlEncodedForm:
+                case .json, .xml, .uri, .urlEncodedForm:
                     // Buffering.
                     bodyExpr = .try(.await(converterExpr))
                 case .binary, .multipart:

Tests/OpenAPIGeneratorCoreTests/Translator/Content/Test_ContentType.swift

@@ -46,7 +46,7 @@ final class Test_ContentType: Test_Core {
                 ), ("text/plain", .binary, "text", "plain", "", "text/plain", "text/plain", "text/plain"),
                 ("*/*", .binary, "*", "*", "", "*/*", "*/*", "*/*"),
                 (
-                    "application/xml", .binary, "application", "xml", "", "application/xml", "application/xml",
+                    "application/xml", .xml, "application", "xml", "", "application/xml", "application/xml",
                     "application/xml"
                 ),
                 (