- fixes #172

- moved error definitions under main namespace - renames types to avoid conflicts with native types or naming conventions Signed-off-by: Vincent Biret <vibiret@microsoft.com>
microsoft · Feb 7, 2022 · e3c02f2 · e3c02f2
1 parent 7b0f3cd
commit e3c02f2
Show file tree

Hide file tree

Showing 19 changed files with 161 additions and 151 deletions.
diff --git a/src/Microsoft.OpenApi.OData.Reader/Generator/OpenApiErrorSchemaGenerator.cs b/src/Microsoft.OpenApi.OData.Reader/Generator/OpenApiErrorSchemaGenerator.cs
@@ -27,28 +27,36 @@ internal static class OpenApiErrorSchemaGenerator
         public static IDictionary<string, OpenApiSchema> CreateODataErrorSchemas(this ODataContext context)
         {
             Utils.CheckArgumentNull(context, nameof(context));
+            var rootNamespaceName = context.GetErrorNamespaceName();
 
             return new Dictionary<string, OpenApiSchema>()
             {
-                // odata.error
-                { "odata.error", CreateErrorSchema() },
-
-                // odata.error.main
-                { "odata.error.main", CreateErrorMainSchema() },
-
-                // odata.error.detail
-                { "odata.error.detail", CreateErrorDetailSchema() },
-
-                // odata.error.innererror
-                { "odata.error.innererror", CreateInnerErrorSchema(context) }
+                { $"{rootNamespaceName}odataerror", CreateErrorSchema(rootNamespaceName) },
+                { $"{rootNamespaceName}mainerror", CreateErrorMainSchema(rootNamespaceName) },
+                { $"{rootNamespaceName}errordetails", CreateErrorDetailSchema() },
+                { $"{rootNamespaceName}innererror", CreateInnerErrorSchema(context) }
             };
         }
 
         /// <summary>
-        /// Create <see cref="OpenApiSchema"/> for "odata.error".
+        /// Gets the error namespace name based on the root namespace of the model.
+        /// </summary>
+        /// <param name="context">The OData to Open API context.</param>
+        /// <returns>The error namespace name.</returns>
+        public static string GetErrorNamespaceName(this ODataContext context) {
+            Utils.CheckArgumentNull(context, nameof(context));
+            var rootNamespaceName = context.Model.DeclaredNamespaces.OrderBy(ns => ns.Count(y => y == '.')).FirstOrDefault();
+            rootNamespaceName += (string.IsNullOrEmpty(rootNamespaceName) ? string.Empty : ".") +
+                                "odataerrors.";
+            return rootNamespaceName;
+        }
+
+        /// <summary>
+        /// Create <see cref="OpenApiSchema"/> for the error.
         /// </summary>
         /// <returns>The created <see cref="OpenApiSchema"/>.</returns>
-        public static OpenApiSchema CreateErrorSchema()
+        /// <param name="rootNamespaceName">The root namespace name. With a trailing dot.</param>
+        public static OpenApiSchema CreateErrorSchema(string rootNamespaceName)
         {
             return new OpenApiSchema
             {
@@ -66,7 +74,7 @@ public static OpenApiSchema CreateErrorSchema()
                             Reference = new OpenApiReference
                             {
                                 Type = ReferenceType.Schema,
-                                Id = "odata.error.main"
+                                Id = $"{rootNamespaceName}mainerror"
                             }
                         }
                     }
@@ -100,10 +108,11 @@ public static OpenApiSchema CreateInnerErrorSchema(ODataContext context)
         }
 
         /// <summary>
-        /// Create <see cref="OpenApiSchema"/> for "odata.error.main".
+        /// Create <see cref="OpenApiSchema"/> for main property of the error.
         /// </summary>
+        /// <param name="rootNamespaceName">The root namespace name. With a trailing dot.</param>
         /// <returns>The created <see cref="OpenApiSchema"/>.</returns>
-        public static OpenApiSchema CreateErrorMainSchema()
+        public static OpenApiSchema CreateErrorMainSchema(string rootNamespaceName)
         {
             return new OpenApiSchema
             {
@@ -133,7 +142,7 @@ public static OpenApiSchema CreateErrorMainSchema()
                                 Reference = new OpenApiReference
                                 {
                                     Type = ReferenceType.Schema,
-                                    Id = "odata.error.detail"
+                                    Id = $"{rootNamespaceName}errordetails"
                                 }
                             }
                         }
@@ -145,7 +154,7 @@ public static OpenApiSchema CreateErrorMainSchema()
                             Reference = new OpenApiReference
                             {
                                 Type = ReferenceType.Schema,
-                                Id = "odata.error.innererror"
+                                Id = $"{rootNamespaceName}innererror"
                             }
                         }
                     }
@@ -154,7 +163,7 @@ public static OpenApiSchema CreateErrorMainSchema()
         }
 
         /// <summary>
