Replace FluentAssertions with Shouldly for test assertions (#147)

* Replace FluentAssertions with Shouldly for test assertions

- Remove FluentAssertions 7.2.0 and add Shouldly 4.2.1 package
- Update GlobalUsings.cs to import Shouldly instead of FluentAssertions
- Migrate 50+ C# problem files with 140+ assertion patterns:
  - .Should().Be() → .ShouldBe()
  - .Should().BeTrue() → .ShouldBeTrue()
  - .Should().BeFalse() → .ShouldBeFalse()
  - .Should().BeEquivalentTo() → .ShouldBeEquivalentTo()
  - .Should().Throw<T>() → .ShouldThrow<T>()
  - Remove AssertionScope usage patterns
- Fix complex object comparisons for ListNode and TreeNode types
- Update CLAUDE.md documentation to reflect Shouldly usage
- All 74 C# + 6 F# tests pass with new assertion library

Benefits: More natural syntax, better error messages, lighter weight

* Fix dotnet format imports ordering in GlobalUsings.cs

* Fix .gitignore alphabetical ordering

* Move global usings from GlobalUsings.cs files to .csproj files

- Convert GlobalUsings.cs files to <Using> ItemGroup in project files
- Remove GlobalUsings.cs files from both LeetCode.CSharp and LeetCode projects
- Maintain same global imports functionality using MSBuild <Using> elements
- Build, tests, and formatting all pass successfully

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: si618

.gitignore

@@ -1,6 +1,7 @@
-.vs
+.plan
+.vs
+*.DotSettings.user
 bin
 obj
-*.DotSettings.user
 BenchmarkDotNet.Artifacts
 /.idea/.idea.LeetCode/.idea/material_theme_project_new.xml

CLAUDE.md

@@ -61,6 +61,6 @@ This is a LeetCode problem solutions repository with benchmarking capabilities,
 
 **Console Application**: Built with Spectre.Console, provides an interactive menu system and CLI commands (app, benchmark, info, list, workflow) for running benchmarks and viewing problem information.
 
-**Testing**: Uses xUnit with FluentAssertions for test assertions. Tests are co-located with problem implementations for easy access.
+**Testing**: Uses xUnit with Shouldly for test assertions. Tests are co-located with problem implementations for easy access.
 
 **Package Management**: Uses Directory.Packages.props for centralized NuGet package version management across all projects.

Directory.Packages.props

@@ -8,7 +8,6 @@
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageVersion>
-    <PackageVersion Include="FluentAssertions" Version="7.2.0" />
     <PackageVersion Include="FsUnit.xUnit" Version="6.0.1" />
     <PackageVersion Include="FSharp.Core" Version="8.0.400" />
     <PackageVersion Include="lolcat" Version="1.1.120" />
@@ -17,6 +16,7 @@
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageVersion>
+    <PackageVersion Include="Shouldly" Version="4.2.1" />
     <PackageVersion Include="Spectre.Console" Version="0.50.0" />
     <PackageVersion Include="Spectre.Console.Analyzer" Version="1.0.0">
       <PrivateAssets>all</PrivateAssets>

LeetCode.CSharp/GlobalUsings.cs

@@ -5,11 +5,23 @@
     <RootNamespace>LeetCode.CSharp</RootNamespace>
   </PropertyGroup>
 
+  <ItemGroup>
+    <Using Include="BenchmarkDotNet.Attributes" />
+    <Using Include="BenchmarkDotNet.Columns" />
+    <Using Include="BenchmarkDotNet.Configs" />
+    <Using Include="BenchmarkDotNet.Diagnosers" />
+    <Using Include="LeetCode.CSharp.Problems" />
+    <Using Include="Shouldly" />
+    <Using Include="System.Diagnostics.CodeAnalysis" />
+    <Using Include="System.Text" />
+    <Using Include="Xunit" />
+  </ItemGroup>
+
   <ItemGroup>
     <PackageReference Include="BenchmarkDotNet" />
-    <PackageReference Include="FluentAssertions" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" />
     <PackageReference Include="coverlet.collector" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" />
+    <PackageReference Include="Shouldly" />
     <PackageReference Include="xunit" />
     <PackageReference Include="xunit.runner.visualstudio" />
   </ItemGroup>

LeetCode.CSharp/LeetCode.CSharp.csproj

@@ -59,7 +59,7 @@ public void AddTwoNumbersTest(int[] list1, int[] list2, int[] expectedList)
         var listNode2 = new ListNode(list2);
         var expectedNode = new ListNode(expectedList);
         var result = AddTwoNumbers(listNode1, listNode2);
-        result.Should().NotBeNull();
+        result.ShouldNotBeNull();
         AssertEqual(result, expectedNode);
 
         return;
@@ -77,7 +77,7 @@ static void AssertEqual(ListNode? l1, ListNode? l2)
                     Assert.Fail("Either both nodes or neither node should be null");
                     return;
                 }
-                l1.val.Should().Be(l2.val);
+                l1.val.ShouldBe(l2.val);
                 l1 = l1.next;
                 l2 = l2.next;
             }

LeetCode.CSharp/Problems/AddTwoNumbers.cs

@@ -40,5 +40,5 @@ public static int BinarySearch(int[] nums, int target)
     [InlineData(new[] { -1, 0, 3, 5, 9, 12 }, 2, -1)]
     [InlineData(new int[] { }, 0, -1)]
     public void BinarySearchTest(int[] nums, int target, int expected) =>
-        BinarySearch(nums, target).Should().Be(expected);
+        BinarySearch(nums, target).ShouldBe(expected);
 }

LeetCode.CSharp/Problems/BinarySearch.cs

@@ -40,7 +40,7 @@ public void CanAttendMeetingsTest()
             new(9, 15)
         };
 
-        CanAttendMeetings(ex1).Should().BeFalse();
-        CanAttendMeetings(ex2).Should().BeTrue();
+        CanAttendMeetings(ex1).ShouldBeFalse();
+        CanAttendMeetings(ex2).ShouldBeTrue();
     }
 }

LeetCode.CSharp/Problems/CanAttendMeetings.cs

@@ -27,5 +27,5 @@ public static int ClimbStairs(int n)
     [Theory]
     [InlineData(2, 2)]
     [InlineData(3, 3)]
-    public void ClimbStairsTest(int n, int expected) => ClimbStairs(n).Should().Be(expected);
+    public void ClimbStairsTest(int n, int expected) => ClimbStairs(n).ShouldBe(expected);
 }

LeetCode.CSharp/Problems/ClimbStairs.cs

@@ -19,5 +19,5 @@ public static bool ContainsDuplicate(int[] nums)
     [InlineData(new[] { 1, 2, 3, 4 }, false)]
     [InlineData(new[] { 1, 1, 1, 3, 3, 4, 3, 2, 4, 2 }, true)]
     public void ContainsDuplicateTest(int[] nums, bool expected) =>
-        ContainsDuplicate(nums).Should().Be(expected);
+        ContainsDuplicate(nums).ShouldBe(expected);
 }