Skip to content

Commit 6191561

Browse files
committed
Test audit methods
1 parent 0add01a commit 6191561

File tree

4 files changed

+77
-0
lines changed

4 files changed

+77
-0
lines changed

coderd/authzquery/audit_test.go

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
package authzquery_test
2+
3+
import (
4+
"testing"
5+
6+
"github.com/coder/coder/coderd/database/dbgen"
7+
8+
"github.com/coder/coder/coderd/database"
9+
"github.com/coder/coder/coderd/rbac"
10+
)
11+
12+
func (suite *MethodTestSuite) TestAuditLogs() {
13+
suite.Run("InsertAuditLog", func() {
14+
suite.RunMethodTest(func(t *testing.T, db database.Store) MethodCase {
15+
return methodCase(inputs(database.InsertAuditLogParams{
16+
ResourceType: database.ResourceTypeOrganization,
17+
Action: database.AuditActionCreate,
18+
}),
19+
asserts(rbac.ResourceAuditLog, rbac.ActionCreate))
20+
})
21+
})
22+
suite.Run("GetAuditLogsOffset", func() {
23+
suite.RunMethodTest(func(t *testing.T, db database.Store) MethodCase {
24+
_ = dbgen.AuditLog(t, db, database.AuditLog{})
25+
_ = dbgen.AuditLog(t, db, database.AuditLog{})
26+
return methodCase(inputs(database.GetAuditLogsOffsetParams{
27+
Limit: 10,
28+
}),
29+
asserts(rbac.ResourceAuditLog, rbac.ActionRead))
30+
})
31+
})
32+
}

coderd/database/dbgen/generator.go

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import (
66
"database/sql"
77
"encoding/hex"
88
"fmt"
9+
"net"
910
"testing"
1011
"time"
1112

@@ -21,6 +22,34 @@ import (
2122
// All methods take in a 'seed' object. Any provided fields in the seed will be
2223
// maintained. Any fields omitted will have sensible defaults generated.
2324

25+
func AuditLog(t *testing.T, db database.Store, seed database.AuditLog) database.AuditLog {
26+
log, err := db.InsertAuditLog(context.Background(), database.InsertAuditLogParams{
27+
ID: takeFirst(seed.ID, uuid.New()),
28+
Time: takeFirst(seed.Time, time.Now()),
29+
UserID: takeFirst(seed.UserID, uuid.New()),
30+
OrganizationID: takeFirst(seed.OrganizationID, uuid.New()),
31+
Ip: pqtype.Inet{
32+
IPNet: takeFirstIP(seed.Ip.IPNet, net.IPNet{}),
33+
Valid: takeFirst(seed.Ip.Valid, false),
34+
},
35+
UserAgent: sql.NullString{
36+
String: takeFirst(seed.UserAgent.String, ""),
37+
Valid: takeFirst(seed.UserAgent.Valid, false),
38+
},
39+
ResourceType: takeFirst(seed.ResourceType, database.ResourceTypeOrganization),
40+
ResourceID: takeFirst(seed.ResourceID, uuid.New()),
41+
ResourceTarget: takeFirst(seed.ResourceTarget, uuid.NewString()),
42+
Action: takeFirst(seed.Action, database.AuditActionCreate),
43+
Diff: takeFirstBytes(seed.Diff, []byte("{}")),
44+
StatusCode: takeFirst(seed.StatusCode, 200),
45+
AdditionalFields: takeFirstBytes(seed.Diff, []byte("{}")),
46+
RequestID: takeFirst(seed.RequestID, uuid.New()),
47+
ResourceIcon: takeFirst(seed.ResourceIcon, ""),
48+
})
49+
require.NoError(t, err, "insert audit log")
50+
return log
51+
}
52+
2453
func Template(t *testing.T, db database.Store, seed database.Template) database.Template {
2554
template, err := db.InsertTemplate(context.Background(), database.InsertTemplateParams{
2655
ID: takeFirst(seed.ID, uuid.New()),

coderd/database/dbgen/generator_test.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,14 @@ import (
1414
func TestGenerator(t *testing.T) {
1515
t.Parallel()
1616

17+
t.Run("AuditLog", func(t *testing.T) {
18+
t.Parallel()
19+
db := databasefake.New()
20+
_ = dbgen.AuditLog(t, db, database.AuditLog{})
21+
logs := must(db.GetAuditLogsOffset(context.Background(), database.GetAuditLogsOffsetParams{Limit: 1}))
22+
require.Len(t, logs, 1)
23+
})
24+
1725
t.Run("APIKey", func(t *testing.T) {
1826
t.Parallel()
1927
db := databasefake.New()

coderd/database/dbgen/take.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,13 @@
11
package dbgen
22

3+
import "net"
4+
5+
func takeFirstIP(values ...net.IPNet) net.IPNet {
6+
return takeFirstF(values, func(v net.IPNet) bool {
7+
return len(v.IP) != 0 && len(v.Mask) != 0
8+
})
9+
}
10+
311
// takeFirstBytes implements takeFirst for []byte.
412
// []byte is not a comparable type.
513
func takeFirstBytes(values ...[]byte) []byte {

0 commit comments

Comments
 (0)