-        /// Create <see cref="OpenApiSchema"/> for "odata.error.detail".
+        /// Create <see cref="OpenApiSchema"/> for detail property of the error.
         /// </summary>
         /// <returns>The created <see cref="OpenApiSchema"/>.</returns>
         public static OpenApiSchema CreateErrorDetailSchema()

diff --git a/src/Microsoft.OpenApi.OData.Reader/Generator/OpenApiResponseGenerator.cs b/src/Microsoft.OpenApi.OData.Reader/Generator/OpenApiResponseGenerator.cs
@@ -81,7 +81,7 @@ public static IDictionary<string, OpenApiResponse> CreateResponses(this ODataCon
 
             var responses =  new Dictionary<string, OpenApiResponse>
             {
-                { "error", CreateErrorResponse() }
+                { "error", context.CreateErrorResponse() }
             };
 
             if(context.Settings.EnableDollarCountPath)
@@ -259,8 +259,9 @@ private static OpenApiResponse CreateCountResponse()
             };
         }
 
-        private static OpenApiResponse CreateErrorResponse()
+        private static OpenApiResponse CreateErrorResponse(this ODataContext context)
         {
+            var errorNamespaceName = context.GetErrorNamespaceName();
             return new OpenApiResponse
             {
                 Description = "error",
@@ -275,7 +276,7 @@ private static OpenApiResponse CreateErrorResponse()
                                 Reference = new OpenApiReference
                                 {
                                     Type = ReferenceType.Schema,
-                                    Id = "odata.error"
+                                    Id = $"{errorNamespaceName}odataerror"
                                 }
                             }
                         }

diff --git a/test/Microsoft.OpenAPI.OData.Reader.Tests/Generator/OpenApiResponseGeneratorTests.cs b/test/Microsoft.OpenAPI.OData.Reader.Tests/Generator/OpenApiResponseGeneratorTests.cs
@@ -113,7 +113,7 @@ public void CanSerializeAsJsonFromTheCreatedResponses()
   ""content"": {
     ""application/json"": {
       ""schema"": {
-        ""$ref"": ""#/components/schemas/odata.error""
+        ""$ref"": ""#/components/schemas/odataerrors.odataerror""
       }
     }
   }

diff --git a/test/Microsoft.OpenAPI.OData.Reader.Tests/Resources/Basic.OpenApi.V2.json b/test/Microsoft.OpenAPI.OData.Reader.Tests/Resources/Basic.OpenApi.V2.json
@@ -2240,18 +2240,18 @@
         }
       }
     },
