Skip to content

Commit 3cb98c0

Browse files
Was85Was85
authored
Merge pull request #13 from Was85/copilot/cleanup-unused-code-and-tests
Clean up unused code, add source generator tests, enable CI test automation
2 parents 10248a9 + 99e1e1f commit 3cb98c0

File tree

11 files changed

+584
-68
lines changed

11 files changed

+584
-68
lines changed

‎.github/workflows/nuget-ci-cd.yml‎

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ jobs:
6666
- name: Setup .NET
6767
uses: actions/setup-dotnet@v3
6868
with:
69-
dotnet-version: 9.0.x
69+
dotnet-version: 10.0.x
7070

7171
- name: Build Solution
7272
run: |
@@ -75,6 +75,21 @@ jobs:
7575
-p:VersionPrefix=${{ needs.calculate_version.outputs.version_prefix }} \
7676
-p:VersionSuffix=${{ needs.calculate_version.outputs.version_suffix }}
7777
78+
- name: Run Tests
79+
run: |
80+
dotnet test ${{ env.SOLUTION_PATH }} \
81+
--configuration Release \
82+
--verbosity normal \
83+
--logger "trx;LogFileName=test-results.trx"
84+
85+
- name: Upload Test Results
86+
uses: actions/upload-artifact@v4
87+
if: always()
88+
with:
89+
name: test-results
90+
path: '**/test-results.trx'
91+
retention-days: 7
92+
7893
- name: Create NuGet Package
7994
run: |
8095
dotnet pack ${{ env.SOLUTION_PATH }} \

‎OpenApiExampleApp.sln‎

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,26 +23,61 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "OpenApiExampleApp", "src\Op
2323
EndProject
2424
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "DemoApi", "exampl\DemoApi\DemoApi.csproj", "{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}"
2525
EndProject
26+
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "OpenApiExampleApp.Tests", "tests\OpenApiExampleApp.Tests\OpenApiExampleApp.Tests.csproj", "{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}"
27+
EndProject
2628
Global
2729
GlobalSection(SolutionConfigurationPlatforms) = preSolution
2830
Debug|Any CPU = Debug|Any CPU
31+
Debug|x64 = Debug|x64
32+
Debug|x86 = Debug|x86
2933
Release|Any CPU = Release|Any CPU
34+
Release|x64 = Release|x64
35+
Release|x86 = Release|x86
3036
EndGlobalSection
3137
GlobalSection(ProjectConfigurationPlatforms) = postSolution
3238
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
3339
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Debug|Any CPU.Build.0 = Debug|Any CPU
40+
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Debug|x64.ActiveCfg = Debug|Any CPU
41+
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Debug|x64.Build.0 = Debug|Any CPU
42+
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Debug|x86.ActiveCfg = Debug|Any CPU
43+
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Debug|x86.Build.0 = Debug|Any CPU
3444
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Release|Any CPU.ActiveCfg = Release|Any CPU
3545
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Release|Any CPU.Build.0 = Release|Any CPU
46+
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Release|x64.ActiveCfg = Release|Any CPU
47+
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Release|x64.Build.0 = Release|Any CPU
48+
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Release|x86.ActiveCfg = Release|Any CPU
49+
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248}.Release|x86.Build.0 = Release|Any CPU
3650
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
3751
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Debug|Any CPU.Build.0 = Debug|Any CPU
52+
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Debug|x64.ActiveCfg = Debug|Any CPU
53+
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Debug|x64.Build.0 = Debug|Any CPU
54+
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Debug|x86.ActiveCfg = Debug|Any CPU
55+
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Debug|x86.Build.0 = Debug|Any CPU
3856
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Release|Any CPU.ActiveCfg = Release|Any CPU
3957
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Release|Any CPU.Build.0 = Release|Any CPU
58+
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Release|x64.ActiveCfg = Release|Any CPU
59+
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Release|x64.Build.0 = Release|Any CPU
60+
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Release|x86.ActiveCfg = Release|Any CPU
61+
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB}.Release|x86.Build.0 = Release|Any CPU
62+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
63+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Debug|Any CPU.Build.0 = Debug|Any CPU
64+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Debug|x64.ActiveCfg = Debug|Any CPU
65+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Debug|x64.Build.0 = Debug|Any CPU
66+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Debug|x86.ActiveCfg = Debug|Any CPU
67+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Debug|x86.Build.0 = Debug|Any CPU
68+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Release|Any CPU.ActiveCfg = Release|Any CPU
69+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Release|Any CPU.Build.0 = Release|Any CPU
70+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Release|x64.ActiveCfg = Release|Any CPU
71+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Release|x64.Build.0 = Release|Any CPU
72+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Release|x86.ActiveCfg = Release|Any CPU
73+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299}.Release|x86.Build.0 = Release|Any CPU
4074
EndGlobalSection
4175
GlobalSection(SolutionProperties) = preSolution
4276
HideSolutionNode = FALSE
4377
EndGlobalSection
4478
GlobalSection(NestedProjects) = preSolution
4579
{CD732490-6E82-4FF7-BAA5-4E5B74CB7248} = {CB929E4B-CFC5-4DEF-B2DF-ECEA46339A08}
4680
{9533ED82-D44A-4EE0-A30F-D70DEB55D1FB} = {221B5EB1-6937-4CFF-B6B8-F59A4A380823}
81+
{B6766971-ED3F-48C2-BFF3-E8DDFA6E0299} = {145F141C-5A21-498E-BFA9-55522D610A93}
4782
EndGlobalSection
4883
EndGlobal

