From 358bb7da672cfecff93a3187d5c69f1f91f44a71 Mon Sep 17 00:00:00 2001
From: Hamed Ansarian <hamed.ansarian@hotmail.com>
Date: Fri, 21 Nov 2025 21:02:13 +0100
Subject: [PATCH] fix: upgrade package versions and target framework to net10.0

---
 Directory.Packages.props                           | 14 +++++++-------
 .../DistributedLockManager.csproj                  |  2 +-
 .../DistributedLockManager.Tests.csproj            |  4 ++--
 .../DistributedLockServiceIntegrationTests.cs      | 14 +++++++-------
 4 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/Directory.Packages.props b/Directory.Packages.props
index 5622959..6ed18f3 100644
--- a/Directory.Packages.props
+++ b/Directory.Packages.props
@@ -5,14 +5,14 @@
   </PropertyGroup>
   <ItemGroup>
     <!-- central versions -->
-    <PackageVersion Include="Aspire.StackExchange.Redis.DistributedCaching" Version="9.5.2" />
-    <PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="9.0.10" />
+    <PackageVersion Include="Aspire.StackExchange.Redis.DistributedCaching" Version="13.0.0" />
+    <PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="10.0.0" />
     <PackageVersion Include="RedLock.net" Version="2.3.2" />
-    <PackageVersion Include="NUnit" Version="3.13.3" />
-    <PackageVersion Include="NUnit3TestAdapter" Version="4.3.1" />
-    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.9.0" />
-    <PackageVersion Include="NSubstitute" Version="5.0.0" />
-    <PackageVersion Include="FluentAssertions" Version="6.9.0" />
+    <PackageVersion Include="NUnit" Version="4.4.0" />
+    <PackageVersion Include="NUnit3TestAdapter" Version="5.2.0" />
+    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.0.1" />
+    <PackageVersion Include="NSubstitute" Version="5.3.0" />
+    <PackageVersion Include="Shouldly" Version="4.3.0" />
     <PackageVersion Include="Testcontainers" Version="4.8.1" />
   </ItemGroup>
 </Project>
\ No newline at end of file
diff --git a/src/DistributedLockManager/DistributedLockManager.csproj b/src/DistributedLockManager/DistributedLockManager.csproj
index 4a563be..4969f80 100644
--- a/src/DistributedLockManager/DistributedLockManager.csproj
+++ b/src/DistributedLockManager/DistributedLockManager.csproj
@@ -1,6 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
-    <TargetFramework>net9.0</TargetFramework>
+    <TargetFramework>net10.0</TargetFramework>
     <Nullable>enable</Nullable>
     <ImplicitUsings>enable</ImplicitUsings>
     <GeneratePackageOnBuild>false</GeneratePackageOnBuild>
diff --git a/tests/DistributedLockManager.Tests/DistributedLockManager.Tests.csproj b/tests/DistributedLockManager.Tests/DistributedLockManager.Tests.csproj
index 8f11734..420c158 100644
--- a/tests/DistributedLockManager.Tests/DistributedLockManager.Tests.csproj
+++ b/tests/DistributedLockManager.Tests/DistributedLockManager.Tests.csproj
@@ -1,6 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
-    <TargetFramework>net9.0</TargetFramework>
+    <TargetFramework>net10.0</TargetFramework>
     <IsPackable>false</IsPackable>
     <Nullable>enable</Nullable>
   </PropertyGroup>
@@ -10,7 +10,7 @@
     <PackageReference Include="NUnit3TestAdapter" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" />
     <PackageReference Include="NSubstitute" />
-    <PackageReference Include="FluentAssertions" />
+    <PackageReference Include="Shouldly" />
     <PackageReference Include="Testcontainers" />
     <PackageReference Include="Aspire.StackExchange.Redis.DistributedCaching" />
   </ItemGroup>
diff --git a/tests/DistributedLockManager.Tests/DistributedLockServiceIntegrationTests.cs b/tests/DistributedLockManager.Tests/DistributedLockServiceIntegrationTests.cs
index 61f0c76..94b5322 100644
--- a/tests/DistributedLockManager.Tests/DistributedLockServiceIntegrationTests.cs
+++ b/tests/DistributedLockManager.Tests/DistributedLockServiceIntegrationTests.cs
@@ -8,6 +8,7 @@
 using DotNet.Testcontainers.Containers;
 using NUnit.Framework;
 using System.Linq;
+using Shouldly;
 
 namespace DistributedLockManager.Tests;
 
@@ -68,9 +69,9 @@ public async Task RunWithLockAsync_acquires_and_executes_action()
         var service = scope.ServiceProvider.GetRequiredService<IDistributedLockService<bool>>();
 
         var executed = false;
-        executed = await service.RunWithLockAsync(async () => await SampleAsync(executed), "integration:test:key", CancellationToken.None, expiryInSecond: 5, waitInSecond: 2, retryInSecond: 1);
+        executed = await service.RunWithLockAsync(SampleAsync, "integration:test:key", CancellationToken.None, expiryInSecond: 5, waitInSecond: 2, retryInSecond: 1);
 
-        Assert.IsTrue(executed, "Action should have been executed when lock acquired");
+        executed.ShouldBeTrue("Action should have been executed when lock acquired");
     }
     
     [Test, Order(2)]
@@ -108,13 +109,12 @@ async Task<bool> CriticalSection()
         await Task.WhenAll(tasks);
 
         // Only one should ever be in the critical section at a time
-        Assert.LessOrEqual(maxConcurrent, 1, $"Expected no concurrent executions, but saw {maxConcurrent}");
-        Assert.IsTrue(tasks.All(t => t.Result), "All tasks should complete successfully");
+        maxConcurrent.ShouldBeLessThanOrEqualTo(1, $"Expected no concurrent executions, but saw {maxConcurrent}");
+        tasks.All(t => t.Result).ShouldBeTrue("All tasks should complete successfully");
     }
 
-    private static async Task<bool> SampleAsync(bool executed)
+    private static async Task<bool> SampleAsync()
     {
-        executed = true;
-        return await Task.FromResult(executed);
+        return await Task.FromResult(true);
     }
 }