go/ucan
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix inconsistent test expectations

Browse Source
This commit is contained in:
Fabio Bozzo
2024-10-23 11:00:28 +02:00
committed by Michael Muré
parent 570bcdcb6c
commit 52ae2eaf60
1 changed files with 0 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
pkg/policy/selector/parsing_test.go
View File

@@ -336,21 +336,6 @@ func TestParse(t *testing.T) {
t.Run("idempotent optional", func(t *testing.T) { t.Run("idempotent optional", func(t *testing.T) {
sel, err := Parse(".foo???") sel, err := Parse(".foo???")
require.NoError(t, err) 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.Equal(t, 1, len(sel))
require.False(t, sel[0].Identity()) require.False(t, sel[0].Identity())
require.True(t, sel[0].Optional()) require.True(t, sel[0].Optional())