‎exampl/DemoApi/Controllers/EdgeCaseController.cs‎

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -9,22 +9,6 @@ namespace DemoApi.Controllers;
99
[Route("api/edge")]
1010
public class EdgeCaseController : ControllerBase
1111
{
12-
// Edge Case: Missing Example Property
13-
[HttpPost("missing-example")]
14-
//[RequestExample(typeof(MissingExampleProvider))]
15-
//public IActionResult AddMissingExample(WeatherForecastDto weatherForecastDto)
16-
//{
17-
// return Created("Get", new { });
18-
//}
19-
20-
//// Edge Case: Non-Static Example Property
21-
//[HttpPost("non-static-example")]
22-
//[RequestExample(typeof(NonStaticExampleProvider))]
23-
//public IActionResult AddNonStaticExample(NonStaticExampleDto nonStaticExampleDto)
24-
//{
25-
// return Created("Get", new { });
26-
//}
27-
2812
// Duplicate Route Example
2913
[HttpPost("duplicate")]
3014
[RequestExample(typeof(BasicExamples), name: "DuplicateExample1")]

‎exampl/DemoApi/Dtos/NonStaticExampleDto.cs‎

Lines changed: 0 additions & 7 deletions
This file was deleted.

‎exampl/DemoApi/Dtos/Temperature.cs‎

Lines changed: 0 additions & 7 deletions
This file was deleted.

‎exampl/DemoApi/RequestExamples/MissingExampleProvider.cs‎

Lines changed: 0 additions & 6 deletions
This file was deleted.

‎exampl/DemoApi/RequestExamples/NonStaticExampleProvider.cs‎

Lines changed: 0 additions & 14 deletions
This file was deleted.

‎exampl/DemoApi/RequestExamples/TemperatureExamples.cs‎

Lines changed: 0 additions & 17 deletions
This file was deleted.

‎tests/OpenApiExampleApp.Tests/OpenApiExampleApp.Tests.csproj‎

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
<Project Sdk="Microsoft.NET.Sdk">
2+
3+
<PropertyGroup>
4+
<TargetFramework>net10.0</TargetFramework>
5+
<ImplicitUsings>enable</ImplicitUsings>
6+
<Nullable>enable</Nullable>
7+
<IsPackable>false</IsPackable>
8+
</PropertyGroup>
9+
10+
<ItemGroup>
11+
<FrameworkReference Include="Microsoft.AspNetCore.App" />
12+
</ItemGroup>
13+
14+
<ItemGroup>
15+
<PackageReference Include="coverlet.collector" Version="6.0.4" />
16+
<PackageReference Include="Microsoft.CodeAnalysis.CSharp" Version="4.14.0" />
17+
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.14.1" />
18+
<PackageReference Include="Microsoft.OpenApi" Version="2.3.9" />
19+
<PackageReference Include="xunit" Version="2.9.3" />
20+
<PackageReference Include="xunit.runner.visualstudio" Version="3.1.4" />
21+
</ItemGroup>
22+
23+
<ItemGroup>
24+
<Using Include="Xunit" />
25+
</ItemGroup>
26+
27+
<ItemGroup>
28+
<ProjectReference Include="..\..\src\OpenApiExampleApp\OpenApiExampleApp.csproj" />
29+
</ItemGroup>
30+
31+
</Project>

0 commit comments

Comments
 (0)