internal refactoring

phnx47 · phnx47 · commit d08b1a93cba6 · 2024-12-15T22:51:27.000+07:00
diff --git a/README.md b/README.md
@@ -4,12 +4,11 @@
 [![nuget](https://img.shields.io/nuget/v/Prometheus.Client.MetricPusher.HostedService?logo=nuget&style=flat-square)](https://www.nuget.org/packages/Prometheus.Client.MetricPusher.HostedService)
 [![nuget](https://img.shields.io/nuget/dt/Prometheus.Client.MetricPusher.HostedService?logo=nuget&style=flat-square)](https://www.nuget.org/packages/Prometheus.Client.MetricPusher.HostedService)
 [![codecov](https://img.shields.io/codecov/c/github/prom-client-net/prom-client-metricpusher-hostedservice?logo=codecov&style=flat-square)](https://app.codecov.io/gh/prom-client-net/prom-client-metricpusher-hostedservice)
-[![codefactor](https://img.shields.io/codefactor/grade/github/prom-client-net/prom-client-metricpusher-hostedservice?logo=codefactor&style=flat-square)](https://www.codefactor.io/repository/github/prom-client-net/prom-client-metricpusher-hostedservice)
 [![license](https://img.shields.io/github/license/prom-client-net/prom-client-metricpusher-hostedservice?style=flat-square)](https://github.com/prom-client-net/prom-client-metricpusher-hostedservice/blob/main/LICENSE)
 
 Extension for [Prometheus.Client](https://github.com/prom-client-net/prom-client)
 
-## Installation
+## Install
 
 ```sh
 dotnet add package Prometheus.Client.MetricPusher.HostedService
diff --git a/src/Defaults.cs b/src/Defaults.cs
@@ -4,5 +4,5 @@ namespace Prometheus.Client.MetricPusher.HostedService;
 
 internal static class Defaults
 {
-    internal static TimeSpan Interval = TimeSpan.FromMilliseconds(1000);
+    internal static TimeSpan PushInterval = TimeSpan.FromMilliseconds(1000);
 }
diff --git a/src/MetricPusherHostedService.cs b/src/MetricPusherHostedService.cs
@@ -5,20 +5,11 @@
 
 namespace Prometheus.Client.MetricPusher.HostedService;
 
-public class MetricPusherHostedService : BackgroundService
+public class MetricPusherHostedService(IMetricPusher pusher, TimeSpan pushInterval) : BackgroundService
 {
-    private readonly IMetricPusher _pusher;
-    private readonly TimeSpan _interval;
-
     public MetricPusherHostedService(IMetricPusher pusher)
-        : this(pusher, Defaults.Interval)
-    {
-    }
-
-    public MetricPusherHostedService(IMetricPusher pusher, TimeSpan interval)
+        : this(pusher, Defaults.PushInterval)
     {
-        _pusher = pusher;
-        _interval = interval;
     }
 
     protected override async Task ExecuteAsync(CancellationToken stoppingToken)
@@ -28,7 +19,7 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)
             await DoPushAsync();
             try
             {
-                await Task.Delay(_interval, stoppingToken);
+                await Task.Delay(pushInterval, stoppingToken);
             }
             catch (TaskCanceledException)
             {
@@ -43,7 +34,7 @@ async Task DoPushAsync()
         {
             try
             {
-                await _pusher.PushAsync();
+                await pusher.PushAsync();
             }
             catch (Exception)
             {
diff --git a/src/Prometheus.Client.MetricPusher.HostedService.csproj b/src/Prometheus.Client.MetricPusher.HostedService.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
     <TargetFramework>net6.0</TargetFramework>
-    <Description>Prometheus.Client.MetricPusher as HostedService</Description>
+    <Description>MetricPusher as HostedService</Description>
     <RepositoryUrl>https://github.com/prom-client-net/prom-client-metricpusher-hostedservice</RepositoryUrl>
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
   </PropertyGroup>
diff --git a/src/ServiceCollectionExtensions.cs b/src/ServiceCollectionExtensions.cs
@@ -8,17 +8,17 @@ public static class ServiceCollectionExtensions
 {
     public static IServiceCollection AddMetricPusherHostedService(this IServiceCollection services, IMetricPusher pusher)
     {
-        return AddMetricPusherHostedService(services, pusher, Defaults.Interval);
+        return AddMetricPusherHostedService(services, pusher, Defaults.PushInterval);
     }
 
-    public static IServiceCollection AddMetricPusherHostedService(this IServiceCollection services, IMetricPusher pusher, TimeSpan interval)
+    public static IServiceCollection AddMetricPusherHostedService(this IServiceCollection services, IMetricPusher pusher, TimeSpan pushInterval)
     {
         if (pusher == null)
             throw new ArgumentNullException(nameof(pusher));
-        if (interval == TimeSpan.Zero)
-            throw new ArgumentOutOfRangeException(nameof(interval));
+        if (pushInterval == TimeSpan.Zero)
+            throw new ArgumentOutOfRangeException(nameof(pushInterval));
 
-        services.AddSingleton<IHostedService, MetricPusherHostedService>(_ => new MetricPusherHostedService(pusher, interval));
+        services.AddSingleton<IHostedService, MetricPusherHostedService>(_ => new MetricPusherHostedService(pusher, pushInterval));
 
         return services;
     }
diff --git a/tests/ServiceCollectionExtensionsTests.cs b/tests/ServiceCollectionExtensionsTests.cs
@@ -3,7 +3,7 @@
 
 namespace Prometheus.Client.MetricPusher.HostedService.Tests;
 
-public class ServiceCollecttionExtensionsTests
+public class ServiceCollectionExtensionsTests
 {
     [Fact]
     public void AddMetricPusherService_WithNullMetricPusher_ThrowsArgumentNullException()

Original file line number	Diff line number	Diff line change
`@@ -4,5 +4,5 @@ namespace Prometheus.Client.MetricPusher.HostedService;`
`4`	`4`
`5`	`5`	`internal static class Defaults`
`6`	`6`	`{`
`7`		`- internal static TimeSpan Interval = TimeSpan.FromMilliseconds(1000);`
	`7`	`+ internal static TimeSpan PushInterval = TimeSpan.FromMilliseconds(1000);`
`8`	`8`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,20 +5,11 @@`
`5`	`5`
`6`	`6`	`namespace Prometheus.Client.MetricPusher.HostedService;`
`7`	`7`
`8`		`-public class MetricPusherHostedService : BackgroundService`
	`8`	`+public class MetricPusherHostedService(IMetricPusher pusher, TimeSpan pushInterval) : BackgroundService`
`9`	`9`	`{`
`10`		`- private readonly IMetricPusher _pusher;`
`11`		`- private readonly TimeSpan _interval;`
`12`		`-`
`13`	`10`	`public MetricPusherHostedService(IMetricPusher pusher)`
`14`		`- : this(pusher, Defaults.Interval)`
`15`		`- {`
`16`		`- }`
`17`		`-`
`18`		`- public MetricPusherHostedService(IMetricPusher pusher, TimeSpan interval)`
	`11`	`+ : this(pusher, Defaults.PushInterval)`
`19`	`12`	`{`
`20`		`- _pusher = pusher;`
`21`		`- _interval = interval;`
`22`	`13`	`}`
`23`	`14`
`24`	`15`	`protected override async Task ExecuteAsync(CancellationToken stoppingToken)`
`@@ -28,7 +19,7 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)`
`28`	`19`	`await DoPushAsync();`
`29`	`20`	`try`
`30`	`21`	`{`
`31`		`- await Task.Delay(_interval, stoppingToken);`
	`22`	`+ await Task.Delay(pushInterval, stoppingToken);`
`32`	`23`	`}`
`33`	`24`	`catch (TaskCanceledException)`
`34`	`25`	`{`
`@@ -43,7 +34,7 @@ async Task DoPushAsync()`
`43`	`34`	`{`
`44`	`35`	`try`
`45`	`36`	`{`
`46`		`- await _pusher.PushAsync();`
	`37`	`+ await pusher.PushAsync();`
`47`	`38`	`}`
`48`	`39`	`catch (Exception)`
`49`	`40`	`{`
Original file line number	Diff line number	Diff line change
`@@ -8,17 +8,17 @@ public static class ServiceCollectionExtensions`
`8`	`8`	`{`
`9`	`9`	`public static IServiceCollection AddMetricPusherHostedService(this IServiceCollection services, IMetricPusher pusher)`
`10`	`10`	`{`
`11`		`- return AddMetricPusherHostedService(services, pusher, Defaults.Interval);`
	`11`	`+ return AddMetricPusherHostedService(services, pusher, Defaults.PushInterval);`
`12`	`12`	`}`
`13`	`13`
`14`		`- public static IServiceCollection AddMetricPusherHostedService(this IServiceCollection services, IMetricPusher pusher, TimeSpan interval)`
	`14`	`+ public static IServiceCollection AddMetricPusherHostedService(this IServiceCollection services, IMetricPusher pusher, TimeSpan pushInterval)`
`15`	`15`	`{`
`16`	`16`	`if (pusher == null)`
`17`	`17`	`throw new ArgumentNullException(nameof(pusher));`
`18`		`- if (interval == TimeSpan.Zero)`
`19`		`- throw new ArgumentOutOfRangeException(nameof(interval));`
	`18`	`+ if (pushInterval == TimeSpan.Zero)`
	`19`	`+ throw new ArgumentOutOfRangeException(nameof(pushInterval));`
`20`	`20`
`21`		`- services.AddSingleton<IHostedService, MetricPusherHostedService>(_ => new MetricPusherHostedService(pusher, interval));`
	`21`	`+ services.AddSingleton<IHostedService, MetricPusherHostedService>(_ => new MetricPusherHostedService(pusher, pushInterval));`
`22`	`22`
`23`	`23`	`return services;`
`24`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`
`4`	`4`	`namespace Prometheus.Client.MetricPusher.HostedService.Tests;`
`5`	`5`
`6`		`-public class ServiceCollecttionExtensionsTests`
	`6`	`+public class ServiceCollectionExtensionsTests`
`7`	`7`	`{`
`8`	`8`	`[Fact]`
`9`	`9`	`public void AddMetricPusherService_WithNullMetricPusher_ThrowsArgumentNullException()`