Update codebase

Author: nemanjarogic

.editorconfig

@@ -66,7 +66,7 @@ public IDisposable Subscribe(ICustomObserver<IEvent> observer) =>
     public IDisposable Subscribe<T>(ICustomObserver<T> newObserver)
         where T : IEvent
     {
-        if (!_typedObservers.TryGetValue(typeof(T), out List<IObserver<IEvent>> observers))
+        if (!_typedObservers.TryGetValue(typeof(T), out var observers))
         {
             observers = new List<IObserver<IEvent>>();
             _typedObservers[typeof(T)] = observers;

src/AdditionalPatterns/EventAggregator/StoreManagement/Components/EventAggregator.cs

@@ -5,8 +5,8 @@ namespace OrderManagement.Infrastructure;
 
 public class OrderManagementContext : DbContext
 {
-    public DbSet<Customer> Customers { get; set; }
-    public DbSet<Order> Orders { get; set; }
+    public DbSet<Customer> Customers => Set<Customer>();
+    public DbSet<Order> Orders => Set<Order>();
 
     protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder) =>
         optionsBuilder.UseSqlite("Data Source=storage.db");

src/AdditionalPatterns/Repository/OrderManagement/Infrastructure/OrderManagementContext.cs

@@ -2,8 +2,8 @@
 
 public interface IServiceLocator
 {
-    void AddService<T>(T service);
-    void AddService<T>(string serviceName, T service);
-    T GetService<T>();
-    object GetService<T>(string serviceName);
+    void AddService<T>(T service) where T : notnull;
+    void AddService<T>(string serviceName, T service) where T : notnull;
+    T GetService<T>() where T : notnull;
+    object GetService<T>(string serviceName) where T : notnull;
 }

src/AdditionalPatterns/ServiceLocator/OrderProcessing/IServiceLocator.cs

@@ -25,16 +25,16 @@ public ServiceLocator()
         };
     }
 
-    public void AddService<T>(T service) =>
+    public void AddService<T>(T service) where T : notnull =>
         _services.Add(typeof(T).Name, service);
 
-    public void AddService<T>(string serviceName, T service) =>
+    public void AddService<T>(string serviceName, T service) where T : notnull =>
         _services.Add(serviceName, service);
 
-    public T GetService<T>() =>
+    public T GetService<T>() where T : notnull =>
         (T)_services[typeof(T).Name];
 
-    public object GetService<T>(string serviceName) =>
+    public object GetService<T>(string serviceName) where T : notnull =>
         (T)_services[serviceName];
 
     /*public void AddService(string serviceName)

src/AdditionalPatterns/ServiceLocator/OrderProcessing/ServiceLocator.cs

@@ -26,7 +26,7 @@ protected Entity()
 
     public static bool operator !=(Entity a, Entity b) => !(a == b);
 
-    public override bool Equals(object obj)
+    public override bool Equals(object? obj)
     {
         if (obj is not Entity other)
         {

src/AdditionalPatterns/Specification/ProductSpecification/Domain/Entity.cs

@@ -5,8 +5,8 @@ namespace UnitOfWorkLibrary.Example1.Infrastructure;
 
 public class OrderManagementContext1 : DbContext
 {
-    public DbSet<Customer> Customers { get; set; }
-    public DbSet<Order> Orders { get; set; }
+    public DbSet<Customer> Customers => Set<Customer>();
+    public DbSet<Order> Orders => Set<Order>();
 
     protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder) =>
         optionsBuilder.UseSqlite("Data Source=storage1.db");

src/AdditionalPatterns/UnitOfWork/UnitOfWorkLibrary/Example1/Infrastructure/OrderManagementContext1.cs

@@ -37,8 +37,11 @@ public virtual IEnumerable<T> Get(Expression<Func<T, bool>> predicate) =>
             .Where(predicate)
             .ToList();
 
-    public virtual T GetById(int id) =>
-        _context.Find<T>(id);
+    public virtual T GetById(int id)
+    {
+        var entity = _context.Find<T>(id);
+        return entity ?? throw new ArgumentException($"Entity with {id} doesn't exists");
+    }
 
     public virtual void Delete(T entity) =>
         _context.Remove(entity);

src/AdditionalPatterns/UnitOfWork/UnitOfWorkLibrary/Example1/Infrastructure/Repositories/Repository1.cs

@@ -5,8 +5,8 @@ namespace UnitOfWorkLibrary.Example2.Infrastructure;
 
 public class OrderManagementContext2 : DbContext, IUnitOfWork2
 {
-    public DbSet<Customer> Customers { get; set; }
-    public DbSet<Order> Orders { get; set; }
+    public DbSet<Customer> Customers => Set<Customer>();
+    public DbSet<Order> Orders => Set<Order>();
 
     public async Task<bool> SaveChangesAndDispatchDomainEventsAsync(CancellationToken cancellationToken = default)
     {

src/AdditionalPatterns/UnitOfWork/UnitOfWorkLibrary/Example2/Infrastructure/OrderManagementContext2.cs

@@ -38,14 +38,13 @@ public int Altitude
 
     public void Climb(int heightToClimb) => Altitude += heightToClimb;
 
-    public override bool Equals(object obj)
+    public override bool Equals(object? obj)
     {
-        if (obj.GetType() != GetType())
+        if (obj is not Aircraft incoming)
         {
             return false;
         }
 
-        var incoming = (Aircraft)obj;
         return CallSign.Equals(incoming.CallSign, StringComparison.Ordinal);
     }