Skip to content

Commit e8937e8

Browse files
committed
review feedback
1 parent 8ab5b7c commit e8937e8

File tree

7 files changed

+13
-19
lines changed

7 files changed

+13
-19
lines changed

docs/usage/openapi.md

+3-3
Original file line numberDiff line numberDiff line change
@@ -36,11 +36,11 @@ public class Startup
3636
{
3737
public void ConfigureServices(IServiceCollection services)
3838
{
39-
IMvcCoreBuilder builder = services.AddMvcCore();
40-
services.AddJsonApi<AppDbContext>(mvcBuilder: builder);
39+
IMvcCoreBuilder mvcBuilder = services.AddMvcCore();
40+
services.AddJsonApi<AppDbContext>(mvcBuilder: mvcBuilder);
4141

4242
// Adds the Swashbuckle integration.
43-
services.AddOpenApi(builder);
43+
services.AddOpenApi(mvcBuilder);
4444
}
4545

4646
public void Configure(IApplicationBuilder app, IWebHostEnvironment env)

src/Examples/JsonApiDotNetCoreExample/JsonApiDotNetCoreExample.csproj

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
</PropertyGroup>
55

66
<ItemGroup>
7-
<ProjectReference Include="..\..\JsonApiDotNetCore.OpenApi\JsonApiDotNetCore.OpenApi.csproj" />
87
<ProjectReference Include="..\..\JsonApiDotNetCore\JsonApiDotNetCore.csproj" />
8+
<ProjectReference Include="..\..\JsonApiDotNetCore.OpenApi\JsonApiDotNetCore.OpenApi.csproj" />
99
</ItemGroup>
1010

1111
<ItemGroup>

src/Examples/JsonApiDotNetCoreExample/Startup.cs

+2-2
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,9 @@ public void ConfigureServices(IServiceCollection services)
4545
#endif
4646
});
4747

48-
IMvcCoreBuilder mvcCoreBuilder = services.AddMvcCore();
48+
IMvcCoreBuilder mvcBuilder = services.AddMvcCore();
4949

50-
services.AddOpenApi(mvcCoreBuilder);
50+
services.AddOpenApi(mvcBuilder);
5151

5252
using (CodeTimingSessionManager.Current.Measure("Configure JSON:API (startup)"))
5353
{

test/OpenApiTests/OpenApiDocumentTests.cs

+2-2
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@ public OpenApiDocumentTests()
2121
public async Task Retrieved_document_matches_expected_document()
2222
{
2323
// Arrange
24-
string embeddedResourceName = $"{nameof(OpenApiTests)}.openapi.json";
24+
string embeddedResourceName = $"{nameof(OpenApiTests)}.swagger.json";
2525
string expectedDocument = await LoadEmbeddedResourceAsync(embeddedResourceName);
26-
string requestUrl = $"swagger/{OpenApiStartup<OpenApiDbContext>.OpenApiDocumentName}/swagger.json";
26+
string requestUrl = $"swagger/v1/swagger.json";
2727

2828
// Act
2929
string actualDocument = await GetAsync(requestUrl);

test/OpenApiTests/OpenApiStartup.cs

+3-9
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,13 @@ namespace OpenApiTests
1313
public sealed class OpenApiStartup<TDbContext> : TestableStartup<TDbContext>
1414
where TDbContext : DbContext
1515
{
16-
internal const string OpenApiDocumentName = nameof(OpenApiTests);
17-
1816
public override void ConfigureServices(IServiceCollection services)
1917
{
20-
IMvcCoreBuilder mvcCoreBuilder = services.AddMvcCore();
18+
IMvcCoreBuilder mvcBuilder = services.AddMvcCore();
2119

22-
services.AddJsonApi<TDbContext>(SetJsonApiOptions, mvcBuilder: mvcCoreBuilder);
20+
services.AddJsonApi<TDbContext>(SetJsonApiOptions, mvcBuilder: mvcBuilder);
2321

24-
services.AddOpenApi(mvcCoreBuilder, options => options.SwaggerDoc(OpenApiDocumentName, new OpenApiInfo
25-
{
26-
Title = OpenApiDocumentName,
27-
Version = "1"
28-
}));
22+
services.AddOpenApi(mvcBuilder);
2923
}
3024

3125
public override void Configure(IApplicationBuilder app, IWebHostEnvironment environment, ILoggerFactory loggerFactory)

test/OpenApiTests/OpenApiTests.csproj

+2-2
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,13 @@
1515
</ItemGroup>
1616

1717
<ItemGroup>
18-
<EmbeddedResource Include="openapi.json" />
18+
<EmbeddedResource Include="swagger.json" />
1919
</ItemGroup>
2020

2121
<!-- Fixes IntelliSense errors on openapi.json in Visual Studio 2019, which uses the schema for OpenAPI 3.1 by default. -->
2222
<ProjectExtensions>
2323
<VisualStudio>
24-
<UserProperties integrationtests_4openapi_1json__JsonSchema="https://raw.githubusercontent.com/OAI/OpenAPI-Specification/main/schemas/v3.0/schema.json" />
24+
<UserProperties swagger_1json__JsonSchema="https://raw.githubusercontent.com/OAI/OpenAPI-Specification/main/schemas/v3.0/schema.json" />
2525
</VisualStudio>
2626
</ProjectExtensions>
2727
</Project>
File renamed without changes.

0 commit comments

Comments
 (0)