Merge pull request #963 from microsoft/mk/fix-example-with-empty-property-name

MaggieKimani1 · web-flow · commit a4ea02fc4f57 · 2022-08-14T01:25:39.000+03:00
Treat an empty property name as valid
diff --git a/src/Microsoft.OpenApi/Writers/OpenApiWriterBase.cs b/src/Microsoft.OpenApi/Writers/OpenApiWriterBase.cs
@@ -393,9 +393,9 @@ private bool IsScopeType(ScopeType type)
         /// <param name="name">property name</param>
         protected void VerifyCanWritePropertyName(string name)
         {
-            if (string.IsNullOrWhiteSpace(name))
+            if (name == null)
             {
-                throw Error.ArgumentNullOrWhiteSpace(nameof(name));
+                throw Error.ArgumentNull(nameof(name));
             }
 
             if (Scopes.Count == 0)

Original file line number	Diff line number	Diff line change
`@@ -393,9 +393,9 @@ private bool IsScopeType(ScopeType type)`
`393`	`393`	`/// <param name="name">property name</param>`
`394`	`394`	`protected void VerifyCanWritePropertyName(string name)`
`395`	`395`	`{`
`396`		`- if (string.IsNullOrWhiteSpace(name))`
	`396`	`+ if (name == null)`
`397`	`397`	`{`
`398`		`- throw Error.ArgumentNullOrWhiteSpace(nameof(name));`
	`398`	`+ throw Error.ArgumentNull(nameof(name));`
`399`	`399`	`}`
`400`	`400`
`401`	`401`	`if (Scopes.Count == 0)`