Fix using forbidden error

Emyrk · Emyrk · commit 62185702599e · 2023-02-09T12:22:07.000-06:00
diff --git a/coderd/coderdtest/authorize.go b/coderd/coderdtest/authorize.go
@@ -445,7 +445,7 @@ func NewAuthTester(ctx context.Context, t *testing.T, client *codersdk.Client, a
 func (a *AuthTester) Test(ctx context.Context, assertRoute map[string]RouteCheck, skipRoutes map[string]string) {
 	// Always fail auth from this point forward
 	a.authorizer.Wrapped = &FakeAuthorizer{
-		AlwaysReturn: rbac.ForbiddenWithInternal(xerrors.New("fake implementation"), nil, nil),
+		AlwaysReturn: rbac.ForbiddenWithInternal(xerrors.New("fake implementation"), rbac.Subject{}, "", rbac.Object{}, nil),
 	}
 
 	routeMissing := make(map[string]bool)
diff --git a/coderd/rbac/builtin_internal_test.go b/coderd/rbac/builtin_internal_test.go
@@ -148,7 +148,6 @@ func TestRegoInputValue(t *testing.T) {
 
 		cmp := manual.Compare(general)
 		require.Equal(t, 0, cmp, "manual and general input values should be equal")
-
 	})
 }
 

Original file line number	Diff line number	Diff line change
`@@ -445,7 +445,7 @@ func NewAuthTester(ctx context.Context, t testing.T, client codersdk.Client, a`
`445`	`445`	`func (a *AuthTester) Test(ctx context.Context, assertRoute map[string]RouteCheck, skipRoutes map[string]string) {`
`446`	`446`	`// Always fail auth from this point forward`
`447`	`447`	`a.authorizer.Wrapped = &FakeAuthorizer{`
`448`		`- AlwaysReturn: rbac.ForbiddenWithInternal(xerrors.New("fake implementation"), nil, nil),`
	`448`	`+ AlwaysReturn: rbac.ForbiddenWithInternal(xerrors.New("fake implementation"), rbac.Subject{}, "", rbac.Object{}, nil),`
`449`	`449`	`}`
`450`	`450`
`451`	`451`	`routeMissing := make(map[string]bool)`
Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,6 @@ func TestRegoInputValue(t *testing.T) {`
`148`	`148`
`149`	`149`	`cmp := manual.Compare(general)`
`150`	`150`	`require.Equal(t, 0, cmp, "manual and general input values should be equal")`
`151`		`-`
`152`	`151`	`})`
`153`	`152`	`}`
`154`	`153`