Skip to content

Commit

Permalink
feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
AlexisJasso committed Apr 25, 2024
1 parent 51247f0 commit 6ae10a3
Show file tree
Hide file tree
Showing 6 changed files with 57 additions and 6 deletions.
2 changes: 1 addition & 1 deletion pager/opsgenie.go
Original file line number Diff line number Diff line change
Expand Up @@ -480,7 +480,7 @@ func (o *Opsgenie) saveEscalationPolicyStepToDB(ctx context.Context, policyID st
// review should certainly be required here.

if t.TargetType == store.TARGET_TYPE_SCHEDULE {
schedules, err := store.UseQueries(ctx).ListExtSchedulesByPrefix(ctx, fmt.Sprintf(`%s%%`, rule.Recipient.Id))
schedules, err := store.UseQueries(ctx).ListExtSchedulesLikeID(ctx, fmt.Sprintf(`%s%%`, rule.Recipient.Id))
if err != nil {
return fmt.Errorf("getting schedules starting with ID %s: %w", rule.Recipient.Id, err)
}
Expand Down
6 changes: 6 additions & 0 deletions pager/opsgenie_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,12 @@ func TestOpsgenie(t *testing.T) {
t.Run("LoadTeamMembers", func(t *testing.T) {
ctx, og := setup(t)

if err := og.LoadUsers(ctx); err != nil {
t.Fatalf("error loading users: %s", err)
}
if err := og.LoadTeams(ctx); err != nil {
t.Fatalf("error loading teams: %s", err)
}
if err := og.LoadTeamMembers(ctx); err != nil {
t.Fatalf("error loading team members: %s", err)
}
Expand Down
25 changes: 24 additions & 1 deletion pager/testdata/TestOpsgenie/LoadTeamMembers.golden.json
Original file line number Diff line number Diff line change
@@ -1 +1,24 @@
null
[
{
"ext_team": {
"id": "b7acbc33-9853-4150-8a4b-10156d9408c8",
"name": "Customer Success",
"slug": "customer-success",
"fh_team_id": {
"String": "",
"Valid": false
},
"is_group": 0,
"to_import": 0
},
"ext_user": {
"id": "b5b92115-bfe7-43eb-8c2a-e467f2e5ddc4",
"name": "john doe",
"email": "[email protected]",
"fh_user_id": {
"String": "",
"Valid": false
}
}
}
]
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
{
"data": {
"id": "b7acbc33-9853-4150-8a4b-10156d9408c8",
"name": "Customer Success",
"description": "",
"members": [
{
"user": {
"id": "b5b92115-bfe7-43eb-8c2a-e467f2e5ddc4",
"username": "[email protected]"
},
"role": "admin"
}
],
"links": {
"web": "https://app.opsgenie.com/teams/dashboard/b7acbc33-9853-4150-8a4b-10156d9408c8/main",
"api": "https://api.opsgenie.com/v2/teams/b7acbc33-9853-4150-8a4b-10156d9408c8"
}
},
"took": 0.029,
"requestId": "5a53826f-7864-4bf2-ada3-2979784d1e98"
}
2 changes: 1 addition & 1 deletion store/queries.sql
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ SELECT * FROM ext_schedules WHERE id = ?;
-- name: ListExtSchedules :many
SELECT * FROM ext_schedules;

-- name: ListExtSchedulesByPrefix :many
-- name: ListExtSchedulesLikeID :many
SELECT * FROM ext_schedules WHERE id LIKE ?;

-- name: InsertExtSchedule :exec
Expand Down
6 changes: 3 additions & 3 deletions store/queries.sql.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 6ae10a3

Please sign in to comment.