Merge pull request #1031 from FirelyTeam/copilot/fix-1029-2

Remove redundant empty check from CqlOperators.Slice method

Author: alexzautke

Cql/CoreTests/PrimitiveTests.cs

@@ -3885,6 +3885,28 @@ public void SliceTakeIsNull()
             Assert.AreEqual(expectedList, slicedList);
         }
 
+        [TestMethod]
+        public void SliceEmptyEnumerableWithIEnumerableNotCollection()
+        {
+            // Test Slice with empty enumerable that's not a collection type
+            // This ensures behavior remains consistent after removing the redundant empty check
+            var rtx = GetNewContext();
+            var inputEnumerable = Enumerable.Empty<int>().Where(x => true); // Creates IEnumerable<int> not a collection
+            var expectedList = new List<int>();
+            
+            // Test various slice operations on empty enumerable
+            var slicedList1 = rtx.Operators.Slice(inputEnumerable, 0, 5);
+            var slicedList2 = rtx.Operators.Slice(inputEnumerable, 2, null);
+            var slicedList3 = rtx.Operators.Slice(inputEnumerable, null, null);
+            
+            Assert.IsNotNull(slicedList1);
+            Assert.IsNotNull(slicedList2);
+            Assert.IsNotNull(slicedList3);
+            CollectionAssert.AreEqual(expectedList, slicedList1.ToList());
+            CollectionAssert.AreEqual(expectedList, slicedList2.ToList());
+            CollectionAssert.AreEqual(expectedList, slicedList3.ToList());
+        }
+
         #endregion
 
         #region ListSkip and ListTake tests

Cql/Cql.Runtime/Operators/CqlOperators.ListOperators.cs

@@ -1048,9 +1048,6 @@ internal partial class CqlOperators
             if (source == null)
                 return null;
 
-            if (!source.Any())
-                return Enumerable.Empty<T>();
-
             if (startIndex == null && endIndex == null)
                 return source;