fix: Fix audit search query (#4352)

BrunoQuaresma · web-flow · commit bf63cc929a3a · 2022-10-03T20:56:54.000-03:00
diff --git a/coderd/audit.go b/coderd/audit.go
@@ -265,18 +265,23 @@ func auditSearchQuery(query string) (database.GetAuditLogsOffsetParams, []coders
 		Username:     parser.String(searchParams, "", "username"),
 		Email:        parser.String(searchParams, "", "email"),
 	}
-
 	return filter, parser.Errors
 }
 
 func resourceTypeFromString(resourceTypeString string) string {
 	switch codersdk.ResourceType(resourceTypeString) {
 	case codersdk.ResourceTypeOrganization:
+		return resourceTypeString
 	case codersdk.ResourceTypeTemplate:
+		return resourceTypeString
 	case codersdk.ResourceTypeTemplateVersion:
+		return resourceTypeString
 	case codersdk.ResourceTypeUser:
+		return resourceTypeString
 	case codersdk.ResourceTypeWorkspace:
+		return resourceTypeString
 	case codersdk.ResourceTypeGitSSHKey:
+		return resourceTypeString
 	case codersdk.ResourceTypeAPIKey:
 		return resourceTypeString
 	}
@@ -286,10 +291,11 @@ func resourceTypeFromString(resourceTypeString string) string {
 func actionFromString(actionString string) string {
 	switch codersdk.AuditAction(actionString) {
 	case codersdk.AuditActionCreate:
+		return actionString
 	case codersdk.AuditActionWrite:
+		return actionString
 	case codersdk.AuditActionDelete:
 		return actionString
-	default:
 	}
 	return ""
 }
diff --git a/coderd/audit_test.go b/coderd/audit_test.go
@@ -130,6 +130,7 @@ func TestAuditLogsFilter(t *testing.T) {
 		}
 
 		for _, testCase := range testCases {
+			testCase := testCase
 			// Test filtering
 			t.Run(testCase.Name, func(t *testing.T) {
 				t.Parallel()

Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,7 @@ func TestAuditLogsFilter(t *testing.T) {`
`130`	`130`	`}`
`131`	`131`
`132`	`132`	`for _, testCase := range testCases {`
	`133`	`+ testCase := testCase`
`133`	`134`	`// Test filtering`
`134`	`135`	`t.Run(testCase.Name, func(t *testing.T) {`
`135`	`136`	`t.Parallel()`