8000 chore: enforcement of dbauthz tests was broken by Emyrk · Pull Request #11218 · coder/coder · GitHub
[go: up one dir, main page]
Skip to content

chore: enforcement of dbauthz tests was broken #11218

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
Dec 15, 2023
Merged

chore: enforcement of dbauthz tests was broken #11218

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
ef12fbb
chore: enforcement of dbauthz tests was broken
Emyrk Dec 14, 2023
15e91ff
Add another skip
Emyrk Dec 14, 2023
a876975
dbauthz: allow asserting errors in tests
johnstcn D 8000 ec 15, 2023
f38be5d
add tailnet methods
johnstcn Dec 15, 2023
193d3b3
add dbcrypt methods
johnstcn Dec 15, 2023
2ddae13
fixup! chore: enforcement of dbauthz tests was broken
johnstcn Dec 15, 2023
c31b190
linting
Emyrk Dec 15, 2023
e930fb2
Add more dbauthz tests
Emyrk Dec 15, 2023
f6fe76f
GetTemplateInsights require an actor to call
Emyrk Dec 15, 2023
48857d2
Add more system functions;
Emyrk Dec 15, 2023
2e2f6be
Implement another chunk
Emyrk Dec 15, 2023
5d2f9b1
Another chunk of tests
Emyrk Dec 15, 2023
6481561
Another chunk
Emyrk Dec 15, 2023
55fbd8f
Update template tests
Emyrk Dec 15, 2023
b5ba222
Passing tests
Emyrk Dec 15, 2023
531c7f8
Lint
Emyrk Dec 15, 2023
440573a
lint
Emyrk Dec 15, 2023
af0b3f5
fmt
Emyrk Dec 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
< 10000 h5 data-view-component="true" class="mb-2"> Diff view
Prev Previous commit
Next Next commit
lint
  • Loading branch information
@Emyrk
Emyrk committed Dec 15, 2023
commit 440573aa887895916f4edc2b8620e9a0480fbbf4
4 changes: 2 additions & 2 deletions coderd/database/dbauthz/dbauthz_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2066,7 +2066,7 @@ func (s *MethodTestSuite) TestSystemFunctions() {
check.Args().Asserts(rbac.ResourceSystem, rbac.ActionDelete)
}))
s.Run("InsertWorkspaceAgentStats", s.Subtest(func(db database.Store, check *expects) {
check.Args(database.InsertWorkspaceAgentStatsParams{}).Asserts(rbac.ResourceSystem, rbac.ActionCreate).Errors(matchAnyError)
check.Args(database.InsertWorkspaceAgentStatsParams{}).Asserts(rbac.ResourceSystem, rbac.ActionCreate).Errors(errMatchAny)
}))
s.Run("InsertWorkspaceAppStats", s.Subtest(func(db database.Store, check *expects) {
check.Args(database.InsertWorkspaceAppStatsParams{}).Asserts(rbac.ResourceSystem, rbac.ActionCreate)
Expand Down Expand Up @@ -2131,7 +2131,7 @@ func (s *MethodTestSuite) TestSystemFunctions() {
check.Args().Asserts(rbac.ResourceSystem, rbac.ActionUpdate)
}))
s.Run("InsertMissingGroups", s.Subtest(func(db database.Store, check *expects) {
check.Args(database.InsertMissingGroupsParams{}).Asserts(rbac.ResourceSystem, rbac.ActionCreate).Errors(matchAnyError)
check.Args(database.InsertMissingGroupsParams{}).Asserts(rbac.ResourceSystem, rbac.ActionCreate).Errors(errMatchAny)
}))
s.Run("UpdateUserLoginType", s.Subtest(func(db database.Store, check *expects) {
u := dbgen.User(s.T(), db, database.User{})
Expand Down
4 changes: 2 additions & 2 deletions coderd/database/dbauthz/setup_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ import (
"github.com/coder/coder/v2/coderd/util/slice"
)

var matchAnyError = errors.New("match any error")
var errMatchAny = errors.New("match any error")

var skipMethods = map[string]string{
"InTx": "Not relevant",
Expand Down Expand Up @@ -177,7 +177,7 @@ func (s *MethodTestSuite) Subtest(testCaseF func(db database.Store, check *expec
if testCase.err == nil {
s.NoError(err, "method %q returned an error", methodName)
} else {
if errors.Is(testCase.err, matchAnyError) {
if errors.Is(testCase.err, errMatchAny) {
// This means we do not care exactly what the error is.
s.Error(err, "method %q returned an error", methodName)
} else {
Expand Down
0