Skip to content

Commit 5ceedcd

Browse files
authored
Merge pull request #173 from Microsoft/FixParameterInValueForV2
Fix to use display values for parameter location for V2 serialization.
2 parents 6341fdb + 7884c87 commit 5ceedcd

File tree

4 files changed

+14
-14
lines changed

4 files changed

+14
-14
lines changed

src/Microsoft.OpenApi/Models/OpenApiParameter.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ public void SerializeAsV2WithoutReference(IOpenApiWriter writer)
236236
else
237237
{
238238
writer.WriteProperty(OpenApiConstants.Name, Name);
239-
writer.WriteProperty(OpenApiConstants.In, In.ToString());
239+
writer.WriteProperty(OpenApiConstants.In, In.GetDisplayName());
240240
}
241241

242242
// description

test/Microsoft.OpenApi.Tests/Models/OpenApiDocumentTests.cs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1762,7 +1762,7 @@ public void SerializeAdvancedDocumentAsV2JsonWorks()
17621762
""parameters"": [
17631763
{
17641764
""name"": ""tags"",
1765-
""in"": ""Query"",
1765+
""in"": ""query"",
17661766
""description"": ""tags to filter by"",
17671767
""type"": ""array"",
17681768
""items"": {
@@ -1771,7 +1771,7 @@ public void SerializeAdvancedDocumentAsV2JsonWorks()
17711771
},
17721772
{
17731773
""name"": ""limit"",
1774-
""in"": ""Query"",
1774+
""in"": ""query"",
17751775
""description"": ""maximum number of results to return"",
17761776
""type"": ""integer"",
17771777
""format"": ""int32""
@@ -1954,7 +1954,7 @@ public void SerializeAdvancedDocumentAsV2JsonWorks()
19541954
""parameters"": [
19551955
{
19561956
""name"": ""id"",
1957-
""in"": ""Path"",
1957+
""in"": ""path"",
19581958
""description"": ""ID of pet to fetch"",
19591959
""required"": true,
19601960
""type"": ""integer"",
@@ -2033,7 +2033,7 @@ public void SerializeAdvancedDocumentAsV2JsonWorks()
20332033
""parameters"": [
20342034
{
20352035
""name"": ""id"",
2036-
""in"": ""Path"",
2036+
""in"": ""path"",
20372037
""description"": ""ID of pet to delete"",
20382038
""required"": true,
20392039
""type"": ""integer"",
@@ -2196,7 +2196,7 @@ public void SerializeAdvancedDocumentWithReferenceAsV2JsonWorks()
21962196
""parameters"": [
21972197
{
21982198
""name"": ""tags"",
2199-
""in"": ""Query"",
2199+
""in"": ""query"",
22002200
""description"": ""tags to filter by"",
22012201
""type"": ""array"",
22022202
""items"": {
@@ -2205,7 +2205,7 @@ public void SerializeAdvancedDocumentWithReferenceAsV2JsonWorks()
22052205
},
22062206
{
22072207
""name"": ""limit"",
2208-
""in"": ""Query"",
2208+
""in"": ""query"",
22092209
""description"": ""maximum number of results to return"",
22102210
""type"": ""integer"",
22112211
""format"": ""int32""
@@ -2289,7 +2289,7 @@ public void SerializeAdvancedDocumentWithReferenceAsV2JsonWorks()
22892289
""parameters"": [
22902290
{
22912291
""name"": ""id"",
2292-
""in"": ""Path"",
2292+
""in"": ""path"",
22932293
""description"": ""ID of pet to fetch"",
22942294
""required"": true,
22952295
""type"": ""integer"",
@@ -2326,7 +2326,7 @@ public void SerializeAdvancedDocumentWithReferenceAsV2JsonWorks()
23262326
""parameters"": [
23272327
{
23282328
""name"": ""id"",
2329-
""in"": ""Path"",
2329+
""in"": ""path"",
23302330
""description"": ""ID of pet to delete"",
23312331
""required"": true,
23322332
""type"": ""integer"",

test/Microsoft.OpenApi.Tests/Models/OpenApiOperationTests.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -421,11 +421,11 @@ public void SerializeAdvancedOperationAsV2JsonWorks()
421421
""parameters"": [
422422
{
423423
""name"": ""parameter1"",
424-
""in"": ""Path""
424+
""in"": ""path""
425425
},
426426
{
427427
""name"": ""parameter2"",
428-
""in"": ""Header""
428+
""in"": ""header""
429429
},
430430
{
431431
""name"": ""body"",
@@ -489,11 +489,11 @@ public void SerializeAdvancedOperationWithTagAndSecurityAsV2JsonWorks()
489489
""parameters"": [
490490
{
491491
""name"": ""parameter1"",
492-
""in"": ""Path""
492+
""in"": ""path""
493493
},
494494
{
495495
""name"": ""parameter2"",
496-
""in"": ""Header""
496+
""in"": ""header""
497497
},
498498
{
499499
""name"": ""body"",

test/Microsoft.OpenApi.Tests/Models/OpenApiParameterTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ public void SerializeReferencedParameterAsV2JsonWithoutReferenceWorks()
190190
var expected =
191191
@"{
192192
""name"": ""name1"",
193-
""in"": ""Path""
193+
""in"": ""path""
194194
}";
195195

196196
// Act

0 commit comments

Comments
 (0)