feat: use C# 9 records where possible

Directory.Build.props

@@ -1,6 +1,6 @@
 <Project>
     <PropertyGroup>
-        <LangVersion>8.0</LangVersion>
+        <LangVersion>9.0</LangVersion>
         <Nullable>Enable</Nullable>
         <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
     </PropertyGroup>

src/Covid19Api.Constants/Urls.cs

@@ -4,7 +4,9 @@ public static class Urls
     {
         // ReSharper disable once S1075
 #pragma warning disable S1075
-        public const string CovidInfoWorldOmetersUrl = "https://worldometers.info/coronavirus";
+        public const string Covid19WorldometerUrl = "https://worldometers.info/coronavirus";
+
+        public const string RestCountriesApiUrl = "https://restcountries.eu/rest/v2/all";
 #pragma warning restore
     }
 }

src/Covid19Api.IoC/Modules/Covid19ApiDbContextModule.cs

@@ -29,7 +29,7 @@ protected override void Load(ContainerBuilder builder)
                 .AsSelf()
                 .InstancePerLifetimeScope();
 
-            builder.Register<Func<IMongoDatabase>>(componentContext =>
+            builder.Register<Func<IMongoDatabase>>(_ =>
                 {
                     return () =>
                     {

src/Covid19Api.Services.Abstractions/Models/CountryMetaData.cs

@@ -1,18 +1,4 @@
 namespace Covid19Api.Services.Abstractions.Models
 {
-    public class CountryMetaData
-    {
-        public CountryMetaData(string name, string alpha2Code, string[] altSpellings)
-        {
-            this.Name = name;
-            this.Alpha2Code = alpha2Code;
-            this.AltSpellings = altSpellings;
-        }
-
-        public string Name { get; }
-
-        public string Alpha2Code { get; }
-
-        public string[] AltSpellings { get; }
-    }
+    public sealed record CountryMetaData(string Name, string Alpha2Code, string[] AltSpellings);
 }

src/Covid19Api.Services/Loader/CountryMetaDataLoader.cs

@@ -2,6 +2,7 @@
 using System.Net.Http;
 using System.Text.Json;
 using System.Threading.Tasks;
+using Covid19Api.Constants;
 using Covid19Api.Services.Abstractions.Loader;
 using Covid19Api.Services.Abstractions.Models;
 using Microsoft.Extensions.Logging;
@@ -10,10 +11,6 @@ namespace Covid19Api.Services.Loader
 {
     public class CountryMetaDataLoader : ICountryMetaDataLoader
     {
-#pragma warning disable S1075
-        private const string ApiUrl = "https://restcountries.eu/rest/v2/all";
-#pragma warning restore
-
         private static readonly JsonSerializerOptions SerializerOptions = new JsonSerializerOptions
         {
             PropertyNamingPolicy = JsonNamingPolicy.CamelCase
@@ -32,7 +29,7 @@ public async Task<CountryMetaData[]> LoadCountryMetaDataAsync()
         {
             var client = this.httpClientFactory.CreateClient();
 
-            var response = await client.GetAsync(ApiUrl);
+            var response = await client.GetAsync(Urls.RestCountriesApiUrl);
 
             if (response.IsSuccessStatusCode)
             {
@@ -44,7 +41,7 @@ public async Task<CountryMetaData[]> LoadCountryMetaDataAsync()
 
             var error = await response.Content.ReadAsStringAsync();
             this.logger.LogError("Failed load country meta-data from {url}! Status-Code: {statusCode} Error:\n{error}",
-                ApiUrl, response.StatusCode, error);
+                Urls.RestCountriesApiUrl, response.StatusCode, error);
 
             return Array.Empty<CountryMetaData>();
         }

src/Covid19Api.Services/Loader/HtmlDocumentLoader.cs

@@ -19,7 +19,7 @@ public async Task<HtmlDocument> LoadAsync()
         {
             var client = this.httpClientFactory.CreateClient();
 
-            var response = await client.GetAsync(Urls.CovidInfoWorldOmetersUrl);
+            var response = await client.GetAsync(Urls.Covid19WorldometerUrl);
 
             var document = new HtmlDocument();
 

src/Covid19Api.UseCases.Abstractions/Commands/AggregateCountryStatisticsCommand.cs

@@ -1,21 +1,7 @@
-using System;
+using System.Collections.Generic;
 using MediatR;
 
 namespace Covid19Api.UseCases.Abstractions.Commands
 {
-    public class AggregateCountryStatisticsCommand : IRequest
-    {
-        public AggregateCountryStatisticsCommand(string[] countries, int month, int year)
-        {
-            this.Countries = countries ?? throw new ArgumentNullException(nameof(countries));
-            this.Month = month;
-            this.Year = year;
-        }
-
-        public string[] Countries { get; }
-
-        public int Month { get; }
-
-        public int Year { get; }
-    }
+    public sealed record AggregateCountryStatisticsCommand(IEnumerable<string> Countries, int Month, int Year) : IRequest;
 }

src/Covid19Api.UseCases.Abstractions/Commands/AggregateGlobalStatisticsCommand.cs

@@ -2,16 +2,5 @@
 
 namespace Covid19Api.UseCases.Abstractions.Commands
 {
-    public class AggregateGlobalStatisticsCommand : IRequest
-    {
-        public AggregateGlobalStatisticsCommand(int month, int year)
-        {
-            this.Month = month;
-            this.Year = year;
-        }
-
-        public int Month { get; }
-
-        public int Year { get; }
-    }
+    public sealed record AggregateGlobalStatisticsCommand(int Month, int Year) : IRequest;
 }

src/Covid19Api.UseCases.Abstractions/Commands/RefreshCountriesStatisticsCommand.cs

@@ -3,13 +3,5 @@
 
 namespace Covid19Api.UseCases.Abstractions.Commands
 {
-    public class RefreshCountriesStatisticsCommand : IRequest
-    {
-        public RefreshCountriesStatisticsCommand(DateTime fetchedAt)
-        {
-            this.FetchedAt = fetchedAt;
-        }
-
-        public DateTime FetchedAt { get; }
-    }
+    public sealed record RefreshCountriesStatisticsCommand(DateTime FetchedAt) : IRequest;
 }

src/Covid19Api.UseCases.Abstractions/Commands/RefreshGlobalStatisticsCommand.cs

@@ -3,13 +3,5 @@
 
 namespace Covid19Api.UseCases.Abstractions.Commands
 {
-    public class RefreshGlobalStatisticsCommand : IRequest
-    {
-        public RefreshGlobalStatisticsCommand(DateTime fetchedAt)
-        {
-            this.FetchedAt = fetchedAt;
-        }
-
-        public DateTime FetchedAt { get; }
-    }
+    public sealed record RefreshGlobalStatisticsCommand(DateTime FetchedAt) : IRequest;
 }

src/Covid19Api.UseCases.Abstractions/Queries/CountryStatistics/LoadHistoricalCountriesStatisticsQuery.cs

@@ -7,15 +7,8 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.CountryStatistics
 {
-    public class LoadHistoricalCountriesStatisticsQuery : ICacheableRequest, IRequest<IEnumerable<CountryStatisticDto>>
+    public sealed record LoadHistoricalCountriesStatisticsQuery(DateTime MinFetchedAt) : ICacheableRequest, IRequest<IEnumerable<CountryStatisticDto>>
     {
-        public LoadHistoricalCountriesStatisticsQuery(DateTime minFetchedAt)
-        {
-            this.MinFetchedAt = minFetchedAt;
-        }
-
-        public DateTime MinFetchedAt { get; }
-
         public CacheConfiguration GetCacheConfiguration()
             => new CacheConfiguration(nameof(LoadHistoricalCountriesStatisticsQuery), TimeSpan.FromMinutes(30));
     }

src/Covid19Api.UseCases.Abstractions/Queries/CountryStatistics/LoadHistoricalCountryStatisticsForCountryQuery.cs

@@ -7,16 +7,9 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.CountryStatistics
 {
-    public class LoadHistoricalCountryStatisticsForCountryQuery : ICacheableRequest,
+    public sealed record LoadHistoricalCountryStatisticsForCountryQuery(string Country) : ICacheableRequest,
         IRequest<IEnumerable<CountryStatisticDto>>
     {
-        public LoadHistoricalCountryStatisticsForCountryQuery(string country)
-        {
-            this.Country = country;
-        }
-
-        public string Country { get; }
-
         public CacheConfiguration GetCacheConfiguration() =>
             new CacheConfiguration($"{nameof(LoadHistoricalCountryStatisticsForCountryQuery)}_{this.Country}",
                 TimeSpan.FromMinutes(30));

src/Covid19Api.UseCases.Abstractions/Queries/CountryStatistics/LoadLatestCountriesStatisticsQuery.cs

@@ -7,7 +7,7 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.CountryStatistics
 {
-    public class LoadLatestCountriesStatisticsQuery : ICacheableRequest, IRequest<IEnumerable<CountryStatisticDto>>
+    public sealed record LoadLatestCountriesStatisticsQuery : ICacheableRequest, IRequest<IEnumerable<CountryStatisticDto>>
     {
         public CacheConfiguration GetCacheConfiguration() =>
             new CacheConfiguration(nameof(LoadLatestCountriesStatisticsQuery), TimeSpan.FromMinutes(30));

src/Covid19Api.UseCases.Abstractions/Queries/CountryStatistics/LoadLatestStatisticsForCountryQuery.cs

@@ -6,15 +6,8 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.CountryStatistics
 {
-    public class LoadLatestStatisticsForCountryQuery : ICacheableRequest, IRequest<CountryStatisticDto>
+    public sealed record LoadLatestStatisticsForCountryQuery(string Country) : ICacheableRequest, IRequest<CountryStatisticDto>
     {
-        public LoadLatestStatisticsForCountryQuery(string country)
-        {
-            this.Country = country;
-        }
-
-        public string Country { get; }
-
         public CacheConfiguration GetCacheConfiguration() =>
             new CacheConfiguration($"{nameof(LoadLatestStatisticsForCountryQuery)}_{this.Country}",
                 TimeSpan.FromMinutes(30));

src/Covid19Api.UseCases.Abstractions/Queries/CountryStatisticsAggregates/LoadCountryStatisticsAggregate.cs

@@ -3,18 +3,6 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.CountryStatisticsAggregates
 {
-    public class LoadCountryStatisticsAggregate : IRequest<CountryStatisticAggregateDto?>
-    {
-        public LoadCountryStatisticsAggregate(string country, int month, int year)
-        {
-            this.Country = country;
-            this.Month = month;
-            this.Year = year;
-        }
-
-        public string Country { get; }
-        public int Month { get; }
-
-        public int Year { get; }
-    }
+    public sealed record LoadCountryStatisticsAggregate
+        (string Country, int Month, int Year) : IRequest<CountryStatisticAggregateDto?>;
 }

src/Covid19Api.UseCases.Abstractions/Queries/CountryStatisticsAggregates/LoadCountryStatisticsAggregatesForCountryInYearQuery.cs

@@ -1,22 +1,9 @@
-using System;
 using System.Collections.Generic;
 using Covid19Api.Presentation.Response;
 using MediatR;
 
 namespace Covid19Api.UseCases.Abstractions.Queries.CountryStatisticsAggregates
 {
-    public class
-        LoadCountryStatisticsAggregatesForCountryInYearQuery : IRequest<IEnumerable<CountryStatisticAggregateDto>>
-    {
-        public LoadCountryStatisticsAggregatesForCountryInYearQuery(string country, int year)
-        {
-            this.Country = country;
-            this.Year = year;
-
-            if (string.IsNullOrWhiteSpace(this.Country)) throw new ArgumentNullException(nameof(country));
-        }
-
-        public string Country { get; }
-        public int Year { get; }
-    }
+    public sealed record LoadCountryStatisticsAggregatesForCountryInYearQuery
+        (string Country, int Year) : IRequest<IEnumerable<CountryStatisticAggregateDto>>;
 }

src/Covid19Api.UseCases.Abstractions/Queries/GlobalStatistics/LoadHistoricalGlobalStatisticsQuery.cs

@@ -7,15 +7,8 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.GlobalStatistics
 {
-    public class LoadHistoricalGlobalStatisticsQuery : ICacheableRequest, IRequest<IEnumerable<GlobalStatisticDto>>
+    public sealed record LoadHistoricalGlobalStatisticsQuery(DateTime MinFetchedAt) : ICacheableRequest, IRequest<IEnumerable<GlobalStatisticDto>>
     {
-        public LoadHistoricalGlobalStatisticsQuery(DateTime minFetchedAt)
-        {
-            this.MinFetchedAt = minFetchedAt;
-        }
-
-        public DateTime MinFetchedAt { get; }
-
         public CacheConfiguration GetCacheConfiguration() =>
             new CacheConfiguration(nameof(LoadHistoricalGlobalStatisticsQuery), TimeSpan.FromMinutes(30));
     }

src/Covid19Api.UseCases.Abstractions/Queries/GlobalStatistics/LoadLatestGlobalStatisticsQuery.cs

@@ -6,7 +6,7 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.GlobalStatistics
 {
-    public class LoadLatestGlobalStatisticsQuery : ICacheableRequest, IRequest<GlobalStatisticDto>
+    public sealed record LoadLatestGlobalStatisticsQuery : ICacheableRequest, IRequest<GlobalStatisticDto>
     {
         public CacheConfiguration GetCacheConfiguration() =>
             new CacheConfiguration(nameof(LoadLatestGlobalStatisticsQuery), TimeSpan.FromMinutes(30));

src/Covid19Api.UseCases.Abstractions/Queries/GlobalStatisticsAggregates/LoadGlobalStatisticsAggregate.cs

@@ -3,16 +3,5 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.GlobalStatisticsAggregates
 {
-    public class LoadGlobalStatisticsAggregate : IRequest<GlobalStatisticAggregateDto?>
-    {
-        public LoadGlobalStatisticsAggregate(int month, int year)
-        {
-            this.Month = month;
-            this.Year = year;
-        }
-
-        public int Month { get; }
-
-        public int Year { get; }
-    }
+    public sealed record LoadGlobalStatisticsAggregate(int Month, int Year) : IRequest<GlobalStatisticAggregateDto?>;
 }

src/Covid19Api.UseCases.Abstractions/Queries/GlobalStatisticsAggregates/LoadGlobalStatisticsAggregatesForYearQuery.cs

@@ -4,13 +4,6 @@
 
 namespace Covid19Api.UseCases.Abstractions.Queries.GlobalStatisticsAggregates
 {
-    public class LoadGlobalStatisticsAggregatesForYearQuery : IRequest<IEnumerable<GlobalStatisticAggregateDto>>
-    {
-        public LoadGlobalStatisticsAggregatesForYearQuery(int year)
-        {
-            this.Year = year;
-        }
-
-        public int Year { get; }
-    }
+    public sealed record LoadGlobalStatisticsAggregatesForYearQuery
+        (int Year) : IRequest<IEnumerable<GlobalStatisticAggregateDto>>;
 }