Minor refactoring

aannenko · Oct 7, 2024 · 6888f2c · 6888f2c
1 parent c6549d8
commit 6888f2c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 12 deletions.
diff --git a/src/TransmissionManager.Database/Services/AppDbContext.cs b/src/TransmissionManager.Database/Services/AppDbContext.cs
@@ -4,8 +4,8 @@
 
 namespace TransmissionManager.Database.Services;
 
-[SuppressMessage("Trimming", "IL2026:Members annotated with 'RequiresUnreferencedCodeAttribute' require dynamic access otherwise can break functionality when trimming application code", Justification = "The app is thoroughly tested after trimming, that includes the used EF Core functionality")]
+[SuppressMessage("Trimming", "IL2026:Members annotated with 'RequiresUnreferencedCodeAttribute' require dynamic access otherwise can break functionality when trimming application code", Justification = "Tested after trimming, including the used EF Core functionality")]
 public sealed class AppDbContext(DbContextOptions<AppDbContext> options) : DbContext(options)
 {
-    public DbSet<Torrent> Torrents { get; set; }
+    public DbSet<Torrent> Torrents => Set<Torrent>();
 }
diff --git a/src/TransmissionManager.Transmission/Extensions/TransmissionServiceCollectionExtensions.cs b/src/TransmissionManager.Transmission/Extensions/TransmissionServiceCollectionExtensions.cs
@@ -42,17 +42,14 @@ private static void ConfigureHttpClient(IServiceProvider services, HttpClient cl
 
     private static void ConfigureResilience(ResiliencePipelineBuilder<HttpResponseMessage> builder)
     {
-        builder.AddRetry(
-            new()
+        builder
+            .AddRetry(new()
             {
-                ShouldHandle = new PredicateBuilder<HttpResponseMessage>()
-                    .HandleResult(static response => response is
-                    {
-                        IsSuccessStatusCode: false,
-                        StatusCode: not HttpStatusCode.Conflict,
-                    })
-            });
+                ShouldHandle = new PredicateBuilder<HttpResponseMessage>().HandleResult(IsRetriableResponse)
+            })
+            .AddTimeout(TimeSpan.FromSeconds(3));
 
-        builder.AddTimeout(TimeSpan.FromSeconds(3));
+        static bool IsRetriableResponse(HttpResponseMessage response) =>
+            response is { IsSuccessStatusCode: false, StatusCode: not HttpStatusCode.Conflict };
     }
 }