Skip to content

Commit

Permalink
test: Fix missing assertions for default result IDs in monitoringtest (
Browse files Browse the repository at this point in the history
…#15135)

Current CompareMonitoringInt is missing assertions on IDRequestCount, IDResponseCount, IDResponseErrorsCount, IDResponseValidCount. Add them back.
  • Loading branch information
carsonip authored Jan 6, 2025
1 parent 9e8c8c6 commit 71b8b00
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions internal/beater/monitoringtest/monitoring.go
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ func CompareMonitoringInt(
// allRequestResultIDs returns all registered request.ResultIDs (needs to be manually maintained)
func allRequestResultIDs() []request.ResultID {
var ids []request.ResultID
ids = append(ids, request.DefaultResultIDs...)
for k := range request.MapResultIDToStatus {
ids = append(ids, k)
}
Expand Down

0 comments on commit 71b8b00

Please sign in to comment.