Skip to content

Commit

Permalink
Update integrations/access/pagerduty/testlib/suite.go
Browse files Browse the repository at this point in the history
Co-authored-by: Tiago Silva <[email protected]>
  • Loading branch information
EdwardDowling and tigrato authored Oct 16, 2024
1 parent 47743e4 commit aae1085
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion integrations/access/pagerduty/testlib/suite.go
Original file line number Diff line number Diff line change
Expand Up @@ -465,7 +465,9 @@ func (s *PagerdutySuiteOSS) TestRecipientsFromAccessMonitoringRule() {
// Incident creation may happen before plugins Access Monitoring Rule cache
// has been updated with new rule. Retry until the new cache picks up the rule.
require.EventuallyWithT(t, func(t *assert.CollectT) {
require.Equal(t, lastRuleUpdated, ruleName)
mu.Lock()
require.Contains(t, collectedNames, ruleName)
mu.UnLock()
}, 3*time.Second, time.Millisecond*100, "new access monitoring rule did not begin applying")

// Test execution: create an access request
Expand Down

0 comments on commit aae1085

Please sign in to comment.