From 52ae2eaf60cf656f9446649f523711f863a22a2b Mon Sep 17 00:00:00 2001 From: Fabio Bozzo Date: Wed, 23 Oct 2024 11:00:28 +0200 Subject: [PATCH] fix inconsistent test expectations --- pkg/policy/selector/parsing_test.go | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/pkg/policy/selector/parsing_test.go b/pkg/policy/selector/parsing_test.go index 6281159..1dac6b6 100644 --- a/pkg/policy/selector/parsing_test.go +++ b/pkg/policy/selector/parsing_test.go @@ -336,21 +336,6 @@ func TestParse(t *testing.T) { t.Run("idempotent optional", func(t *testing.T) { sel, err := Parse(".foo???") require.NoError(t, err) - //require.Equal(t, 2, len(sel)) - //require.True(t, sel[0].Identity()) - //require.False(t, sel[0].Optional()) - //require.False(t, sel[0].Iterator()) - //require.Empty(t, sel[0].Slice()) - //require.Empty(t, sel[0].Field()) - //require.Empty(t, sel[0].Index()) - //require.False(t, sel[1].Identity()) - //require.True(t, sel[1].Optional()) - //require.False(t, sel[1].Iterator()) - //require.Empty(t, sel[1].Slice()) - //require.Equal(t, sel[1].Field(), "foo") - //require.Empty(t, sel[1].Index()) - - // TODO: understand why the original test expects a leading identity segment require.Equal(t, 1, len(sel)) require.False(t, sel[0].Identity()) require.True(t, sel[0].Optional())