-    "odata.error": {
+    "DefaultNs.odataerrors.odataerror": {
       "required": [
         "error"
       ],
       "type": "object",
       "properties": {
         "error": {
-          "$ref": "#/definitions/odata.error.main"
+          "$ref": "#/definitions/DefaultNs.odataerrors.mainerror"
         }
       }
     },
-    "odata.error.main": {
+    "DefaultNs.odataerrors.mainerror": {
       "required": [
         "code",
         "message"
@@ -2270,15 +2270,15 @@
         "details": {
           "type": "array",
           "items": {
-            "$ref": "#/definitions/odata.error.detail"
+            "$ref": "#/definitions/DefaultNs.odataerrors.errordetails"
           }
         },
         "innererror": {
-          "$ref": "#/definitions/odata.error.innererror"
+          "$ref": "#/definitions/DefaultNs.odataerrors.innererror"
         }
       }
     },
-    "odata.error.detail": {
+    "DefaultNs.odataerrors.errordetails": {
       "required": [
         "code",
         "message"
@@ -2296,7 +2296,7 @@
         }
       }
     },
-    "odata.error.innererror": {
+    "DefaultNs.odataerrors.innererror": {
       "description": "The structure of this object is service-specific",
       "type": "object"
     },
@@ -2391,7 +2391,7 @@
     "error": {
       "description": "error",
       "schema": {
-        "$ref": "#/definitions/odata.error"
+        "$ref": "#/definitions/DefaultNs.odataerrors.odataerror"
       }
     },
     "ODataCountResponse": {

diff --git a/test/Microsoft.OpenAPI.OData.Reader.Tests/Resources/Basic.OpenApi.V2.yaml b/test/Microsoft.OpenAPI.OData.Reader.Tests/Resources/Basic.OpenApi.V2.yaml
@@ -1487,14 +1487,14 @@ definitions:
         '@odata.type': DefaultNs.City
       CountryOrRegion:
         '@odata.type': DefaultNs.CountryOrRegion
-  odata.error:
+  DefaultNs.odataerrors.odataerror:
     required:
       - error
     type: object
     properties:
       error:
-        $ref: '#/definitions/odata.error.main'
-  odata.error.main:
+        $ref: '#/definitions/DefaultNs.odataerrors.mainerror'
+  DefaultNs.odataerrors.mainerror:
     required:
       - code
       - message
@@ -1509,10 +1509,10 @@ definitions:
       details:
         type: array
         items:
-          $ref: '#/definitions/odata.error.detail'
+          $ref: '#/definitions/DefaultNs.odataerrors.errordetails'
       innererror:
-        $ref: '#/definitions/odata.error.innererror'
-  odata.error.detail:
+        $ref: '#/definitions/DefaultNs.odataerrors.innererror'
+  DefaultNs.odataerrors.errordetails:
     required:
       - code
       - message
@@ -1524,7 +1524,7 @@ definitions:
         type: string
       target:
         type: string
-  odata.error.innererror:
+  DefaultNs.odataerrors.innererror:
     description: The structure of this object is service-specific
     type: object
   ODataCountResponse:
@@ -1594,7 +1594,7 @@ responses:
   error:
     description: error
     schema:
-      $ref: '#/definitions/odata.error'
+      $ref: '#/definitions/DefaultNs.odataerrors.odataerror'
   ODataCountResponse:
     description: The count of the resource
     schema:

diff --git a/test/Microsoft.OpenAPI.OData.Reader.Tests/Resources/Basic.OpenApi.json b/test/Microsoft.OpenAPI.OData.Reader.Tests/Resources/Basic.OpenApi.json
@@ -2487,18 +2487,18 @@
           }
         }
       },
-      "odata.error": {
+      "DefaultNs.odataerrors.odataerror": {
         "required": [
           "error"
         ],
         "type": "object",
         "properties": {
           "error": {
-            "$ref": "#/components/schemas/odata.error.main"
+            "$ref": "#/components/schemas/DefaultNs.odataerrors.mainerror"
           }
         }
       },
-      "odata.error.main": {
+      "DefaultNs.odataerrors.mainerror": {
         "required": [
           "code",
           "message"
@@ -2518,15 +2518,15 @@
           "details": {
             "type": "array",
             "items": {
-              "$ref": "#/components/schemas/odata.error.detail"
+              "$ref": "#/components/schemas/DefaultNs.odataerrors.errordetails"
             }
           },
           "innererror": {
-            "$ref": "#/components/schemas/odata.error.innererror"
+            "$ref": "#/components/schemas/DefaultNs.odataerrors.innererror"
           }
         }
       },
-      "odata.error.detail": {
+      "DefaultNs.odataerrors.errordetails": {
         "required": [
           "code",
           "message"
@@ -2545,7 +2545,7 @@
           }
         }
       },
-      "odata.error.innererror": {
+      "DefaultNs.odataerrors.innererror": {
         "type": "object",
         "description": "The structure of this object is service-specific"
       },
@@ -2608,7 +2608,7 @@
         "content": {
           "application/json": {
             "schema": {
-              "$ref": "#/components/schemas/odata.error"
+              "$ref": "#/components/schemas/DefaultNs.odataerrors.odataerror"
             }
           }
         }

diff --git a/test/Microsoft.OpenAPI.OData.Reader.Tests/Resources/Basic.OpenApi.yaml b/test/Microsoft.OpenAPI.OData.Reader.Tests/Resources/Basic.OpenApi.yaml
@@ -1649,14 +1649,14 @@ components:
           '@odata.type': DefaultNs.City
         CountryOrRegion:
           '@odata.type': DefaultNs.CountryOrRegion
-    odata.error:
+    DefaultNs.odataerrors.odataerror:
       required:
         - error
       type: object
       properties:
         error:
-          $ref: '#/components/schemas/odata.error.main'
-    odata.error.main:
+          $ref: '#/components/schemas/DefaultNs.odataerrors.mainerror'
+    DefaultNs.odataerrors.mainerror:
       required:
         - code
         - message
@@ -1672,10 +1672,10 @@ components:
         details:
           type: array
           items:
-            $ref: '#/components/schemas/odata.error.detail'
+            $ref: '#/components/schemas/DefaultNs.odataerrors.errordetails'
         innererror:
-          $ref: '#/components/schemas/odata.error.innererror'
-    odata.error.detail:
+          $ref: '#/components/schemas/DefaultNs.odataerrors.innererror'
+    DefaultNs.odataerrors.errordetails:
       required:
         - code
         - message
@@ -1688,7 +1688,7 @@ components:
         target:
           type: string
           nullable: true
-    odata.error.innererror:
+    DefaultNs.odataerrors.innererror:
       type: object
       description: The structure of this object is service-specific
     ODataCountResponse:
@@ -1732,7 +1732,7 @@ components:
       content:
         application/json:
           schema:
-            $ref: '#/components/schemas/odata.error'
+            $ref: '#/components/schemas/DefaultNs.odataerrors.odataerror'
     ODataCountResponse:
       description: The count of the resource
       content: