Removing JSON serialization files from coverage

Author: ProgrammingByPermutation

src/Nullinside.Api.Common.AspNetCore/Middleware/BasicAuthorizationRequirement.cs

@@ -1,10 +1,13 @@
+using System.Diagnostics.CodeAnalysis;
+
 using Microsoft.AspNetCore.Authorization;
 
 namespace Nullinside.Api.Common.AspNetCore.Middleware;
 
 /// <summary>
 ///   Represents a requirement where a user is expected to have one role.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class BasicAuthorizationRequirement : IAuthorizationRequirement {
   /// <summary>
   ///   Initializes a new instance of the <see cref="BasicAuthorizationRequirement" /> class.

src/Nullinside.Api.Common/Desktop/GithubLatestReleaseJson.cs

@@ -1,8 +1,11 @@
-namespace Nullinside.Api.Common.Desktop;
+using System.Diagnostics.CodeAnalysis;
+
+namespace Nullinside.Api.Common.Desktop;
 
 /// <summary>
 ///   The response information from GitHub's API.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class GithubLatestReleaseJson {
   /// <summary>
   ///   The url of the resource.

src/Nullinside.Api.Common/Docker/Support/DockerComposeLsOutput.cs

@@ -1,8 +1,11 @@
+using System.Diagnostics.CodeAnalysis;
+
 namespace Nullinside.Api.Common.Docker.Support;
 
 /// <summary>
 ///   The `docker compose ls --format 'json'` output.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class DockerComposeLsOutput {
   /// <summary>
   ///   The name of the docker compose project.

src/Nullinside.Api.Common/Docker/Support/DockerResource.cs

@@ -1,9 +1,12 @@
+using System.Diagnostics.CodeAnalysis;
+
 namespace Nullinside.Api.Common.Docker.Support;
 
 /// <summary>
 ///   A docker resource representing either a docker compose project
 ///   or a single docker container.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class DockerResource {
   /// <summary>
   ///   Initializes a new instance of the <see cref="DockerResource" /> class.

src/Nullinside.Api.Common/Exceptions/RetryException.cs

@@ -1,8 +1,11 @@
-namespace Nullinside.Api.Common.Exceptions;
+using System.Diagnostics.CodeAnalysis;
+
+namespace Nullinside.Api.Common.Exceptions;
 
 /// <summary>
 ///   An exception thrown if an action continues to fail after retrying.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class RetryException : Exception {
   /// <summary>
   ///   Initializes a new instance of the <see cref="RetryException" /> class.

src/Nullinside.Api.Common/Twitch/Json/TwitchModeratedChannelsResponse.cs

@@ -1,8 +1,11 @@
-namespace Nullinside.Api.Common.Twitch.Json;
+using System.Diagnostics.CodeAnalysis;
+
+namespace Nullinside.Api.Common.Twitch.Json;
 
 /// <summary>
 ///   The response to a query for what channels a user is moderator for.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class TwitchModeratedChannelsResponse {
   /// <summary>
   ///   The list of channels the user moderates for.
@@ -18,6 +21,7 @@ public class TwitchModeratedChannelsResponse {
 /// <summary>
 ///   A channel the user moderates.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class TwitchModeratedChannel {
   /// <summary>
   ///   The twitch id.
@@ -38,6 +42,7 @@ public class TwitchModeratedChannel {
 /// <summary>
 ///   Pagination information.
 /// </summary>
+[ExcludeFromCodeCoverage]
 public class Pagination {
   /// <summary>
   ///   The cursor to pass to "after" for pagination.

src/Nullinside.Api.Tests/Nullinside.Api/Controllers/DockerControllerTests.cs

@@ -106,7 +106,7 @@ public async Task OnOffComposeProjectsWork() {
     Assert.That(obj.StatusCode, Is.EqualTo(200));
 
     // Ensure we called the 3rd party API with a value of "Good" to turn on a compose.
-    bool deployments = (bool)obj.Value;
+    bool deployments = (bool)obj.Value!;
     Assert.That(deployments, Is.True);
   }
 
@@ -137,7 +137,7 @@ public async Task OnOffContainerWork() {
     Assert.That(obj.StatusCode, Is.EqualTo(200));
 
     // Ensure we called the 3rd party API with a value of "Good" to turn on a container.
-    bool deployments = (bool)obj.Value;
+    bool deployments = (bool)obj.Value!;
     Assert.That(deployments, Is.True);
   }
 

src/Nullinside.Api.Tests/Nullinside.Api/Controllers/UserControllerTests.cs

@@ -72,7 +72,7 @@ public async Task PerformGoogleLoginExisting() {
     Assert.That(this._db.Users.Count(), Is.EqualTo(1));
 
     // We should have saved the token in the existing user's database. 
-    Assert.That(obj.Url.EndsWith(this._db.Users.First().Token), Is.True);
+    Assert.That(obj.Url.EndsWith(this._db.Users.First().Token!), Is.True);
   }
 
   /// <summary>
@@ -92,15 +92,15 @@ public async Task PerformGoogleLoginNewUser() {
     Assert.That(this._db.Users.Count(), Is.EqualTo(1));
 
     // We should have saved the token in the existing user's database. 
-    Assert.That(obj.Url.EndsWith(this._db.Users.First().Token), Is.True);
+    Assert.That(obj.Url.EndsWith(this._db.Users.First().Token!), Is.True);
   }
 
   /// <summary>
   ///   Tests that we handle DB errors correctly in the google login.
   /// </summary>
   [Test]
   public async Task GoToErrorOnDbException() {
-    this._db.Users = null;
+    this._db.Users = null!;
 
     // Make the call and ensure it's successful.
     var controller = new TestableUserController(_configuration, _db);
@@ -157,7 +157,7 @@ public async Task PerformTwitchLoginExisting() {
     Assert.That(this._db.Users.Count(), Is.EqualTo(1));
 
     // We should have saved the token in the existing user's database. 
-    Assert.That(obj.Url.EndsWith(this._db.Users.First().Token), Is.True);
+    Assert.That(obj.Url.EndsWith(this._db.Users.First().Token!), Is.True);
   }
 
   /// <summary>
@@ -184,7 +184,7 @@ public async Task PerformTwitchLoginNewUser() {
     Assert.That(this._db.Users.Count(), Is.EqualTo(1));
 
     // We should have saved the token in the existing user's database. 
-    Assert.That(obj.Url.EndsWith(this._db.Users.First().Token), Is.True);
+    Assert.That(obj.Url.EndsWith(this._db.Users.First().Token!), Is.True);
   }
 
   /// <summary>
@@ -226,7 +226,7 @@ public async Task PerformTwitchLoginWithNoEmailAccount() {
   /// </summary>
   [Test]
   public async Task PerformTwitchLoginDbFailure() {
-    _db.Users = null;
+    _db.Users = null!;
 
     // Tells us twitch parsed the code successfully.
     _twitchApi.Setup(a => a.CreateAccessToken(It.IsAny<string>(), It.IsAny<CancellationToken>()))
@@ -265,9 +265,9 @@ public void GetRoles() {
     Assert.That(obj.StatusCode, Is.EqualTo(200));
 
     // Ensure we got the role we put in.
-    var roles = obj.Value.GetType().GetProperty("roles").GetValue(obj.Value) as IEnumerable<string>;
-    Assert.That(roles.Count(), Is.EqualTo(1));
-    Assert.That(roles.First(), Is.EqualTo("candy"));
+    var roles = obj.Value!.GetType().GetProperty("roles")!.GetValue(obj.Value) as IEnumerable<string>;
+    Assert.That(roles!.Count(), Is.EqualTo(1));
+    Assert.That(roles!.First(), Is.EqualTo("candy"));
   }
 
   /// <summary>
@@ -303,7 +303,7 @@ public async Task ValidateFailWithoutToken() {
   /// </summary>
   [Test]
   public async Task ValidateFailOnDbFailure() {
-    this._db.Users = null;
+    this._db.Users = null!;
 
     // Make the call and ensure it fails.
     var controller = new TestableUserController(_configuration, _db);
@@ -329,7 +329,7 @@ public TestableUserController(IConfiguration configuration, INullinsideContext d
   /// <inheritdoc />
   protected override Task<GoogleJsonWebSignature.Payload?> GenerateUserObject(GoogleOpenIdToken creds) {
     if (null != Email) {
-      return Task.FromResult(new GoogleJsonWebSignature.Payload() { Email = Email });
+      return Task.FromResult<GoogleJsonWebSignature.Payload?>(new GoogleJsonWebSignature.Payload() { Email = Email });
     }
 
     return Task.FromResult<GoogleJsonWebSignature.Payload?>(null);