address review

polyadic · Aug 30, 2023 · e4edc51 · e4edc51
1 parent c41638e
commit e4edc51
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 16 deletions.
diff --git a/Funcky.Async.Test/Extensions/AsyncEnumerableExtensions/SlidingWindowTest.cs b/Funcky.Async.Test/Extensions/AsyncEnumerableExtensions/SlidingWindowTest.cs
@@ -74,9 +74,9 @@ public async Task SlidingWindowReturnsASequenceOfConsecutiveWindowsAsync()
 
         await AsyncAssert.Collection(
             source.SlidingWindow(width),
-            window => { Assert.Equal(Enumerable.Range(0, width), window); },
-            window => { Assert.Equal(Enumerable.Range(1, width), window); },
-            window => { Assert.Equal(Enumerable.Range(2, width), window); });
+            window => Assert.Equal(Enumerable.Range(0, width), window),
+            window => Assert.Equal(Enumerable.Range(1, width), window),
+            window => Assert.Equal(Enumerable.Range(2, width), window));
     }
 
     [Fact]

diff --git a/Funcky.Async/Funcky.Async.csproj b/Funcky.Async/Funcky.Async.csproj
@@ -14,8 +14,7 @@
         <EnableNETAnalyzers>true</EnableNETAnalyzers>
         <AnalysisModeReliability>All</AnalysisModeReliability>
         <EnablePackageValidation>true</EnablePackageValidation>
-        <NoWarn>$(NoWarn);RS0026</NoWarn>
-        <!-- RS0026: Do not add multiple overloads with optional parameters -->
+        <NoWarn>$(NoWarn);RS0026</NoWarn><!-- RS0026: Do not add multiple overloads with optional parameters -->
     </PropertyGroup>
     <PropertyGroup>
         <RootNamespace>Funcky</RootNamespace>

diff --git a/Funcky.Test/Extensions/EnumerableExtensions/SlidingWindowTest.cs b/Funcky.Test/Extensions/EnumerableExtensions/SlidingWindowTest.cs
@@ -70,8 +70,8 @@ public void SlidingWindowReturnsASequenceOfConsecutiveWindows()
 
         Assert.Collection(
             source.SlidingWindow(width),
-            window => { Assert.Equal(Enumerable.Range(0, width), window); },
-            window => { Assert.Equal(Enumerable.Range(1, width), window); },
-            window => { Assert.Equal(Enumerable.Range(2, width), window); });
+            window => Assert.Equal(Enumerable.Range(0, width), window),
+            window => Assert.Equal(Enumerable.Range(1, width), window),
+            window => Assert.Equal(Enumerable.Range(2, width), window));
     }
 }
diff --git a/Funcky.Test/Extensions/StringExtensions/ChunkOnStringTest.cs b/Funcky.Test/Extensions/StringExtensions/ChunkOnStringTest.cs
@@ -7,10 +7,10 @@ public class ChunkOnStringTest
 {
     [Property]
     public Property ChunkingAnEmptyStringWillAlwaysYieldAndEmptySequence(PositiveInt width)
-            => string.Empty
-                .Chunk(width.Get)
-                .None()
-                .ToProperty();
+        => string.Empty
+            .Chunk(width.Get)
+            .None()
+            .ToProperty();
 
     [Fact]
     public void GivenAnSingleElementListWeGetEnumerableWithOneElement()

diff --git a/Funcky.Test/Extensions/StringExtensions/SlidingWindowOnStringTest.cs b/Funcky.Test/Extensions/StringExtensions/SlidingWindowOnStringTest.cs
@@ -20,10 +20,10 @@ public void SlidingWindowReturnsAListOfOverlappingPartialStrings()
 
         Assert.Collection(
             source.SlidingWindow(width),
-            window => { Assert.Equal("epsi", window); },
-            window => { Assert.Equal("psil", window); },
-            window => { Assert.Equal("silo", window); },
-            window => { Assert.Equal("ilon", window); });
+            window => Assert.Equal("epsi", window),
+            window => Assert.Equal("psil", window),
+            window => Assert.Equal("silo", window),
+            window => Assert.Equal("ilon", window));
     }
 
     [Fact]