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

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 Dec 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
Diff view
Next Next commit
chore: enforcement of dbauthz tests was broken
  • Loading branch information
@Emyrk
Emyrk committed Dec 15, 2023
commit ef12fbb7b79d00e1a77da24081aafba33f7c2046
10 changes: 6 additions & 4 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,9 +28,10 @@ import (
)

var skipMethods = map[string]string{
"InTx": "Not relevant",
"Ping": "Not relevant",
"Wrappers": "Not relevant",
"InTx": "Not relevant",
"Ping": "Not relevant",
"Wrappers": "Not relevant",
"AquireLock": "Not relevant",
}

// TestMethodTestSuite runs MethodTestSuite.
Expand Down Expand Up @@ -62,7 +63,8 @@ func (s *MethodTestSuite) SetupSuite() {
mockStore.EXPECT().Wrappers().Return([]string{}).AnyTimes()
az := dbauthz.New(mockStore, nil, slog.Make(), coderdtest.AccessControlStorePointer())
// Take the underlying type of the interface.
azt := reflect.TypeOf(az).Elem()
azt := reflect.TypeOf(az)
require.Greater(s.T(), azt.NumMethod(), 0, "no methods found on querier")
s.methodAccounting = make(map[string]int)
for i := 0; i < azt.NumMethod(); i++ {
method := azt.Method(i)
Expand Down