Adds test coverage for non generated features

.github/workflows/build.yml

@@ -28,4 +28,24 @@ jobs:
         run: dotnet build --no-incremental /p:SignAssembly=true /p:AssemblyOriginatorKeyFile=../key.snk
 
       - name: Test SDK
-        run: dotnet test
+        run: dotnet test --configuration Release --verbosity normal --collect:"XPlat Code Coverage" --results-directory ./coverage -p:ExcludeByFile="**/GitHub/**/*.cs" 
+
+      - name: Code Coverage Report
+        uses: irongut/CodeCoverageSummary@v1.3.0
+        with:
+          filename: coverage/**/coverage.cobertura.xml
+          badge: true
+          fail_below_min: false
+          format: markdown
+          hide_branch_rate: false
+          hide_complexity: true
+          indicators: true
+          output: both
+          thresholds: '60 80'
+
+      - name: Add Coverage PR Comment
+        uses: marocchino/sticky-pull-request-comment@v2
+        if: github.event_name == 'pull_request'
+        with:
+          recreate: true
+          path: code-coverage-results.md

.gitignore

@@ -104,9 +104,8 @@ node_modules/
 # Local History for Visual Studio Code
 .history/
 
-# Windows Installer files from build outputs
-*.cab
-*.msi
-*.msix
-*.msm
-*.msp
+# Tests / Coverage
+testresults/
+[Tt]est[Rr]esult*/
+**/coverage.json
+coverage/*

GitHub.Octokit.sln

@@ -13,7 +13,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Solution Items", "Solution
 		README.md = README.md
 	EndProjectSection
 EndProject
-Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Tests", "Tests\Tests.csproj", "{7EF0200D-9F10-4A77-8F73-3E26D3EBD326}"
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Tests", "test\Tests.csproj", "{7EF0200D-9F10-4A77-8F73-3E26D3EBD326}"
 EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution

README.md

@@ -59,6 +59,12 @@ Currently this project is fairly simple (we hope it can stay that way).  All of
  - **Middleware** - this represents object and handlers that can mutate the request and are "injected" into the request/response flow.
  - **Octokit** - types which represent request/response objects
 
+## Testing
+
+- Run tests: `dotnet test`
+- Run coverage: `dotnet test /p:CollectCoverage=true`
+
+
 ## More details on this SDK and repo
 
 - [Code of conduct](Docs/CODE_OF_CONDUCT.md)

src/Authentication/TokenAuthenticationProvider.cs

@@ -36,8 +36,8 @@ public class TokenAuthenticationProvider : IAuthenticationProvider
     /// <exception cref="ArgumentNullException"></exception>
     public TokenAuthenticationProvider(string clientId, string token)
     {
-        ArgumentNullException.ThrowIfNullOrEmpty(clientId);
-        ArgumentNullException.ThrowIfNullOrEmpty(token);
+        ArgumentException.ThrowIfNullOrEmpty(clientId);
+        ArgumentException.ThrowIfNullOrEmpty(token);
 
         ClientId = clientId;
         Token = token;
@@ -52,10 +52,10 @@ public TokenAuthenticationProvider(string clientId, string token)
     /// <param name="cancellationToken"></param>
     /// <returns></returns>
     /// <exception cref="ArgumentNullException"></exception>
-    public Task AuthenticateRequestAsync(RequestInformation request, Dictionary<string, object>? additionalAuthenticationContext = null, CancellationToken cancellationToken = default)
+    public Task AuthenticateRequestAsync(RequestInformation? request, Dictionary<string, object>? additionalAuthenticationContext = null, CancellationToken cancellationToken = default)
     {
         ArgumentNullException.ThrowIfNull(request);
-        ArgumentNullException.ThrowIfNullOrEmpty(Token);
+        ArgumentException.ThrowIfNullOrEmpty(Token);
 
         if (!request.Headers.ContainsKey(AuthorizationHeaderKey))
         {

src/Middleware/APIVersionHandler.cs

@@ -8,7 +8,7 @@ namespace GitHub.Octokit.Client.Middleware;
 /// <summary>
 /// Represents a handler that adds the API version header to outgoing HTTP requests.
 /// </summary>
-class APIVersionHandler(APIVersionOptions? apiVersionOptions = null) : DelegatingHandler
+public class APIVersionHandler(APIVersionOptions? apiVersionOptions = null) : DelegatingHandler
 {
     private const string ApiVersionHeaderKey = "X-GitHub-Api-Version";
 

src/Middleware/Options/UserAgentOptions.cs

@@ -13,7 +13,7 @@ public class UserAgentOptions : IRequestOption
     /// Gets or sets the product name used in the user agent request header.
     /// Defaults to <c>"dotnet-sdk"</c>.
     /// </summary>
-    public string ProductName { get; set; } = "dotnet-sdk";
+    public string ProductName { get; set; } = "GitHub.Octokit.dotnet-sdk";
 
     /// <summary>
     /// Gets or sets the product version used in the user agent request header.

test/Authentication/TokenAuthenticationProviderTest.cs

@@ -0,0 +1,44 @@
+using GitHub.Octokit.Authentication;
+using Microsoft.Kiota.Abstractions;
+using Xunit;
+
+public class TokenAuthenticationProviderTests
+{
+    private const string ValidClientId = "validClientId";
+    private const string ValidToken = "validToken";
+    private TokenAuthenticationProvider _provider;
+
+    public TokenAuthenticationProviderTests()
+    {
+        _provider = new TokenAuthenticationProvider(ValidClientId, ValidToken);
+    }
+
+    [Fact]
+    public void Constructor_ThrowsException_WhenClientIdIsEmpty()
+    {
+        Assert.Throws<ArgumentException>(() => new TokenAuthenticationProvider("", ValidToken));
+    }
+
+    [Fact]
+    public void Constructor_ThrowsException_WhenTokenIsEmpty()
+    {
+        Assert.Throws<ArgumentException>(() => new TokenAuthenticationProvider(ValidClientId, ""));
+    }
+
+    [Fact]
+    public async Task AuthenticateRequestAsync_ThrowsException_WhenRequestIsNull()
+    {
+        await Assert.ThrowsAsync<ArgumentNullException>(() => _provider.AuthenticateRequestAsync(null, null));
+    }
+
+    [Fact]
+    public async Task AuthenticateRequestAsync_AddsAuthorizationHeader_WhenRequestIsValid()
+    {
+        var request = new RequestInformation();
+        await _provider.AuthenticateRequestAsync(request);
+        var headerToken = request.Headers["Authorization"].FirstOrDefault<string>();
+
+        Assert.True(request.Headers.ContainsKey("Authorization"));
+        Assert.Equal($"Bearer {ValidToken}", headerToken);
+    }
+}

test/Client/ClientFactoryTest.cs

@@ -0,0 +1,68 @@
+using GitHub.Octokit.Client;
+using GitHub.Octokit.Client.Middleware;
+using Xunit;
+
+
+public class TestHandler1 : DelegatingHandler { }
+public class TestHandler2 : DelegatingHandler { }
+
+public class ClientFactoryTests
+{
+
+    [Fact]
+    public void Creates_Client_With_Default_Timeout()
+    {
+        var clientFactory = ClientFactory.Create();
+        Assert.Equal(TimeSpan.FromSeconds(100), clientFactory.Timeout);
+    }
+
+    [Fact]
+    public void Creates_Client_Persists_Set_Timeout()
+    {
+        var clientFactory = ClientFactory.Create();
+        clientFactory.Timeout = TimeSpan.FromSeconds(5);
+        Assert.Equal(TimeSpan.FromSeconds(5), clientFactory.Timeout);
+    }
+
+    [Fact]
+    public void Create_Returns_NonNullHttpClient()
+    {
+        HttpMessageHandler handler = new HttpClientHandler();
+        var client = ClientFactory.Create(handler);
+        Assert.NotNull(client);
+    }
+
+    [Fact]
+    public void CreateDefaultHandlers_Returns_Expected_Handlers()
+    {
+        var handlers = ClientFactory.CreateDefaultHandlers();
+        Assert.Contains(handlers, h => h is APIVersionHandler);
+        Assert.Contains(handlers, h => h is UserAgentHandler);
+    }
+
+    // ChainHandlersCollectionAndGetFirstLink
+    [Fact]
+    public void ChainHandlersCollectionAndGetFirstLink_ChainsHandlersCorrectly()
+    {
+        var handlers = new DelegatingHandler[]
+        {
+            new TestHandler1(),
+            new TestHandler2()
+        };
+        var firstHandler = ClientFactory.ChainHandlersCollectionAndGetFirstLink(null, handlers);
+        Assert.IsType<TestHandler1>(firstHandler);
+        Assert.IsType<TestHandler2>(firstHandler.InnerHandler);
+    }
+
+
+    // GetDefaultHttpMessageHandler
+    [Fact]
+    public void GetDefaultHttpMessageHandler_Returns_NonNullHttpMessageHandler()
+    {
+        var handler = ClientFactory.GetDefaultHttpMessageHandler();
+        Assert.NotNull(handler);
+    }
+
+}
+
+

Tests/Client/RequestAdaptorTest.cs → test/Client/RequestAdaptorTest.cs

@@ -1,6 +1,5 @@
 using GitHub.Octokit.Authentication;
 using GitHub.Octokit.Client;
-using NSubstitute;
 using Xunit;
 
 public class RequestAdapterTests
@@ -16,7 +15,7 @@ public void Creates_RequestAdaptor_With_Defaults()
     [Fact]
     public void Creates_RequestAdaptor_With_GenericHttpClient()
     {
-        var httpClient = Substitute.For<HttpClient>();
+        var httpClient = new HttpClient();
         var requestAdapter = RequestAdapter.Create(new TokenAuthenticationProvider("Octokit.Gen", "JRRTOLKIEN"), httpClient);
         Assert.NotNull(requestAdapter);
     }

Tests/Tests.csproj → test/Tests.csproj

@@ -28,22 +28,28 @@
     </Content>
   </ItemGroup>
 
-
-
   <ItemGroup>
+    <PackageReference Include="coverlet.collector" Version="6.0.1">
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+      <PrivateAssets>all</PrivateAssets>
+    </PackageReference>
+    <PackageReference Include="coverlet.msbuild" Version="6.0.1">
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+      <PrivateAssets>all</PrivateAssets>
+    </PackageReference>
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
     <PackageReference Include="xunit" Version="2.6.2" />
     <PackageReference Include="xunit.runner.visualstudio" Version="2.5.4" PrivateAssets="all" />
-    <PackageReference Include="NSubstitute" Version="5.1.0" />
+    <PackageReference Include="Moq" Version="4.20.70" />
   </ItemGroup>
 
   <ItemGroup>
     <PackageReference Include="System.Net.Http" Version="4.3.4" />
     <PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" Version="1.0.3" PrivateAssets="all" />
-
   </ItemGroup>
 
   <ItemGroup>
     <ProjectReference Include="..\src\GitHub.Octokit.SDK.csproj" />
   </ItemGroup>
 </Project>
+