Use spread operator

fluentassertions · Jan 1, 2024 · e1f7606 · e1f7606
1 parent 9222e0f
commit e1f7606
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 16 deletions.
diff --git a/Src/FluentAssertions/Common/TypeMemberReflector.cs b/Src/FluentAssertions/Common/TypeMemberReflector.cs
@@ -18,7 +18,7 @@ public TypeMemberReflector(Type typeToReflect, MemberVisibility visibility)
     {
         Properties = LoadProperties(typeToReflect, visibility);
         Fields = LoadFields(typeToReflect, visibility);
-        Members = Properties.Concat<MemberInfo>(Fields).ToArray();
+        Members = [.. Properties, .. Fields];
     }
 
     public MemberInfo[] Members { get; }

diff --git a/Tests/FluentAssertions.Specs/AssertionExtensionsSpecs.cs b/Tests/FluentAssertions.Specs/AssertionExtensionsSpecs.cs
@@ -129,21 +129,21 @@ public void Should_methods_have_a_matching_overload_to_guard_against_chaining_an
             .Where(m => m.Name == "Should")
             .ToList();
 
-        List<Type> realOverloads = shouldOverloads
-            .Where(m => !IsGuardOverload(m))
-            .Select(t => GetMostParentType(t.ReturnType))
-            .Distinct()
-            .Concat(new[]
-            {
-                // @jnyrup: DateTimeRangeAssertions and DateTimeOffsetRangeAssertions are manually added here,
-                // because they expose AndConstraints,
-                // and hence should have a guarding Should(DateTimeRangeAssertions _) overloads,
-                // but they do not have a regular Should() overload,
-                // as they are always constructed through the fluent API.
-                typeof(DateTimeRangeAssertions<>),
-                typeof(DateTimeOffsetRangeAssertions<>),
-            })
-            .ToList();
+        List<Type> realOverloads =
+        [
+            ..shouldOverloads
+                .Where(m => !IsGuardOverload(m))
+                .Select(t => GetMostParentType(t.ReturnType))
+                .Distinct(),
+
+            // @jnyrup: DateTimeRangeAssertions and DateTimeOffsetRangeAssertions are manually added here,
+            // because they expose AndConstraints,
+            // and hence should have a guarding Should(DateTimeRangeAssertions _) overloads,
+            // but they do not have a regular Should() overload,
+            // as they are always constructed through the fluent API.
+            typeof(DateTimeRangeAssertions<>),
+            typeof(DateTimeOffsetRangeAssertions<>)
+        ];
 
         List<Type> fakeOverloads = shouldOverloads
             .Where(m => IsGuardOverload(m))