mirror of
https://github.com/niusmallnan/steve.git
synced 2025-07-04 02:06:18 +00:00
Merge pull request #68 from cmurphy/rbac_partition_testing
Add tests for RBAC partitioner
This commit is contained in:
commit
48efe7f967
252
pkg/stores/proxy/rbac_store_test.go
Normal file
252
pkg/stores/proxy/rbac_store_test.go
Normal file
@ -0,0 +1,252 @@
|
|||||||
|
package proxy
|
||||||
|
|
||||||
|
import (
|
||||||
|
"testing"
|
||||||
|
|
||||||
|
"github.com/rancher/apiserver/pkg/types"
|
||||||
|
"github.com/rancher/steve/pkg/accesscontrol"
|
||||||
|
"github.com/rancher/steve/pkg/stores/partition"
|
||||||
|
"github.com/rancher/wrangler/pkg/schemas"
|
||||||
|
"github.com/stretchr/testify/assert"
|
||||||
|
"k8s.io/apimachinery/pkg/util/sets"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestAll(t *testing.T) {
|
||||||
|
tests := []struct {
|
||||||
|
name string
|
||||||
|
apiOp *types.APIRequest
|
||||||
|
id string
|
||||||
|
schema *types.APISchema
|
||||||
|
wantPartitions []partition.Partition
|
||||||
|
}{
|
||||||
|
{
|
||||||
|
name: "all passthrough",
|
||||||
|
apiOp: &types.APIRequest{},
|
||||||
|
schema: &types.APISchema{
|
||||||
|
Schema: &schemas.Schema{
|
||||||
|
ID: "foo",
|
||||||
|
Attributes: map[string]interface{}{
|
||||||
|
"access": accesscontrol.AccessListByVerb{
|
||||||
|
"list": accesscontrol.AccessList{
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "*",
|
||||||
|
ResourceName: "*",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantPartitions: passthroughPartitions,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "global access for global request",
|
||||||
|
apiOp: &types.APIRequest{},
|
||||||
|
schema: &types.APISchema{
|
||||||
|
Schema: &schemas.Schema{
|
||||||
|
ID: "foo",
|
||||||
|
Attributes: map[string]interface{}{
|
||||||
|
"access": accesscontrol.AccessListByVerb{
|
||||||
|
"list": accesscontrol.AccessList{
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "*",
|
||||||
|
ResourceName: "r1",
|
||||||
|
},
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "*",
|
||||||
|
ResourceName: "r2",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantPartitions: []partition.Partition{
|
||||||
|
Partition{
|
||||||
|
Names: sets.NewString("r1", "r2"),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "namespace access for global request",
|
||||||
|
apiOp: &types.APIRequest{},
|
||||||
|
schema: &types.APISchema{
|
||||||
|
Schema: &schemas.Schema{
|
||||||
|
ID: "foo",
|
||||||
|
Attributes: map[string]interface{}{
|
||||||
|
"namespaced": true,
|
||||||
|
"access": accesscontrol.AccessListByVerb{
|
||||||
|
"list": accesscontrol.AccessList{
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n1",
|
||||||
|
ResourceName: "*",
|
||||||
|
},
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n2",
|
||||||
|
ResourceName: "*",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantPartitions: []partition.Partition{
|
||||||
|
Partition{
|
||||||
|
Namespace: "n1",
|
||||||
|
All: true,
|
||||||
|
},
|
||||||
|
Partition{
|
||||||
|
Namespace: "n2",
|
||||||
|
All: true,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "namespace access for namespaced request",
|
||||||
|
apiOp: &types.APIRequest{
|
||||||
|
Namespace: "n1",
|
||||||
|
},
|
||||||
|
schema: &types.APISchema{
|
||||||
|
Schema: &schemas.Schema{
|
||||||
|
ID: "foo",
|
||||||
|
Attributes: map[string]interface{}{
|
||||||
|
"namespaced": true,
|
||||||
|
"access": accesscontrol.AccessListByVerb{
|
||||||
|
"list": accesscontrol.AccessList{
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n1",
|
||||||
|
ResourceName: "*",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantPartitions: passthroughPartitions,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
// we still get a partition even if there is no access to it, it will be rejected by the API server later
|
||||||
|
name: "namespace access for invalid namespaced request",
|
||||||
|
apiOp: &types.APIRequest{
|
||||||
|
Namespace: "n2",
|
||||||
|
},
|
||||||
|
schema: &types.APISchema{
|
||||||
|
Schema: &schemas.Schema{
|
||||||
|
ID: "foo",
|
||||||
|
Attributes: map[string]interface{}{
|
||||||
|
"namespaced": true,
|
||||||
|
"access": accesscontrol.AccessListByVerb{
|
||||||
|
"list": accesscontrol.AccessList{
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n1",
|
||||||
|
ResourceName: "*",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantPartitions: []partition.Partition{
|
||||||
|
Partition{
|
||||||
|
Namespace: "n2",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "by names access for global request",
|
||||||
|
apiOp: &types.APIRequest{},
|
||||||
|
schema: &types.APISchema{
|
||||||
|
Schema: &schemas.Schema{
|
||||||
|
ID: "foo",
|
||||||
|
Attributes: map[string]interface{}{
|
||||||
|
"namespaced": true,
|
||||||
|
"access": accesscontrol.AccessListByVerb{
|
||||||
|
"list": accesscontrol.AccessList{
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n1",
|
||||||
|
ResourceName: "r1",
|
||||||
|
},
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n1",
|
||||||
|
ResourceName: "r2",
|
||||||
|
},
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n2",
|
||||||
|
ResourceName: "r1",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantPartitions: []partition.Partition{
|
||||||
|
Partition{
|
||||||
|
Namespace: "n1",
|
||||||
|
Names: sets.NewString("r1", "r2"),
|
||||||
|
},
|
||||||
|
Partition{
|
||||||
|
Namespace: "n2",
|
||||||
|
Names: sets.NewString("r1"),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "by names access for namespaced request",
|
||||||
|
apiOp: &types.APIRequest{
|
||||||
|
Namespace: "n1",
|
||||||
|
},
|
||||||
|
schema: &types.APISchema{
|
||||||
|
Schema: &schemas.Schema{
|
||||||
|
ID: "foo",
|
||||||
|
Attributes: map[string]interface{}{
|
||||||
|
"namespaced": true,
|
||||||
|
"access": accesscontrol.AccessListByVerb{
|
||||||
|
"list": accesscontrol.AccessList{
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n1",
|
||||||
|
ResourceName: "r1",
|
||||||
|
},
|
||||||
|
accesscontrol.Access{
|
||||||
|
Namespace: "n2",
|
||||||
|
ResourceName: "r1",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantPartitions: []partition.Partition{
|
||||||
|
Partition{
|
||||||
|
Namespace: "n1",
|
||||||
|
Names: sets.NewString("r1"),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "by id",
|
||||||
|
apiOp: &types.APIRequest{},
|
||||||
|
id: "n1/r1",
|
||||||
|
schema: &types.APISchema{
|
||||||
|
Schema: &schemas.Schema{
|
||||||
|
ID: "foo",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantPartitions: []partition.Partition{
|
||||||
|
Partition{
|
||||||
|
Namespace: "n1",
|
||||||
|
Names: sets.NewString("r1"),
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, test := range tests {
|
||||||
|
t.Run(test.name, func(t *testing.T) {
|
||||||
|
partitioner := rbacPartitioner{}
|
||||||
|
verb := "list"
|
||||||
|
gotPartitions, gotErr := partitioner.All(test.apiOp, test.schema, verb, test.id)
|
||||||
|
assert.Nil(t, gotErr)
|
||||||
|
assert.Equal(t, test.wantPartitions, gotPartitions)
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
Loading…
Reference in New Issue
Block a user