From 0c9ddd4de7b1aff87dde912c104d9dcc27c4a834 Mon Sep 17 00:00:00 2001 From: Carlos Alexandro Becker Date: Thu, 14 Aug 2025 16:15:56 -0300 Subject: [PATCH] test: simplify Signed-off-by: Carlos Alexandro Becker --- internal/csync/slices_test.go | 25 +++---------------------- 1 file changed, 3 insertions(+), 22 deletions(-) diff --git a/internal/csync/slices_test.go b/internal/csync/slices_test.go index e2d8bb30bbf62829f9bbcc45a49a8b94b624ccb1..85aedbaba40103ff9a8979e5c70299223f74591f 100644 --- a/internal/csync/slices_test.go +++ b/internal/csync/slices_test.go @@ -21,13 +21,7 @@ func TestLazySlice_Seq(t *testing.T) { time.Sleep(10 * time.Millisecond) // Small delay to ensure loading happens return data }) - - var result []string - for v := range s.Seq() { - result = append(result, v) - } - - require.Equal(t, data, result) + require.Equal(t, data, slices.Collect(s.Seq())) }) } @@ -46,30 +40,17 @@ func TestLazySlice_SeqWaitsForLoading(t *testing.T) { }) require.False(t, loaded.Load(), "should not be loaded immediately") - - var result []string - for v := range s.Seq() { - result = append(result, v) - } - + require.Equal(t, data, slices.Collect(s.Seq())) require.True(t, loaded.Load(), "should be loaded after Seq") - require.Equal(t, data, result) }) } func TestLazySlice_EmptySlice(t *testing.T) { t.Parallel() - s := NewLazySlice(func() []string { return []string{} }) - - var result []string - for v := range s.Seq() { - result = append(result, v) - } - - require.Empty(t, result) + require.Empty(t, slices.Collect(s.Seq())) } func TestLazySlice_EarlyBreak(t *testing.T) {